Re: [PULL 05/30] Makefile: wrap ctags in quiet-command calls
On 1/19/21 3:27 PM, Daniel P. Berrangé wrote: > On Tue, Jan 19, 2021 at 03:24:56PM +0100, Philippe Mathieu-Daudé wrote: >> +Daniel >> >> On 1/19/21 11:00 AM, Alex Bennée wrote: >>> Philippe Mathieu-Daudé writes: On Fri, Jan 15, 2021 at 2:08 PM Alex Bennée wrote: > > For prettier output. > > Signed-off-by: Alex Bennée > Reviewed-by: Willian Rampazzo > Reviewed-by: Philippe Mathieu-Daudé > Message-Id: <20210114165730.31607-6-alex.ben...@linaro.org> > > diff --git a/Makefile b/Makefile > index 0c509a7704..bbab640b31 100644 > --- a/Makefile > +++ b/Makefile > @@ -250,8 +250,13 @@ find-src-path = find "$(SRC_PATH)/" -path > "$(SRC_PATH)/meson" -prune -o \( -name > > .PHONY: ctags > ctags: > - rm -f "$(SRC_PATH)/"tags > - $(find-src-path) -exec ctags -f "$(SRC_PATH)/"tags --append {} + > + $(call quiet-command, \ > + rm -f "$(SRC_PATH)/"tags, \ > + "CTAGS", "Remove old tags") > + $(call quiet-command, \ > + $(find-src-path) -exec ctags\ > + -f "$(SRC_PATH)/"tags --append {} +,\ > + "CTAGS", "Re-index $(SRC_PATH)") > > .PHONY: gtags > gtags: > -- > 2.20.1 > Build now fails if ctags is not installed: $ if test -n "$MAKE_CHECK_ARGS"; then make -j"$JOBS" $MAKE_CHECK_ARGS ; fi CTAGS Remove old tags CTAGS Re-index /builds/philmd/qemu find: 'ctags': No such file or directory find: 'ctags': No such file or directory find: 'ctags': No such file or directory make: *** [Makefile:254: ctags] Error 1 make: *** Waiting for unfinished jobs >>> >>> Wait what, how? Have you got ctags in your MAKE_CHECK_ARGS? How did it >>> not fail before? >>> >>> I suppose we could add checks for all the tooling in meson but it seems >>> a little overkill for a developer convenience. >> >> I figured out I was still using the Docker images generated after >> testing Daniel's 'start using libvirt-ci's "lcitool" for dockerfiles' >> series: >> https://www.mail-archive.com/qemu-devel@nongnu.org/msg772839.html >> >> Daniel, lcitool's ansible/vars/projects/qemu.yml apparently has >> a strong dependency on ctags. > > Huh ? It doesn't mention ctags at all. If it is present, it is only as > a side-effect of a dependancy from some other package we genuinely do need. Sorry, I meant QEMU as a strong dependency on ctags. I'll send a patch to lcitool. Regards, Phil.
Re: [PULL 05/30] Makefile: wrap ctags in quiet-command calls
On Tue, Jan 19, 2021 at 03:24:56PM +0100, Philippe Mathieu-Daudé wrote: > +Daniel > > On 1/19/21 11:00 AM, Alex Bennée wrote: > > Philippe Mathieu-Daudé writes: > >> On Fri, Jan 15, 2021 at 2:08 PM Alex Bennée wrote: > >>> > >>> For prettier output. > >>> > >>> Signed-off-by: Alex Bennée > >>> Reviewed-by: Willian Rampazzo > >>> Reviewed-by: Philippe Mathieu-Daudé > >>> Message-Id: <20210114165730.31607-6-alex.ben...@linaro.org> > >>> > >>> diff --git a/Makefile b/Makefile > >>> index 0c509a7704..bbab640b31 100644 > >>> --- a/Makefile > >>> +++ b/Makefile > >>> @@ -250,8 +250,13 @@ find-src-path = find "$(SRC_PATH)/" -path > >>> "$(SRC_PATH)/meson" -prune -o \( -name > >>> > >>> .PHONY: ctags > >>> ctags: > >>> - rm -f "$(SRC_PATH)/"tags > >>> - $(find-src-path) -exec ctags -f "$(SRC_PATH)/"tags --append {} + > >>> + $(call quiet-command, \ > >>> + rm -f "$(SRC_PATH)/"tags, \ > >>> + "CTAGS", "Remove old tags") > >>> + $(call quiet-command, \ > >>> + $(find-src-path) -exec ctags\ > >>> + -f "$(SRC_PATH)/"tags --append {} +,\ > >>> + "CTAGS", "Re-index $(SRC_PATH)") > >>> > >>> .PHONY: gtags > >>> gtags: > >>> -- > >>> 2.20.1 > >>> > >> > >> Build now fails if ctags is not installed: > >> > >> $ if test -n "$MAKE_CHECK_ARGS"; then make -j"$JOBS" $MAKE_CHECK_ARGS ; fi > >> CTAGS Remove old tags > >> CTAGS Re-index /builds/philmd/qemu > >> find: 'ctags': No such file or directory > >> find: 'ctags': No such file or directory > >> find: 'ctags': No such file or directory > >> make: *** [Makefile:254: ctags] Error 1 > >> make: *** Waiting for unfinished jobs > > > > Wait what, how? Have you got ctags in your MAKE_CHECK_ARGS? How did it > > not fail before? > > > > I suppose we could add checks for all the tooling in meson but it seems > > a little overkill for a developer convenience. > > I figured out I was still using the Docker images generated after > testing Daniel's 'start using libvirt-ci's "lcitool" for dockerfiles' > series: > https://www.mail-archive.com/qemu-devel@nongnu.org/msg772839.html > > Daniel, lcitool's ansible/vars/projects/qemu.yml apparently has > a strong dependency on ctags. Huh ? It doesn't mention ctags at all. If it is present, it is only as a side-effect of a dependancy from some other package we genuinely do need. Regards, Daniel -- |: https://berrange.com -o-https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o-https://fstop138.berrange.com :| |: https://entangle-photo.org-o-https://www.instagram.com/dberrange :|
Re: [PULL 05/30] Makefile: wrap ctags in quiet-command calls
+Daniel On 1/19/21 11:00 AM, Alex Bennée wrote: > Philippe Mathieu-Daudé writes: >> On Fri, Jan 15, 2021 at 2:08 PM Alex Bennée wrote: >>> >>> For prettier output. >>> >>> Signed-off-by: Alex Bennée >>> Reviewed-by: Willian Rampazzo >>> Reviewed-by: Philippe Mathieu-Daudé >>> Message-Id: <20210114165730.31607-6-alex.ben...@linaro.org> >>> >>> diff --git a/Makefile b/Makefile >>> index 0c509a7704..bbab640b31 100644 >>> --- a/Makefile >>> +++ b/Makefile >>> @@ -250,8 +250,13 @@ find-src-path = find "$(SRC_PATH)/" -path >>> "$(SRC_PATH)/meson" -prune -o \( -name >>> >>> .PHONY: ctags >>> ctags: >>> - rm -f "$(SRC_PATH)/"tags >>> - $(find-src-path) -exec ctags -f "$(SRC_PATH)/"tags --append {} + >>> + $(call quiet-command, \ >>> + rm -f "$(SRC_PATH)/"tags, \ >>> + "CTAGS", "Remove old tags") >>> + $(call quiet-command, \ >>> + $(find-src-path) -exec ctags\ >>> + -f "$(SRC_PATH)/"tags --append {} +,\ >>> + "CTAGS", "Re-index $(SRC_PATH)") >>> >>> .PHONY: gtags >>> gtags: >>> -- >>> 2.20.1 >>> >> >> Build now fails if ctags is not installed: >> >> $ if test -n "$MAKE_CHECK_ARGS"; then make -j"$JOBS" $MAKE_CHECK_ARGS ; fi >> CTAGS Remove old tags >> CTAGS Re-index /builds/philmd/qemu >> find: 'ctags': No such file or directory >> find: 'ctags': No such file or directory >> find: 'ctags': No such file or directory >> make: *** [Makefile:254: ctags] Error 1 >> make: *** Waiting for unfinished jobs > > Wait what, how? Have you got ctags in your MAKE_CHECK_ARGS? How did it > not fail before? > > I suppose we could add checks for all the tooling in meson but it seems > a little overkill for a developer convenience. I figured out I was still using the Docker images generated after testing Daniel's 'start using libvirt-ci's "lcitool" for dockerfiles' series: https://www.mail-archive.com/qemu-devel@nongnu.org/msg772839.html Daniel, lcitool's ansible/vars/projects/qemu.yml apparently has a strong dependency on ctags. Regards, Phil.
Re: [PULL 05/30] Makefile: wrap ctags in quiet-command calls
Philippe Mathieu-Daudé writes: > Hi Alex, > > On Fri, Jan 15, 2021 at 2:08 PM Alex Bennée wrote: >> >> For prettier output. >> >> Signed-off-by: Alex Bennée >> Reviewed-by: Willian Rampazzo >> Reviewed-by: Philippe Mathieu-Daudé >> Message-Id: <20210114165730.31607-6-alex.ben...@linaro.org> >> >> diff --git a/Makefile b/Makefile >> index 0c509a7704..bbab640b31 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -250,8 +250,13 @@ find-src-path = find "$(SRC_PATH)/" -path >> "$(SRC_PATH)/meson" -prune -o \( -name >> >> .PHONY: ctags >> ctags: >> - rm -f "$(SRC_PATH)/"tags >> - $(find-src-path) -exec ctags -f "$(SRC_PATH)/"tags --append {} + >> + $(call quiet-command, \ >> + rm -f "$(SRC_PATH)/"tags, \ >> + "CTAGS", "Remove old tags") >> + $(call quiet-command, \ >> + $(find-src-path) -exec ctags\ >> + -f "$(SRC_PATH)/"tags --append {} +,\ >> + "CTAGS", "Re-index $(SRC_PATH)") >> >> .PHONY: gtags >> gtags: >> -- >> 2.20.1 >> > > Build now fails if ctags is not installed: > > $ if test -n "$MAKE_CHECK_ARGS"; then make -j"$JOBS" $MAKE_CHECK_ARGS ; fi > CTAGS Remove old tags > CTAGS Re-index /builds/philmd/qemu > find: 'ctags': No such file or directory > find: 'ctags': No such file or directory > find: 'ctags': No such file or directory > make: *** [Makefile:254: ctags] Error 1 > make: *** Waiting for unfinished jobs Wait what, how? Have you got ctags in your MAKE_CHECK_ARGS? How did it not fail before? I suppose we could add checks for all the tooling in meson but it seems a little overkill for a developer convenience. -- Alex Bennée
Re: [PULL 05/30] Makefile: wrap ctags in quiet-command calls
Hi Alex, On Fri, Jan 15, 2021 at 2:08 PM Alex Bennée wrote: > > For prettier output. > > Signed-off-by: Alex Bennée > Reviewed-by: Willian Rampazzo > Reviewed-by: Philippe Mathieu-Daudé > Message-Id: <20210114165730.31607-6-alex.ben...@linaro.org> > > diff --git a/Makefile b/Makefile > index 0c509a7704..bbab640b31 100644 > --- a/Makefile > +++ b/Makefile > @@ -250,8 +250,13 @@ find-src-path = find "$(SRC_PATH)/" -path > "$(SRC_PATH)/meson" -prune -o \( -name > > .PHONY: ctags > ctags: > - rm -f "$(SRC_PATH)/"tags > - $(find-src-path) -exec ctags -f "$(SRC_PATH)/"tags --append {} + > + $(call quiet-command, \ > + rm -f "$(SRC_PATH)/"tags, \ > + "CTAGS", "Remove old tags") > + $(call quiet-command, \ > + $(find-src-path) -exec ctags\ > + -f "$(SRC_PATH)/"tags --append {} +,\ > + "CTAGS", "Re-index $(SRC_PATH)") > > .PHONY: gtags > gtags: > -- > 2.20.1 > Build now fails if ctags is not installed: $ if test -n "$MAKE_CHECK_ARGS"; then make -j"$JOBS" $MAKE_CHECK_ARGS ; fi CTAGS Remove old tags CTAGS Re-index /builds/philmd/qemu find: 'ctags': No such file or directory find: 'ctags': No such file or directory find: 'ctags': No such file or directory make: *** [Makefile:254: ctags] Error 1 make: *** Waiting for unfinished jobs
[PULL 05/30] Makefile: wrap ctags in quiet-command calls
For prettier output. Signed-off-by: Alex Bennée Reviewed-by: Willian Rampazzo Reviewed-by: Philippe Mathieu-Daudé Message-Id: <20210114165730.31607-6-alex.ben...@linaro.org> diff --git a/Makefile b/Makefile index 0c509a7704..bbab640b31 100644 --- a/Makefile +++ b/Makefile @@ -250,8 +250,13 @@ find-src-path = find "$(SRC_PATH)/" -path "$(SRC_PATH)/meson" -prune -o \( -name .PHONY: ctags ctags: - rm -f "$(SRC_PATH)/"tags - $(find-src-path) -exec ctags -f "$(SRC_PATH)/"tags --append {} + + $(call quiet-command, \ + rm -f "$(SRC_PATH)/"tags, \ + "CTAGS", "Remove old tags") + $(call quiet-command, \ + $(find-src-path) -exec ctags\ + -f "$(SRC_PATH)/"tags --append {} +,\ + "CTAGS", "Re-index $(SRC_PATH)") .PHONY: gtags gtags: -- 2.20.1