Re: CVS commit: src/external/mit/expat/dist

[Matthias Scheler <tron%NetBSD.org@localhost>]

On Sat, Feb 11, 2012 at 10:40:03PM +0200, Alan Barrett wrote:
> On Sat, 11 Feb 2012, Matthias Scheler wrote:
> >Log Message:
> >Initial import of "expat" 2.0.1 into base:
> >This is James Clark's expat XML parser library in C. It is a stream
> >oriented parser that requires setting handlers to deal with the
> >structure that the parser discovers in the document.
> >
> >This is the first part of the fix for PR lib/40346.
> 
> Regardless of whether or not you got approval from releng, I think
> it's inappropriate to make major or controversial changes without
> public discussion.

As far as I can tell from reading the PR you were the only objecting to it.
Concerning your objection:

There is nothing wrong with the "apache" package in particuluar. As long
as "libexpat" in "/usr/X11R7" all packages using "libexpat" that get built
on a system with native X11 (e.g. the TNF package build machines) won't
work on systems without X11 installed. And there a lot of package (like
e.g. the "apache" package) where that is no good reason for that.

> The only semi-public discussion that I am aware
> of was in the comments on PR 40346, where there was not consensus
> that adding expat to base was a good idea.  One of the last comments
> there said that discussion on that issue should go to
> tech-userlevel, but there was no discussion in tech-userlevel.

Well, it is on the list of "Important bugs" for NetBSD 6.0.
So my assumption was that it should be fixed.

        Kind regards

-- 
Matthias Scheler                                  http://zhadum.org.uk/