2009-06-21 23:25:15 Greg Stein napisał(a):
> 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.
So the design is broken.
Even libaprutil-1.so isn't linked against libraries printed by
`apu-1-config --libs`, so linking against these libraries would slow down
Subversion at run-time (ld.so would have to load attitional libraries).
$ apu-1-config --libs
-lldap -llber -llber -ldb-4.7 -lgdbm -lexpat
$ scanelf -qF '%F: %n' /usr/lib64/libaprutil-1.so
/usr/lib64/libaprutil-1.so: libexpat.so.1,libapr-1.so.0,libcrypt.so.1,libpthread.so.0,libc.so.6
> 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
> it.
Fixed in r38132.
--
Arfrever Frehtes Taifersar Arahesis
Received on 2009-06-22 00:47:57 CEST