[Mesa-dev] [PATCH 2/3] panfrost: Make the gitlab-ci.yml file more robust
Rohan Garg
rohan.garg at collabora.com
Fri Jun 21 10:37:07 UTC 2019
On Friday, 21 June 2019 07:40:02 CEST Tomeu Vizoso wrote:
> On 6/20/19 6:33 PM, Rohan Garg wrote:
> >> Not sure I understand how this works. Isn't this going to cause
> >> unnecessary container builds?
> >>
> >> And, why cannot developers just add whatever they want to DEBIAN_TAG
> >> to that end?
> >
> > This will spin a container for each branch, yes, though I think that is
> > how it should be.
>
> Well, things are like this right now precisely because people wanted to
> avoid what you are proposing. I would be quite sad if every time I push a
> new branch I had to wait for the container to be rebuilt. Also note that
> this is how other projects based on wayland/ci-templates work, including
> the rest of mesa.
>
> > The patch allows for someone working in a branch to
> > ( if the need be ) customize their containers for their branch.
>
> As I said, you can easily do that atm by just updating DEBIAN_TAG.
>
> > To that extent, I've simply enabled the CI to auto generate a DEBIAN_TAG
> > which depends on the branch name so that a developer doesn't have to
> > change
> > the DEBIAN_TAG themselves.
>
> But, why is that such a problem?
>
Fair enough, I'll rework the patch to drop the modified DEBIAN_TAG then.
Cheers
Rohan Garg
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20190621/2322fb0f/attachment.sig>
More information about the mesa-dev
mailing list