On Tue, Oct 26, 2021 at 5:47 AM Martin Liška <mli...@suse.cz> wrote: > > On 10/25/21 18:10, Eric Gallager wrote: > > On Mon, Oct 25, 2021 at 7:35 AM Martin Liška <mli...@suse.cz> wrote: > >> > >> Hello. > >> > >> The patch adds missing Makefile mostlyclean. > >> > >> Ready to be installed? > >> Thanks, > >> Martin > >> > > > > Generally the way the various "*clean" targets are arranged, in order > > of cleanliness, from least clean to most clean, is: > > mostlyclean > > clean > > distclean > > maintainer-clean > > ...with each target depending on the previous one in the order. So > > thus, instead of mostlyclean depending on clean, it'd be the other way > > around, with clean depending on mostlyclean. See how the gcc/ > > subdirectory does it, for example. See the "Standard Targets for > > Users" section of the GNU Coding Standards: > > https://www.gnu.org/prep/standards/html_node/Standard-Targets.html#Standard-Targets > > Thank you for the explanation. > > There's updated version of the patch. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed?
Hi, the patch looks ok to me, although I can't approve it... Who is the libcody maintainer, anyways? Nathan? Should that be listed in the MAINTAINERS file? Thanks, Eric Gallager > Thanks, > Martin > > > > >> PR other/102657 > >> > >> libcody/ChangeLog: > >> > >> * Makefile.in: Add mostlyclean Makefile target. > >> --- > >> libcody/Makefile.in | 4 +++- > >> 1 file changed, 3 insertions(+), 1 deletion(-) > >> > >> diff --git a/libcody/Makefile.in b/libcody/Makefile.in > >> index b8b45a2e310..d8f1e8216d4 100644 > >> --- a/libcody/Makefile.in > >> +++ b/libcody/Makefile.in > >> @@ -111,7 +111,7 @@ maintainer-clean:: distclean > >> clean:: > >> rm -f $(shell find $(srcdir) -name '*~') > >> > >> -.PHONY: all check clean distclean maintainer-clean > >> +.PHONY: all check clean distclean maintainer-clean mostlyclean > >> > >> CXXFLAGS/ := -I$(srcdir) > >> LIBCODY.O := buffer.o client.o fatal.o netclient.o netserver.o \ > >> @@ -127,6 +127,8 @@ clean:: > >> rm -f $(LIBCODY.O) $(LIBCODY.O:.o=.d) > >> rm -f libcody.a > >> > >> +mostlyclean: clean > >> + > >> CXXFLAGS/fatal.cc = -DSRCDIR='"$(srcdir)"' > >> > >> fatal.o: Makefile revision > >> -- > >> 2.33.1 > >>