[libvirt] [PATCH v2 1/2] tests: add targets for building libvirt inside docker containers
Andrea Bolognani
abologna at redhat.com
Mon Mar 18 17:03:21 UTC 2019
On Fri, 2019-03-15 at 17:20 +0100, Andrea Bolognani wrote:
> On Fri, 2019-03-15 at 15:06 +0000, Daniel P. Berrangé wrote:
> > > > + @echo "Available targets:"
> > > > + @echo
> > > > + @echo " cibuild-\$$IMAGE - run a default 'make'"
> > > > + @echo " cicheck-\$$IMAGE - run a 'make check'"
> > > > + @echo " cishell-\$$IMAGE - run an interactive shell"
> > >
> > > Just a thought: instead of
> > >
> > > make ci-build-centos-7 MAKE_ARGS=check
> > >
> > > and in the future
> > >
> > > make ci-build-debian-9-cross-aarch64
> > >
> > > would it make sense to have something like
> > >
> > > make ci-build OS=centos-7 MAKE_ARGS=check
> > > make ci-build OS=debian-9 CROSS=aarch64
> > >
> > > instead? A bit more typing, perhaps, but it looks kinda better
> > > in my opinion, with the variable parts clearly presented as such...
> >
> > I rather prefer the more concise target names - I don't think it
> > really adds anything to use variables
>
> I disagree on concise: they're definitely shorter, but that's
> because all the information is squished together, which makes it
> harder to parse at a glance.
>
> When naming Docker images we don't have much of a choice, because
> we have pretty much the same restrictions as when naming files, but
> that's not the case here so we could do better...
I see QEMU uses
$ make docker
...
docker-TEST at IMAGE: Run "TEST" in container "IMAGE".
Note: "TEST" is one of the listed test name,
or a script name under $QEMU_SRC/tests/docker/;
"IMAGE" is one of the listed container name."
I think adopting that convention, thus ending up with
$ make ci-build at centos-7 MAKE_ARGS=check
$ make ci-build at debian-9-cross-aarch64
would be a reasonable compromise between your approach and mine.
--
Andrea Bolognani / Red Hat / Virtualization
More information about the libvir-list
mailing list