public inbox for newlib@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] newlib: enable automake subdir-objects in all dirs
@ 2022-01-17 19:02 Mike Frysinger
  2022-01-18 10:12 ` Corinna Vinschen
  0 siblings, 1 reply; 2+ messages in thread
From: Mike Frysinger @ 2022-01-17 19:02 UTC (permalink / raw)
  To: newlib

Currently this is only enabled in the top-level as that's the only
place where it seemed to be used.  But the libc/sys/phoenix/ dir
also uses this functionality, but fails to explicitly enable it.
Automake workedaround it, but generated warnings.  Move the option
to NEWLIB_CONFIGURE so all dirs get it automatically iff they end
up using the option.  If they don't use the option, there's no
difference to the generated code.
---
 newlib/Makefile.am                  |   2 +-
 newlib/Makefile.in                  |   2 +-
 newlib/acinclude.m4                 |   2 +-
 newlib/libc/sys/phoenix/Makefile.in | 242 ++++++++++++++++------------
 4 files changed, 141 insertions(+), 107 deletions(-)

diff --git a/newlib/Makefile.am b/newlib/Makefile.am
index b2d363863380..6a28215d3620 100644
--- a/newlib/Makefile.am
+++ b/newlib/Makefile.am
@@ -1,6 +1,6 @@
 ## Process this file with automake to generate Makefile.in
 
-AUTOMAKE_OPTIONS = dejagnu subdir-objects
+AUTOMAKE_OPTIONS = dejagnu
 ACLOCAL_AMFLAGS = -I . -I .. -I ../config
 
 # Variables that will accumulate in subdirs.
diff --git a/newlib/acinclude.m4 b/newlib/acinclude.m4
index 2500ba6df17c..bdefd963542a 100644
--- a/newlib/acinclude.m4
+++ b/newlib/acinclude.m4
@@ -120,7 +120,7 @@ AC_SUBST(newlib_basedir)
 
 AC_CANONICAL_HOST
 
-AM_INIT_AUTOMAKE([foreign no-installinfo no-dependencies no-dist no-define 1.15.1])
+AM_INIT_AUTOMAKE([foreign no-installinfo no-dependencies no-dist no-define subdir-objects 1.15.1])
 AM_MAINTAINER_MODE()
 AM_SILENT_RULES(yes)
 
-- 
2.33.0


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] newlib: enable automake subdir-objects in all dirs
  2022-01-17 19:02 [PATCH] newlib: enable automake subdir-objects in all dirs Mike Frysinger
@ 2022-01-18 10:12 ` Corinna Vinschen
  0 siblings, 0 replies; 2+ messages in thread
From: Corinna Vinschen @ 2022-01-18 10:12 UTC (permalink / raw)
  To: newlib

On Jan 17 14:02, Mike Frysinger wrote:
> Currently this is only enabled in the top-level as that's the only
> place where it seemed to be used.  But the libc/sys/phoenix/ dir
> also uses this functionality, but fails to explicitly enable it.
> Automake workedaround it, but generated warnings.  Move the option
> to NEWLIB_CONFIGURE so all dirs get it automatically iff they end
> up using the option.  If they don't use the option, there's no
> difference to the generated code.
> ---
>  newlib/Makefile.am                  |   2 +-
>  newlib/Makefile.in                  |   2 +-
>  newlib/acinclude.m4                 |   2 +-
>  newlib/libc/sys/phoenix/Makefile.in | 242 ++++++++++++++++------------
>  4 files changed, 141 insertions(+), 107 deletions(-)

Thanks, please push.

Corinna


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-01-18 10:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-17 19:02 [PATCH] newlib: enable automake subdir-objects in all dirs Mike Frysinger
2022-01-18 10:12 ` Corinna Vinschen

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).