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

Bohuslav Kabrda bkabrda at redhat.com
Mon Feb 4 11:57:45 UTC 2013


----- Original Message -----
> 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

I'll be pulling today since I want to finally get the fulltext search working. It'll require some DB updates etc, so please wait until I pull. I'll let you know.
Thanks.

-- 
Regards,
Bohuslav "Slavek" Kabrda.


More information about the copr-devel mailing list