* [PATCH] fixincludes: add AC_CONFIG_MACRO_DIRS to configure.ac
@ 2024-04-30 2:24 Simon Marchi
2024-04-30 8:54 ` Christophe Lyon
0 siblings, 1 reply; 4+ messages in thread
From: Simon Marchi @ 2024-04-30 2:24 UTC (permalink / raw)
To: gcc-patches; +Cc: Simon Marchi
Add an "AC_CONFIG_MACRO_DIRS" call in configure.ac, with the same
directories as specified in "ACLOCAL_AMFLAGS", in Makefile.in.
This makes it possible to re-generate aclocal.m4 using "autoreconf".
---
fixincludes/configure | 1 +
fixincludes/configure.ac | 1 +
2 files changed, 2 insertions(+)
diff --git a/fixincludes/configure b/fixincludes/configure
index 662c94dc1120..7147c9a618aa 100755
--- a/fixincludes/configure
+++ b/fixincludes/configure
@@ -4627,6 +4627,7 @@ $as_echo "$ac_cv_path_SED" >&6; }
rm -f conftest.sed
+
# Figure out what compiler warnings we can enable.
# See config/warnings.m4 for details.
diff --git a/fixincludes/configure.ac b/fixincludes/configure.ac
index 4e78511d20fc..3d177699ebfa 100644
--- a/fixincludes/configure.ac
+++ b/fixincludes/configure.ac
@@ -7,6 +7,7 @@ AC_CANONICAL_SYSTEM
AC_PROG_CC
AC_USE_SYSTEM_EXTENSIONS
AC_PROG_SED
+AC_CONFIG_MACRO_DIRS([.. ../config])
# Figure out what compiler warnings we can enable.
# See config/warnings.m4 for details.
base-commit: 22b20ac6c6aead2d3f36c413a77dd0b80adfec39
--
2.44.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fixincludes: add AC_CONFIG_MACRO_DIRS to configure.ac
2024-04-30 2:24 [PATCH] fixincludes: add AC_CONFIG_MACRO_DIRS to configure.ac Simon Marchi
@ 2024-04-30 8:54 ` Christophe Lyon
2024-04-30 15:47 ` Simon Marchi
0 siblings, 1 reply; 4+ messages in thread
From: Christophe Lyon @ 2024-04-30 8:54 UTC (permalink / raw)
To: Simon Marchi; +Cc: gcc-patches
On Tue, 30 Apr 2024 at 04:25, Simon Marchi <simon.marchi@polymtl.ca> wrote:
>
> Add an "AC_CONFIG_MACRO_DIRS" call in configure.ac, with the same
> directories as specified in "ACLOCAL_AMFLAGS", in Makefile.in.
>
> This makes it possible to re-generate aclocal.m4 using "autoreconf".
Thanks, this LGTM, although like in your other patch, we need a
ChangeLog entry in the commit message.
Christophe
> ---
> fixincludes/configure | 1 +
> fixincludes/configure.ac | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/fixincludes/configure b/fixincludes/configure
> index 662c94dc1120..7147c9a618aa 100755
> --- a/fixincludes/configure
> +++ b/fixincludes/configure
> @@ -4627,6 +4627,7 @@ $as_echo "$ac_cv_path_SED" >&6; }
> rm -f conftest.sed
>
>
> +
> # Figure out what compiler warnings we can enable.
> # See config/warnings.m4 for details.
>
> diff --git a/fixincludes/configure.ac b/fixincludes/configure.ac
> index 4e78511d20fc..3d177699ebfa 100644
> --- a/fixincludes/configure.ac
> +++ b/fixincludes/configure.ac
> @@ -7,6 +7,7 @@ AC_CANONICAL_SYSTEM
> AC_PROG_CC
> AC_USE_SYSTEM_EXTENSIONS
> AC_PROG_SED
> +AC_CONFIG_MACRO_DIRS([.. ../config])
>
> # Figure out what compiler warnings we can enable.
> # See config/warnings.m4 for details.
>
> base-commit: 22b20ac6c6aead2d3f36c413a77dd0b80adfec39
> --
> 2.44.0
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fixincludes: add AC_CONFIG_MACRO_DIRS to configure.ac
2024-04-30 8:54 ` Christophe Lyon
@ 2024-04-30 15:47 ` Simon Marchi
2024-05-02 13:36 ` Christophe Lyon
0 siblings, 1 reply; 4+ messages in thread
From: Simon Marchi @ 2024-04-30 15:47 UTC (permalink / raw)
To: Christophe Lyon; +Cc: gcc-patches
On 4/30/24 4:54 AM, Christophe Lyon wrote:
> On Tue, 30 Apr 2024 at 04:25, Simon Marchi <simon.marchi@polymtl.ca> wrote:
>>
>> Add an "AC_CONFIG_MACRO_DIRS" call in configure.ac, with the same
>> directories as specified in "ACLOCAL_AMFLAGS", in Makefile.in.
>>
>> This makes it possible to re-generate aclocal.m4 using "autoreconf".
>
> Thanks, this LGTM, although like in your other patch, we need a
> ChangeLog entry in the commit message.
>
> Christophe
Here is the ChangeLog entry:
fixincludes/ChangeLog:
* configure.ac: Add AC_CONFIG_MACRO_DIRS.
* configure: Re-generate.
I don't have access to the gcc repo, so if/when this patch gets
approved, can you push it on my behalf?
Thanks,
Simon
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fixincludes: add AC_CONFIG_MACRO_DIRS to configure.ac
2024-04-30 15:47 ` Simon Marchi
@ 2024-05-02 13:36 ` Christophe Lyon
0 siblings, 0 replies; 4+ messages in thread
From: Christophe Lyon @ 2024-05-02 13:36 UTC (permalink / raw)
To: Simon Marchi; +Cc: gcc-patches
On Tue, 30 Apr 2024 at 17:47, Simon Marchi <simon.marchi@polymtl.ca> wrote:
>
> On 4/30/24 4:54 AM, Christophe Lyon wrote:
> > On Tue, 30 Apr 2024 at 04:25, Simon Marchi <simon.marchi@polymtl.ca> wrote:
> >>
> >> Add an "AC_CONFIG_MACRO_DIRS" call in configure.ac, with the same
> >> directories as specified in "ACLOCAL_AMFLAGS", in Makefile.in.
> >>
> >> This makes it possible to re-generate aclocal.m4 using "autoreconf".
> >
> > Thanks, this LGTM, although like in your other patch, we need a
> > ChangeLog entry in the commit message.
> >
> > Christophe
>
> Here is the ChangeLog entry:
>
> fixincludes/ChangeLog:
>
> * configure.ac: Add AC_CONFIG_MACRO_DIRS.
> * configure: Re-generate.
>
> I don't have access to the gcc repo, so if/when this patch gets
> approved, can you push it on my behalf?
Sure!
>
> Thanks,
>
> Simon
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-05-02 13:36 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-30 2:24 [PATCH] fixincludes: add AC_CONFIG_MACRO_DIRS to configure.ac Simon Marchi
2024-04-30 8:54 ` Christophe Lyon
2024-04-30 15:47 ` Simon Marchi
2024-05-02 13:36 ` Christophe Lyon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).