Re: [PATCH v2 1/2] tests: add targets for building libvirt inside docker containers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@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@centos-7 MAKE_ARGS=check
  $ make ci-build@debian-9-cross-aarch64

would be a reasonable compromise between your approach and mine.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux