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