[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r38120 - in trunk: . build/ac-macros subversion/bindings/swig/perl/native

From: Greg Stein <gstein_at_gmail.com>
Date: Sun, 21 Jun 2009 23:25:15 +0200

On Sun, Jun 21, 2009 at 03:40, Arfrever Frehtes Taifersar
Arahesis<Arfrever.FTA_at_gmail.com> wrote:
>> If expat was being found just fine before r38120
> It was being found only because APR-Util was accidentally linked against Expat
> and apu-1-config was providing path to Expat directory which was used during
> building of APR-Util.

That is NOT an "accident". That is entirely by design.

APRUTIL is *supposed* to be linked against Expat on the system, and
SVN is *supposed* to use that fact to find Expat for its own usage.

>> Seems like a regression in functionality to me...
> Actually it's an enhancement :) .

If I'm required to pass more stuff on the command line, then it is a
*regression*. Period. That is the very definition of regression.

And if that regression causes a build break, then you better get this
fixed before somebody decides to revert it. The last time you broke
our build, we said "it is fair to revert when a change breaks the
build". You've now broken the build, so it is just a small matter of
time before somebody reverts this change. You have until then to fix


Received on 2009-06-21 23:25:41 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.