Re: [PATCH] Cleanup Makefile.am CLEANFILES definitions

2024-11-23 Thread Michael Pratt
Hi Mark, On 11/21/24 7:44 PM, Mark Wielaard wrote: > Hi Michael, > > On Tue, Nov 19, 2024 at 04:18:17AM +, Michael Pratt wrote: > > > > > This is because config/eu.am defines a default CLEANFILES. So those > > > Makefile.am files should add to CLEANFILES with += instead of > > > r

Re: [PATCH] Cleanup Makefile.am CLEANFILES definitions

2024-11-21 Thread Mark Wielaard
Hi Michael, On Tue, Nov 19, 2024 at 04:18:17AM +, Michael Pratt wrote: > > > This is because config/eu.am defines a default CLEANFILES. So those > > Makefile.am files should add to CLEANFILES with += instead of > > redefining CLEANFILES with =. > > May I suggest to instead simply use > "D

Re: [PATCH] Cleanup Makefile.am CLEANFILES definitions

2024-11-18 Thread Michael Pratt
Hi Mark, > Since b2f225d6bff8 ("Consolidate and add files to clean target variables") > autoreconf (automake) produces these warnings: oops, sorry... > This is because config/eu.am defines a default CLEANFILES. So those > Makefile.am files should add to CLEANFILES with += instead of > redef

Re: [PATCH] Cleanup Makefile.am CLEANFILES definitions

2024-11-18 Thread Aaron Merey
On Mon, Nov 18, 2024 at 1:12 PM Mark Wielaard wrote: > > Since b2f225d6bff8 ("Consolidate and add files to clean target variables") > autoreconf (automake) produces these warnings: > > debuginfod/Makefile.am:130: warning: CLEANFILES multiply defined in condition > TRUE ... > config/eu.am:138: ...

[PATCH] Cleanup Makefile.am CLEANFILES definitions

2024-11-18 Thread Mark Wielaard
Since b2f225d6bff8 ("Consolidate and add files to clean target variables") autoreconf (automake) produces these warnings: debuginfod/Makefile.am:130: warning: CLEANFILES multiply defined in condition TRUE ... config/eu.am:138: ... 'CLEANFILES' previously defined here debuginfod/Makefile.am:32: