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 591897 - autoconf-2.64 compat: Don't use $(builddir)
autoconf-2.64 compat: Don't use $(builddir)
Status: RESOLVED FIXED
Product: gjs
Classification: Bindings
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gjs-maint
gjs-maint
Depends on:
Blocks:
 
 
Reported: 2009-08-15 13:00 UTC by Owen Taylor
Modified: 2009-08-24 21:56 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
autoconf-2.64 compat: Don't use $(builddir) (1.14 KB, patch)
2009-08-15 13:00 UTC, Owen Taylor
needs-work Details | Review
autoconf-2.64 compat: Don't use $(builddir) or $(abs_top_builddir) (6.71 KB, patch)
2009-08-15 23:05 UTC, Owen Taylor
none Details | Review
autoconf-2.64 compat: Don't use $(builddir) or $(abs_top_builddir) (6.71 KB, patch)
2009-08-15 23:41 UTC, Owen Taylor
none Details | Review

Description Owen Taylor 2009-08-15 13:00:47 UTC
$(builddir) is not a standard automake variable. With autoconf < 2.64
it ends up getting set in every Makefile.in to '.' (because autoconf
defines it), but that is no longer the case for 2.64.

Since $(builddir) was always '.', just use that instead.
Comment 1 Owen Taylor 2009-08-15 13:00:52 UTC
Created attachment 140838 [details] [review]
autoconf-2.64 compat: Don't use $(builddir)
Comment 2 Owen Taylor 2009-08-15 13:07:08 UTC
Patch is not complete, there is more usage of $(builddir) and also of $(abs_top_builddir) which is also not set for autoconf-2.64.
Comment 3 Owen Taylor 2009-08-15 23:05:40 UTC
Created attachment 140848 [details] [review]
autoconf-2.64 compat: Don't use $(builddir) or $(abs_top_builddir)

$(builddir) is not a standard automake variable. With autoconf < 2.64
it ends up getting set in every Makefile.in to '.' (because autoconf
defines it), but that is no longer the case for 2.64.

Since $(builddir) was always '.', just use that instead.

The $(abs_top_srcdir) $(abs_top_builddir) variables are also artifacts
of old autoconf rather than something intentional in automake. To
stop using those:

 - Set the paths in TESTS_ENVIRONMENT to just be relative to
   the top build directory - we never change from there when
   running tests.
 - Make gjs-unit determine the paths to various directories by
   determining them relative to the binary.
Comment 4 Owen Taylor 2009-08-15 23:41:46 UTC
Created attachment 140852 [details] [review]
autoconf-2.64 compat: Don't use $(builddir) or $(abs_top_builddir)

all the variable building right.

(This is actually the same as the last, testing out git-bz with new
bugzilla, sorry for noise)
Comment 5 Owen Taylor 2009-08-24 21:56:41 UTC
Pushed with a quick review from Lucas on IRC.