Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 28 Feb 2017 21:53:41 -0800 (PST)
From:      "Rodney W. Grimes" <freebsd-rwg@pdx.rh.CN85.dnsmgr.net>
To:        "Ngie Cooper (yaneurabeya)" <yaneurabeya@gmail.com>
Cc:        Ngie Cooper <ngie@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r314464 - head/usr.sbin/yppush
Message-ID:  <201703010553.v215rfIj054071@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <BEB56497-2C99-4BB4-A552-CF5F39034D27@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
-- Start of PGP signed section.
[ Charset UTF-8 unsupported, converting... ]
> 
> > On Feb 28, 2017, at 21:38, Rodney W. Grimes <freebsd-rwg@pdx.rh.CN85.dnsmgr.net> wrote:
> > 
> > Would it be possible to not do this as 1000 commits?
> 
> 	Yes, but I?m trying to do this in a way that allows me to backport all of the changes as easily as possible to ^/stable/10 (on the other side of the coin, I?ve run into dependent commits of other MFCs that required sweeping changes in order to make my changes possible).
> 	I do think your point is valid though and I can probably strike some middle ground with my changes. I think grouping it up by 2nd level directory (like I did with usr.bin/svn) makes the most sense.
> Thank you,

Yes, that is what I was thinking.  Note that I dont agree with this change,
but I have probably missed a discussion that is allowing it to move forward.
It makes the logfiles of make output now src tree possition dependent,
but I am probalby the only person in the universe that has ever run a
diff on the output of make world.

It may also be possible to split this into some phases to reduce some of
your issues, only do the {.CURDIR}/../../foo -> ${SRCTOP}/bar/foo in one pass and
some of the other fixes in another pass.


-- 
Rod Grimes                                                 rgrimes@freebsd.org



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201703010553.v215rfIj054071>