Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 242082 - eautoheader ignores AT_M4DIR or "-I" parameter and thus doesn't run autoheader
Summary: eautoheader ignores AT_M4DIR or "-I" parameter and thus doesn't run autoheader
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-14 17:58 UTC by Tiziano Müller (RETIRED)
Modified: 2014-11-21 09:31 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tiziano Müller (RETIRED) gentoo-dev 2008-10-14 17:58:36 UTC
I have this configure.ac and need to run eautoheader besides eaclocal.

AC_PREREQ(2.50)
AC_DEFUN([SMB_MODULE_DEFAULT], [echo -n ""])
AC_DEFUN([SMB_LIBRARY_ENABLE], [echo -n ""])
AC_DEFUN([SMB_ENABLE], [echo -n ""])
AC_INIT(tdb, 1.1.1)
AC_CONFIG_SRCDIR([common/tdb.c])
AC_CONFIG_HEADERS([include/config.h])
AC_LIBREPLACE_ALL_CHECKS
AC_LD_SONAMEFLAG
AC_LD_PICFLAG
AC_LD_SHLIBEXT
AC_LIBREPLACE_SHLD_FLAGS
m4_include(libtdb.m4)
AC_OUTPUT(Makefile tdb.pc)

The problem is that eautoreconf runs autotools_check_macro which in turn calls autoconf without passing along a "-I ../path" parameter. Autoconf then silently fails and eautoreconf doesn't run autoconf.
There are various ways to fix this. One is here: http://git.overlays.gentoo.org/gitweb/?p=dev/dev-zero.git;a=blob_plain;f=eclass/autotools.eclass;hb=HEAD
Comment 1 Doug Goldstein (RETIRED) gentoo-dev 2008-10-14 18:23:53 UTC
Do you have a specific diff to the current autotools.eclass... reading a whole file for the change isn't that helpful
Comment 2 Tiziano Müller (RETIRED) gentoo-dev 2008-10-20 15:06:11 UTC
I found another solution:
  M4PATH="../path" eautoreconf
works.

Question is now whether this is ok. If not: should AT_M4DIR be used or do we need a new AT_M4PATH variable?
Comment 3 SpanKY gentoo-dev 2008-10-26 07:17:11 UTC
i dont think having people set vars in their ebuilds is the way to go ... please do as Doug asked and post a patch that addresses this in the eclass transparently for everyone
Comment 4 SpanKY gentoo-dev 2014-11-21 09:31:13 UTC
i think this was fixed here:
http://sources.gentoo.org/eclass/autotools.eclass?r1=1.101&r2=1.102

current code appears to use the macrodir:
autotools_check_macro() {
...
        WANT_AUTOCONF="2.5" autoconf \
            $(autotools_m4dir_include) \
            ${ALL_AUTOTOOLS_MACROS[@]/#/--trace=} > ${trace_file} 2>/dev/null
...
}