Re: Future changes in the new package and new branch processes

2014-10-13 Thread Pierre-Yves Chibon
On Mon, Sep 08, 2014 at 11:55:06AM +0200, Pierre-Yves Chibon wrote: On Mon, Sep 08, 2014 at 11:31:58AM +0200, Tomas Tomecek wrote: Quoting Pierre-Yves Chibon (2014-09-05 17:08:39) New procedure = * packager opens a review-request on bugzilla * reviewer sets the

Re: Future changes in the new package and new branch processes

2014-09-09 Thread Tomas Tomecek
Quoting Pierre-Yves Chibon (2014-09-08 11:55:06) On Mon, Sep 08, 2014 at 11:31:58AM +0200, Tomas Tomecek wrote: Quoting Pierre-Yves Chibon (2014-09-05 17:08:39) New procedure = * packager opens a review-request on bugzilla * reviewer sets the fedora-review flag to ?

Re: Future changes in the new package and new branch processes

2014-09-09 Thread Tomas Tomecek
Quoting Christopher (2014-09-08 21:35:23) It'd be great if the fedpkg tool could do some of this. For example, fedpkg could create git repos locally, from a template and a few questions, for new packages, which could be pushed somewhere for review (usually GitHub, I'd imagine). It could even

Re: Future changes in the new package and new branch processes

2014-09-08 Thread Marcela Mašláňová
On 09/05/2014 05:08 PM, Pierre-Yves Chibon wrote: Dear all, In the last months, Till and I together with infrastructure and release-engineering have been thinking and working on how we could improve the current workflow for new package and new branch. To give you an idea, this is the current

Re: Future changes in the new package and new branch processes

2014-09-08 Thread David Howells
Pierre-Yves Chibon pin...@pingoured.fr wrote: * packager creates the scm-request and set fedora-cvs flag to ? I find this step counter intuitive. I accidentally set it to '+' rather than '?' and then was confused about why things weren't progressing. Can it be split into two flags? On

Re: Future changes in the new package and new branch processes

2014-09-08 Thread Tomas Tomecek
Quoting Pierre-Yves Chibon (2014-09-05 17:08:39) New procedure = * packager opens a review-request on bugzilla * reviewer sets the fedora-review flag to ? * reviewer does the review * reviewer sets the fedora-review flag to + * packager goes to pkgdb2 to request new package

Re: Future changes in the new package and new branch processes

2014-09-08 Thread Pierre-Yves Chibon
On Mon, Sep 08, 2014 at 10:15:33AM +0100, David Howells wrote: Pierre-Yves Chibon pin...@pingoured.fr wrote: * packager creates the scm-request and set fedora-cvs flag to ? I find this step counter intuitive. I accidentally set it to '+' rather than '?' and then was confused about why

Re: Future changes in the new package and new branch processes

2014-09-08 Thread Pierre-Yves Chibon
On Mon, Sep 08, 2014 at 11:31:58AM +0200, Tomas Tomecek wrote: Quoting Pierre-Yves Chibon (2014-09-05 17:08:39) New procedure = * packager opens a review-request on bugzilla * reviewer sets the fedora-review flag to ? * reviewer does the review * reviewer sets the

Re: Future changes in the new package and new branch processes

2014-09-08 Thread Till Maas
On Sun, Sep 07, 2014 at 05:23:15AM +0200, Ralf Corsepius wrote: I think the only safe way is to create an empty branch and not to populate it, because there are many constraints to be considered before a package can My proposal is to point new branches to the first commit in the master branch

Re: Future changes in the new package and new branch processes

2014-09-08 Thread Ralf Corsepius
On 09/08/2014 06:12 PM, Till Maas wrote: On Sun, Sep 07, 2014 at 05:23:15AM +0200, Ralf Corsepius wrote: I think the only safe way is to create an empty branch and not to populate it, because there are many constraints to be considered before a package can My proposal is to point new

Re: Future changes in the new package and new branch processes

2014-09-08 Thread Till Maas
On Mon, Sep 08, 2014 at 06:46:10PM +0200, Ralf Corsepius wrote: On 09/08/2014 06:12 PM, Till Maas wrote: This is also what is done initially, when a new repo is set up. Empty branches, i.e. only the ACL but no commit in the branch might lead to maintainers accidentally creating the wrong

