Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


On Wed, Apr 06, 2011 at 03:59:34PM +0200, Petr Mladek wrote:

I have found that there already was a check for --hash-style. So, I used
your draft and itegrated it into the existing test, see
http://cgit.freedesktop.org/libreoffice/bootstrap/commit/?h=libreoffice-3-4&id=34e8665b5d3544c48086701d9f758ef0b9d900f8

Er, wrong move :-(

Please, do not feel offended that I did not use your patch as is. The
patch was nice but needed more love.

I'm well aware of that, I'm no autoconf specialist and it was a
work-in-progress.

Let me be offended on a different point ;-)

the test you have reused and the one I just wrote are for two different things.
We have two things here:

- the first check tested for --hash-style=gnu option support.
  ld supports it, result = yes

- the check I just wrote tested if binaries linked with the --hash-style=gnu
  can be run
  something in the system doesn't grok it, result = no

By integrating the two autoconf checks, we're back to a situation where
the build process happily links binaries with --hash-style=gnu and crashes
a few moments later trying to run them.

What I wanted to do is set a reasonable default of "gnu" or "sysv" for the
--with-hash-style option depending if the machine can run such binaries or
not.

-- 
Francois Tigeot

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.