After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 743148 - patch to enable HEAD install in Homebrew
patch to enable HEAD install in Homebrew
Status: RESOLVED OBSOLETE
Product: libxslt
Classification: Platform
Component: general
git master
Other Mac OS
: Normal enhancement
: ---
Assigned To: Daniel Veillard
libxml QA maintainers
Depends on:
Blocks:
 
 
Reported: 2015-01-18 23:04 UTC by Patrick Huck
Modified: 2021-07-05 11:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch using glibtoolize if libtoolize not found (808 bytes, application/octet-stream)
2015-01-18 23:04 UTC, Patrick Huck
  Details
using autoreconf instead of single calls (1.38 KB, patch)
2015-01-19 06:08 UTC, Patrick Huck
none Details | Review

Description Patrick Huck 2015-01-18 23:04:56 UTC
Created attachment 294820 [details]
patch using glibtoolize if libtoolize not found

Homebrew installs libtoolize as glibtoolize to not interfere with the one shipped with MacOS. However, libtoolize is needed to install libxslt from git master and provide a HEAD option to `brew install`. The attached patch to autogen.sh uses glibtoolize if libtoolize is not found. There might be a more elegant way of solving this, but this "quick fix" suffices to get the cutting edge libxslt compiled via Homebrew. The same patch has been included in the according Homebrew pull request at https://github.com/Homebrew/homebrew/pull/35986 but the maintainers would prefer for it to be merged upstream instead.
Comment 1 Daniel Macks 2015-01-19 03:36:52 UTC
Looking at other packages, some use a $LIBTOOLIZE env-var (if set) to override a PATH search (that way anyone can choose anything...users on systems that have other filenames, or to use an explicit full path to get something that is not first-in-PATH).

But more significantly, autogen.sh appears to run each of libtoolize, alocal, automake, autoconf explicitly after testing for each explicitly. Why not *just* use autoreconf, which does all of them automatically? It also supports the LIBTOOLIZE env var, as well as similar vars for all of the other autotool components.
Comment 2 Patrick Huck 2015-01-19 06:07:12 UTC
(In reply to comment #1)
> But more significantly, autogen.sh appears to run each of libtoolize, alocal,
> automake, autoconf explicitly after testing for each explicitly. Why not *just*
> use autoreconf, which does all of them automatically?
Indeed, but I tired to interfere with the maintainer's code as little as possible. Anyways, in the newly attached patch I replaced the single calls of libtoolize, aclocal, automake, and autoconf with one call of autoreconf. After testing, it still seems to compile fine.
Comment 3 Patrick Huck 2015-01-19 06:08:01 UTC
Created attachment 294841 [details] [review]
using autoreconf instead of single calls
Comment 4 GNOME Infrastructure Team 2021-07-05 11:01:21 UTC
GNOME is going to shut down bugzilla.gnome.org in favor of gitlab.gnome.org.
As part of that, we are mass-closing older open tickets in bugzilla.gnome.org
which have not seen updates for a longer time (resources are unfortunately
quite limited so not every ticket can get handled).

If you can still reproduce the situation described in this ticket in a recent
and supported software version, then please follow
  https://wiki.gnome.org/GettingInTouch/BugReportingGuidelines
and create a new ticket at
  https://gitlab.gnome.org/GNOME/libxslt/-/issues/

Thank you for your understanding and your help.