Re: Future changes in the new package and new branch processes

2014-09-08 Thread Pierre-Yves Chibon
On Mon, Sep 08, 2014 at 06:46:10PM +0200, Ralf Corsepius wrote: On 09/08/2014 06:12 PM, Till Maas wrote: On Sun, Sep 07, 2014 at 05:23:15AM +0200, Ralf Corsepius wrote: I think the only safe way is to create an empty branch and not to populate it, because there are many constraints to be

Re: Future changes in the new package and new branch processes

2014-09-08 Thread Christopher
On Mon, Sep 8, 2014 at 5:31 AM, Tomas Tomecek ttome...@redhat.com wrote: Quoting Pierre-Yves Chibon (2014-09-05 17:08:39) New procedure = * packager opens a review-request on bugzilla * reviewer sets the fedora-review flag to ? * reviewer does the review * reviewer sets

Re: Future changes in the new package and new branch processes

2014-09-07 Thread Orion Poplawski
On 09/06/2014 09:23 PM, Ralf Corsepius wrote: On 09/06/2014 03:09 PM, Pierre-Yves Chibon wrote: On Sat, Sep 06, 2014 at 07:52:52AM +0200, Ralf Corsepius wrote: On 09/05/2014 05:08 PM, Pierre-Yves Chibon wrote: I wonder if we should rather create an empty branch and let the packager merge the

Re: Future changes in the new package and new branch processes

2014-09-06 Thread Pierre-Yves Chibon
On Fri, Sep 05, 2014 at 06:35:45PM +0200, Haïkel wrote: 2014-09-05 17:08 GMT+02:00 Pierre-Yves Chibon pin...@pingoured.fr: Dear all, In the last months, Till and I together with infrastructure and release-engineering have been thinking and working on how we could improve the current

Re: Future changes in the new package and new branch processes

2014-09-06 Thread Pierre-Yves Chibon
On Fri, Sep 05, 2014 at 08:12:49PM +0200, Zbigniew Jędrzejewski-Szmek wrote: On Fri, Sep 05, 2014 at 05:08:39PM +0200, Pierre-Yves Chibon wrote: Dear all, In the last months, Till and I together with infrastructure and release-engineering have been thinking and working on how we could

Re: Future changes in the new package and new branch processes

2014-09-06 Thread Pierre-Yves Chibon
On Sat, Sep 06, 2014 at 07:52:52AM +0200, Ralf Corsepius wrote: On 09/05/2014 05:08 PM, Pierre-Yves Chibon wrote: * packager requests new branch in pkgdb (2 clicks) = requests added to the scm admin queue * cvsadmin checks the request/package (check if package exists in the RHEL for

Re: Future changes in the new package and new branch processes

2014-09-06 Thread Ralf Corsepius
On 09/06/2014 03:09 PM, Pierre-Yves Chibon wrote: On Sat, Sep 06, 2014 at 07:52:52AM +0200, Ralf Corsepius wrote: On 09/05/2014 05:08 PM, Pierre-Yves Chibon wrote: * cvsadmin approves the creation of the new branch in pkgdb = branch creation broadcasted on fedmsg * git adjusted

Re: Future changes in the new package and new branch processes

2014-09-05 Thread Haïkel
2014-09-05 17:08 GMT+02:00 Pierre-Yves Chibon pin...@pingoured.fr: Dear all, In the last months, Till and I together with infrastructure and release-engineering have been thinking and working on how we could improve the current workflow for new package and new branch. To give you an idea,

Re: Future changes in the new package and new branch processes

2014-09-05 Thread Zbigniew Jędrzejewski-Szmek
On Fri, Sep 05, 2014 at 05:08:39PM +0200, Pierre-Yves Chibon wrote: Dear all, In the last months, Till and I together with infrastructure and release-engineering have been thinking and working on how we could improve the current workflow for new package and new branch. To give you an

Re: Future changes in the new package and new branch processes

2014-09-05 Thread Ralf Corsepius
On 09/05/2014 05:08 PM, Pierre-Yves Chibon wrote: * packager requests new branch in pkgdb (2 clicks) = requests added to the scm admin queue * cvsadmin checks the request/package (check if package exists in the RHEL for EPEL branch request - check if the user is a packager done in