[PATCH 2/3] Add the API entry to create a new build in a copr repository

Pierre-Yves Chibon pingou at pingoured.fr
Mon Feb 4 11:08:18 UTC 2013


On Mon, 2013-02-04 at 06:03 -0500, Bohuslav Kabrda wrote:
> ----- Original Message -----
> > On Sun, 2013-02-03 at 20:43 +0100, Pierre-Yves Chibon wrote:
> > > On Sat, 2013-02-02 at 17:53 +0100, Pierre-Yves Chibon wrote:
> > > > ---
> > > > 
> > > > Although this is looking fine especially since it is so close
> > > > from the code in
> > > > the normal controler, I have not tested it, so I guess it might
> > > > break, future
> > > > patch will come if so :)
> > > 
> > > I'm thinking, let's not integrate this patch yet and merge it only
> > > when
> > > I will have tested it.
> > > 
> > > This was more meant to give an idea on what it will look like.
> > > 
> > > I'll resend a patch for using the 'Authorization' header so that
> > > this
> > > one at least can be applied.
> > 
> > Ok the week-end has been long... I meant this for the commit
> > integrating
> > the 'build-copr' action in the CLI.
> > 
> Ok, I hope I applied everything that I should have applied. If not, give me a shout :)


Looks right to me, thanks for pushing all this :)

When I'll work again on the CLI (some evening this week I guess), I'll
need to pull on the server, is this fine for everyone ?


Thanks,
Pierre


More information about the copr-devel mailing list