There is no reason to run 'true' for empty targets
Details
- Reviewers
mat - Group Reviewers
portmgr - Commits
- rP438106: Stop creating stubs targets calling /usr/bin/true
Diff Detail
- Lint
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 8631 Build 8957: arc lint + arc unit
Event Timeline
Mk/bsd.port.mk | ||
---|---|---|
2785–2792 | Maybe change that and put the ignorelist-verbose: taget before the first .if test, and remove the .else. | |
2871–2872 | Maybe put the .if defined(NO_CHECKSUM) around the call to checksum.sh and remove this. | |
2878–2881 | Maybe move this to the test: definition at the end. | |
2890–2891 | Mmm, NO_DESCRIBE is not currently in use, and I don't feel like it is a good idea anyway, many remove it. | |
2901–2902 | Maybe just remove this, if it is created, it goes in the targets pipeline, if not, it does not. | |
2907–2908 | GNOME_OPTION_MSG was removed ages ago, maybe just remove all this. | |
3298–3300 | Why not just remove the .elif ? If it does not exist, it just won't be added to the targets pipeline. | |
3407–3409 | Nothing currently uses install-mtree, time to go ? | |
3678–3680 | maybe change that to create a dependency below so that the target is not created if it is not needed, like .if target(pre-distclean) distclean: pre-distclean .endif |
Mk/bsd.port.mk | ||
---|---|---|
3678–3680 | It was my first reaction, yes, I'll have a look at if it was ever used. |
Mk/bsd.port.mk | ||
---|---|---|
3678–3680 | I think we should just keep this removal for another patchset :) |
Looks good to me, it might need an exp-run, I'll leave the other portmgr members look at it here.