Opened 11 years ago

Last modified 11 years ago

#38579 new enhancement

RFE: make the aclocal.m4 file in base safe to regenerate with `aclocal`

Reported by: cooljeanius (Eric Gallager) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: base Version:
Keywords: haspatch Cc:
Port:

Description

Regenerating aclocal.m4 with aclocal is a standard part of an autotools workflow. Automake tells you to do it, libtoolize tells you to do it, gettextize tells you to do it, and I'm sure there are other tools that tell you to do it, too, that I'm just not thinking of now. However, the aclocal.m4 file in base contains some hard-coded macros in it, meaning that if you run aclocal, you're going to overwrite them and then be missing some of the macros you need. This is pretty simple to fix: you could rename the current aclocal.m4 file to acinclude.m4, or move it into the m4/ directory in base, or both. After doing that, you can regenerate the aclocal.m4 file with aclocal -I m4.

Attachments (2)

patch-base.diff (42.8 KB) - added by cooljeanius (Eric Gallager) 11 years ago.
patch to apply to the base directory
patch-m4-acinclude.m4.diff (3.5 KB) - added by cooljeanius (Eric Gallager) 11 years ago.
patch to apply to aclocal.m4 after moving it to m4/acinclude.m4

Download all attachments as: .zip

Change History (7)

comment:1 Changed 11 years ago by cooljeanius (Eric Gallager)

Also I think I brought this up in IRC once...

comment:2 Changed 11 years ago by cooljeanius (Eric Gallager)

Actually this is a little more complicated than that; some "underquoted macro" problems have to be fixed, too, for it to be safe to run aclocal. I'll attach a diff next. (After that the "haspatch" keyword can be added)

Changed 11 years ago by cooljeanius (Eric Gallager)

Attachment: patch-base.diff added

patch to apply to the base directory

comment:3 Changed 11 years ago by cooljeanius (Eric Gallager)

Hm, it doesn't seem to include what the actual contents of acinclude.m4 are, it just says that it's "Only in ./m4"... guess I should attach a separate diff for that one...

Changed 11 years ago by cooljeanius (Eric Gallager)

Attachment: patch-m4-acinclude.m4.diff added

patch to apply to aclocal.m4 after moving it to m4/acinclude.m4

comment:4 Changed 11 years ago by cooljeanius (Eric Gallager)

This can be marked with the keyword "haspatch"

comment:5 Changed 11 years ago by mf2k (Frank Schima)

Keywords: haspatch added
Version: 2.1.3
Note: See TracTickets for help on using tickets.