Page MenuHomeFreeBSD

Simplify distrib-dirs a bit
ClosedPublic

Authored by bapt on Oct 16 2015, 6:21 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 2, 9:52 AM
Unknown Object (File)
Mon, Dec 2, 9:52 AM
Unknown Object (File)
Mon, Dec 2, 9:52 AM
Unknown Object (File)
Mon, Dec 2, 9:52 AM
Unknown Object (File)
Mon, Dec 2, 12:43 AM
Unknown Object (File)
Sat, Nov 23, 3:05 PM
Unknown Object (File)
Sat, Nov 23, 11:59 AM
Unknown Object (File)
Sep 23 2024, 6:04 PM
Subscribers
None

Details

Summary

By using bmake's multivariable for loops we can simplify a bit

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bapt retitled this revision from to Simplify distrib-dirs a bit.
bapt updated this object.
bapt edited the test plan for this revision. (Show Details)
bapt added reviewers: bdrewery, imp.
bdrewery edited edge metadata.
bdrewery added inline comments.
etc/Makefile
367 โ†—(On Diff #9442)

Adding DESTDIR in here early seems wrong. It looks like it will cause line 369 to go from older {DISTBASE}{_d} to newer {DISTBASE}{DESTDIR}{_d} rather than expected {DESTDIR}{DISTBASE}{_d} done later. It also messes with the sed line for the metalog and adds in {DESTDIR} where it wasn't before.

This revision now requires changes to proceed.Oct 16 2015, 2:28 PM
etc/Makefile
367 โ†—(On Diff #9442)

Good catch oups I'll fix

I'm unclear on our position on bmake in head. The meta mode merge pretty much nailed fmake in the coffin, but imp seemed to think we could or should still support fmake in some situations. I forget what he said exactly.

I would prefer we only support bmake. I think given it is the default make in 10 there is not much reason to support fmake at this point.

I'm unclear on our position on bmake in head. The meta mode merge pretty much nailed fmake in the coffin, but imp seemed to think we could or should still support fmake in some situations. I forget what he said exactly.

I would prefer we only support bmake. I think given it is the default make in 10 there is not much reason to support fmake at this point.

I am not suggesting your change shouldn't go in, there's no reason it shouldn't IMO as we use the same .for X Y elsewhere now and bmakeisms all over.

On head we do only support bmake, on 10 we do support both

I forgot that review :)
Let's finish it

bapt marked 2 inline comments as done.Nov 12 2017, 3:35 PM
This revision is now accepted and ready to land.Jun 19 2018, 6:28 PM
This revision was automatically updated to reflect the committed changes.