Page MenuHomeFreeBSD

Stop using DO_NADA when not needed
ClosedPublic

Authored by bapt on Apr 8 2017, 7:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 18 2024, 12:56 AM
Unknown Object (File)
Feb 18 2024, 12:55 AM
Unknown Object (File)
Feb 18 2024, 12:55 AM
Unknown Object (File)
Feb 18 2024, 12:55 AM
Unknown Object (File)
Feb 18 2024, 12:55 AM
Unknown Object (File)
Dec 22 2023, 9:58 PM
Unknown Object (File)
Sep 1 2023, 10:51 PM
Unknown Object (File)
Jul 13 2023, 6:31 AM
Subscribers

Details

Summary

There is no reason to run 'true' for empty targets

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 8631
Build 8957: arc lint + arc unit

Event Timeline

mat added inline comments.
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
bapt marked 2 inline comments as done.Apr 9 2017, 11:05 AM
bapt added inline comments.
Mk/bsd.port.mk
2901–2902

This one must exist to be hooked on

3407–3409

Can't :) added a comment :)

3678–3680

I think we should just nuke pre-distclean

bapt marked an inline comment as done.

Fix bad rebase

bapt marked 2 inline comments as done.Apr 9 2017, 11:10 AM
bapt marked an inline comment as done.Apr 9 2017, 11:21 AM
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 :)

Mk/bsd.port.mk
2901–2902

Mmmm, it does not need to be hooked in, it's in _SANITY_SEQ, so, I don't understand.

3678–3680

So, I asked git log -Spre-distclean -p, it was added in 1996 and never, ever used, so, yeah, remove it :-)

Remove more unused targets

bapt marked 2 inline comments as done.Apr 9 2017, 2:32 PM

Looks good to me, it might need an exp-run, I'll leave the other portmgr members look at it here.

This revision is now accepted and ready to land.Apr 9 2017, 3:00 PM
This revision was automatically updated to reflect the committed changes.