Message: previous - next
Month: June 2012

Re: [trinity-devel] FTBFS tqca-tls (x86_64) can't find TQt3 (i686 builds?)

From: "David C. Rankin" <drankinatty@...>
Date: Sun, 24 Jun 2012 10:32:14 -0500
On 06/23/2012 02:52 AM, Darrell Anderson wrote:
> Commit a5dfd481?
> tqca-tls builds here on 64-bit, but Slackware uses lib${LIBSUFFIX}, as do many other distros.
> I don't know how to make any of that work on both types of directories.
> Darrell

Thanks Darrell,

  How do we fix or undo the commit that breaks building for distros that use
../lib on x86_64??



  Why was this change made without regard to accommodating the prior build
behavior? That is one thing we have to avoid doing. I recall the lib64
discussion and I specifically noted that Arch and others use /lib regardless. We
can't simply implement changes that break build behavior for some and not others
and just -- leave it to the other guy to sort it out.

  Issues like this need to be solved TDE wide, not just until it "works for me."


  That being said, I think this in this case, it is just broken logic leading to
the build failure. Somewhere in this range from grep:

configure:      TQString expandLibs(const TQString &lib)
configure:              return TQString("-L") + lib;
configure:      bool findLibrary(const TQString &name, TQString *lib)
configure:                      *lib = "";
configure:              if(checkLibrary("/usr/local/lib", name)) {
configure:                      *lib = "/usr/local/lib";
configure:              if(checkLibrary("/usr/local/lib64", name)) {
configure:                      *lib = "/usr/local/lib64";
configure:      if [ ! -f "$QTDIR/lib/" ]; then
configure:              if [ ! -f "$QTDIR/lib64/" ]; then
configure:                      echo " installed in $QTDIR/lib/"
configure:                      echo "or $QTDIR/lib64/."

  If somebody knows how to fix this, please let us know so that a patch can be
tested and pushed. configure logic and undoing git changes has never been a
strong point so I would welcome suggestions here.

David C. Rankin, J.D.,P.E.