[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 05/30] Makefile: wrap ctags in quiet-command calls
From: |
Daniel P . Berrangé |
Subject: |
Re: [PULL 05/30] Makefile: wrap ctags in quiet-command calls |
Date: |
Tue, 19 Jan 2021 14:27:00 +0000 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
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é <philmd@redhat.com> writes:
> >> On Fri, Jan 15, 2021 at 2:08 PM Alex Bennée <alex.bennee@linaro.org> wrote:
> >>>
> >>> For prettier output.
> >>>
> >>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> >>> Reviewed-by: Willian Rampazzo <willianr@redhat.com>
> >>> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> >>> Message-Id: <20210114165730.31607-6-alex.bennee@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 :|
- [PULL 00/30] testing, gdbstub and semihosting, Alex Bennée, 2021/01/15
- [PULL 01/30] tests/docker: Remove Debian 9 remnant lines, Alex Bennée, 2021/01/15
- [PULL 03/30] shippable.yml: Remove jobs duplicated on Gitlab-CI, Alex Bennée, 2021/01/15
- [PULL 02/30] Makefile: add GNU global tags support, Alex Bennée, 2021/01/15
- [PULL 06/30] Makefile: wrap etags in quiet-command calls, Alex Bennée, 2021/01/15
- [PULL 04/30] Add newline when generating Dockerfile, Alex Bennée, 2021/01/15
- [PULL 05/30] Makefile: wrap ctags in quiet-command calls, Alex Bennée, 2021/01/15
[PULL 17/30] gdbstub: add support to Xfer:auxv:read: packet, Alex Bennée, 2021/01/15
[PULL 25/30] semihosting: Support SYS_HEAPINFO when env->boot_info is not set, Alex Bennée, 2021/01/15
[PULL 13/30] test/guest-debug: echo QEMU command as well, Alex Bennée, 2021/01/15
[PULL 14/30] configure: gate our use of GDB to 8.3.1 or above, Alex Bennée, 2021/01/15
[PULL 21/30] target/arm: use official org.gnu.gdb.aarch64.sve layout for registers, Alex Bennée, 2021/01/15