[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: CVS commit: pkgsrc
On Mon, May 01, 2017 at 07:17:10AM +0100, Jonathan Perkin wrote:
> But now you've broken over 250 dependencies via strigi and others
> because cppunit installs headers which require C++11.
> I'd rather we had more testing/review of the CXX_REQD stuff I posted
> so we can fix this properly rather than back-and-forth of breakages.
I agree that we need more testing.
The reason I suggested this type of change is that it's hard to keep
track of changes like buildlink introducing -std=c++11.
Recently a similar change was made to icu introducing -std=c++11 (since
reverted) and this broke harfbuzz on netbsd-6, as its compiler is too
old to recognise the flag. it took some effort to understand where the
flag is coming from (I didn't notice the icu buildlink being included
via options.mk, and thought I checked every dependency, and now need to
check dependencies of dependencies).
It's a slight duplication but I think the end result is much simpler,
and we should avoid introducing potentially breaking flags via
Main Index |
Thread Index |