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 599865 - bootstrap fails when top_srcdir is an absolute path
bootstrap fails when top_srcdir is an absolute path
Status: RESOLVED WONTFIX
Product: gnome-doc-utils
Classification: Deprecated
Component: build utils
unspecified
Other All
: Normal normal
: ---
Assigned To: gnome-doc-utils maintainers
gnome-doc-utils maintainers
gnome[unmaintained]
: 667579 696993 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-10-28 00:12 UTC by Yaakov Selkowitz
Modified: 2020-03-03 18:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
use abs_top_{build,src}dir (589 bytes, patch)
2009-10-28 00:12 UTC, Yaakov Selkowitz
none Details | Review
0001-Fix-building-out-of-tree.patch (1.08 KB, patch)
2012-02-05 16:32 UTC, Hib Eris
none Details | Review

Description Yaakov Selkowitz 2009-10-28 00:12:47 UTC
Created attachment 146392 [details] [review]
use abs_top_{build,src}dir

Currently in bootstrap.make:

_xml2po = PYTHONPATH="$(shell pwd)/$(top_builddir)/xml2po:$(shell
pwd)/$(top_srcdir)/xml2po:$(PYTHONPATH)" "$(shell
pwd)/$(top_builddir)/xml2po/xml2po"

But if $top_srcdir is itself an absolute path, prepending $PWD to it is
incorrect and breaks the build.

I'm attaching a patch for 0.18.0 to use $abs_top_{build,src}dir to avoid any
such problems.
Comment 1 Christian Persch 2009-10-28 12:35:28 UTC
Bug 591897 seems to say that one shouldn't use abs_top_builddir?
Comment 2 Yaakov Selkowitz 2009-10-28 18:10:45 UTC
(In reply to comment #1)
> Bug 591897 seems to say that one shouldn't use abs_top_builddir?

The autoconf 2.64 manual[1] documents it, so I don't see why not.

[1] http://www.gnu.org/software/autoconf/manual/autoconf.html#Preset-Output-Variables
Comment 3 Yaakov Selkowitz 2011-02-01 02:55:25 UTC
abs_top_builddir is still documented in 2.68 with no indication that it is deprecated.  I believe that the statement to the contrary in bug 591897 was erroneous.
Comment 4 Hib Eris 2012-02-05 15:55:37 UTC
*** Bug 667579 has been marked as a duplicate of this bug. ***
Comment 5 Hib Eris 2012-02-05 16:02:19 UTC
(In reply to comment #3)
> abs_top_builddir is still documented in 2.68 with no indication that it is
> deprecated.  I believe that the statement to the contrary in bug 591897 was
> erroneous.

It seems that the issue in bug 591897 is not with autoconf 2.68, but with automake 1.9. Autoconf always sets abs_top_builddir and abs_top_srcdir, but automake 1.9 does not include those settings in the Makefile.in's it generates.
Comment 6 Hib Eris 2012-02-05 16:32:57 UTC
Created attachment 206852 [details] [review]
0001-Fix-building-out-of-tree.patch

Updated patch to not depend on automake to set abs_top_{build,src}dir.
Comment 7 Hib Eris 2012-03-06 10:50:21 UTC
ping!

Can someone please review my patch?
Comment 8 Yaakov Selkowitz 2012-04-16 21:54:51 UTC
Is there a reason that you need to support boostrapping with automake < 1.11?
Comment 9 Hib Eris 2012-04-17 06:22:37 UTC
(In reply to comment #8)
> Is there a reason that you need to support boostrapping with automake < 1.11?

Why are you asking this? My patch https://bugzilla.gnome.org/attachment.cgi?id=206852 works with all automake versions.

From my patch:
# automake does not guarantee setting the abs_top_srcdir and 
# abs_top_buildir variables (and automake 1.9 in fact does not set them),
# so we make sure we get them from autoconf.

So, even if we do not want to support bootstrapping with automake < 1.11, we should handle bootstrapping with a future automake 1.12 that might not set the  internal variables abs_top{build,src}dir. That is why my patch uses the autoconf variables instead of automake's.

Now, if only we could get someone to actually apply 
https://bugzilla.gnome.org/attachment.cgi?id=206852
this bug could finally be closed!
Comment 10 Gilles Dartiguelongue 2013-05-10 11:13:03 UTC
Here is the patch we had to apply in Gentoo on 0.20.10 in order to support out-of-tree build.

This is required for us to support multiple python installation at once.

http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/app-text/gnome-doc-utils/files/gnome-doc-utils-0.20.10-fix-out-of-tree-build.patch
Comment 11 Gilles Dartiguelongue 2013-05-10 11:21:21 UTC
*** Bug 696993 has been marked as a duplicate of this bug. ***
Comment 12 André Klapper 2020-03-03 18:34:47 UTC
gnome-doc-utils has been superseded by yelp-xsl, yelp-tools, and itstool.

gnome-doc-utils will not see any further development, hence closing as WONTFIX.

See https://gitlab.gnome.org/Infrastructure/Infrastructure/issues/255