diff mbox

[xf86-video-intel,1/3] build: Make autoreconf honour ACLOCAL_FLAGS

Message ID 1358518388-3447-1-git-send-email-damien.lespiau@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lespiau, Damien Jan. 18, 2013, 2:13 p.m. UTC
When running autoreconf, it's possible to give flags to the underlying
aclocal by declaring a ACLOCAL_AMFLAGS variable in the top level
Makefile.am.

Putting ${ACLOCAL_FLAGS} there allows the user to set an environment
variable up before running autogen.sh and pull in the right directories
to look for m4 macros, say an up-to-date version of the xorg-util macros.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 Makefile.am |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Comments

Chris Wilson Feb. 4, 2013, 8:40 p.m. UTC | #1
On Fri, Jan 18, 2013 at 02:13:06PM +0000, Damien Lespiau wrote:
> When running autoreconf, it's possible to give flags to the underlying
> aclocal by declaring a ACLOCAL_AMFLAGS variable in the top level
> Makefile.am.
> 
> Putting ${ACLOCAL_FLAGS} there allows the user to set an environment
> variable up before running autogen.sh and pull in the right directories
> to look for m4 macros, say an up-to-date version of the xorg-util macros.
> 
> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>

Thanks for the ping, now pushed.

Had a momentarily doubt about 2/3 as it look like it could trigger
spurious rebuilds of the asm, but then remembered that you would have
had to ./configure --enable-gen4asm in the first place...
-Chris
diff mbox

Patch

diff --git a/Makefile.am b/Makefile.am
index b3d37b2..5001674 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -18,6 +18,7 @@ 
 #  IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
 #  CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
+ACLOCAL_AMFLAGS = ${ACLOCAL_FLAGS}
 
 SUBDIRS = man