cvs commit: src/nrelease Makefile README src/nrelease/installer/etc/defaults pfi.conf src/nrelease/installer/usr/local/bin installer
Matthew Dillon
dillon at apollo.backplane.com
Sun Dec 11 11:01:43 PST 2005
:> +all: release
:> +
:>
:> This makes buildworld kick `release' target in nrelease(because of SUBDIR).
:> If it's not intended, can you change it to something like this?
:>
:> .MAIN: release
:
:That would achieve the desired effect. Two questions:
:(a) Do we want to have a default target or not? If not, I'd change it
:to "all:".
No, I don't want there to be a default target, unless all it does is
print out directions on available targets and what they do. It's too
easy for a new DragonFly developer to make a mistake and wind up with
an uninstaller ISO. That might not matter for those of us with fast
machines, but it's really annoying for someone who doesn't. That's
why I don't want to use the make.conf mechanism to setup release builds.
:(b) Why don't we add a cleankernel target to the top-level, which would
:remove the need for the recursion into nrelease. Alternatively, query
:the top-level Makefile for the kernel directory.
:
:Joerg
Maybe. But if you make that change do a complete test to make sure
that both buildworld and make installer_release work with a completely
cleaned out /usr/obj before committing.
-Matt
Matthew Dillon
<dillon at xxxxxxxxxxxxx>
More information about the Commits
mailing list