Re: [LTP] [PATCH 2/4] openposix: Makefile: Add the standard "maintainer-clean" target

["Ricardo B. Marli��re via ltp" <ltp@lists.linux.it>]

On Tue Apr 1, 2025 at 4:38 AM -03, Petr Vorel wrote:
> Hi Ricardo,
>
>> The include/mk/automake.mk file define the standard "maintainer-clean" and
>> "ac-maintainer-clean" targets and therein it calls the same targets under
>> each AUTOCONFED_SUBDIRS, but in the case of the openposix test suite, they
>> don't exist. Add them and also remove some dangling files in ac-clean.
>
>> This fixes the following error when calling `make maintainer-clean` from
>> the project's top level directory:
>
>> make[1]: Entering directory './testcases/open_posix_testsuite'
>> make[1]: *** No rule to make target 'ac-maintainer-clean'.  Stop.
>> make[1]: Leaving directory './testcases/open_posix_testsuite'
>> make: *** [./include/mk/automake.mk:68: ac-maintainer-clean] Error 2
>
> Reviewed-by: Petr Vorel <pvorel@suse.cz>
>
> Thanks for fixing dark corners of LTP build system.
>
> What a strange target name. Obviously it was meant to be run before release
> file, but it was not even called in Makefile.release which I removed in
> 1088d93d99.
>
> I was thinking whether we really need this target. Maybe it could be part of
> distclean.

Strange indeed, I was not aware of it until now... As I mentioned in the
cover letter, its a standard target. Both approaches make sense to me,
either we keep it or unify it with distclean. Not sure what others
prefer, but I would like to have a proper clean without relying on git.

Thank you,
-	Ricardo.


>
> Kind regards,
> Petr
>
>> Signed-off-by: Ricardo B. Marlière <rbm@suse.com>
>> ---
>>  testcases/open_posix_testsuite/Makefile | 10 +++++++++-
>>  1 file changed, 9 insertions(+), 1 deletion(-)
>
>> diff --git a/testcases/open_posix_testsuite/Makefile b/testcases/open_posix_testsuite/Makefile
>> index 8b4c8c0a24b833fa0fb21ce3e253629f358f1800..c0ccd499b8b9c93e45133f5b3f189eda7d09beb3 100644
>> --- a/testcases/open_posix_testsuite/Makefile
>> +++ b/testcases/open_posix_testsuite/Makefile
>> @@ -31,11 +31,16 @@ AUTOGENERATED_FILES = include/mk/config.mk
>
>>  .PHONY: ac-clean
>>  ac-clean: clean
>> +	$(RM) -rf autom4te.cache
>> +	$(RM) -f config.log config.status
>> +
>> +.PHONE: ac-maintainer-clean
>> +ac-maintainer-clean:
>> +	$(RM) -f configure
>
>>  .PHONY: clean
>>  clean:
>>  	$(RM) -f $(LOGFILE)*
>> -	$(RM) -f config.log config.status
>>  	@for dir in $(SUBDIRS) tools; do \
>>  		$(MAKE) -C $$dir clean >/dev/null; \
>>  	done
>> @@ -51,6 +56,9 @@ distclean-makefiles:
>>  		$(MAKE) -C $$dir $@; \
>>  	done
>
>> +.PHONY: maintainer-clean
>> +maintainer-clean: distclean-makefiles ac-maintainer-clean
>> +
>>  $(AUTOGENERATED_FILES): $(top_builddir)/config.status
>>  	$(SHELL) $^


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp