DragonFly BSD
DragonFly commits List (threaded) for 2005-12
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

Re: cvs commit: src/nrelease Makefile README src/nrelease/installer/etc/defaults pfi.conf src/nrelease/installer/usr/local/bin installer


From: Matthew Dillon <dillon@xxxxxxxxxxxxxxxxxxxx>
Date: Sun, 11 Dec 2005 11:00:34 -0800 (PST)

:>   +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@xxxxxxxxxxxxx>



[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]