public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Dan Douglas <ormaaj@gmail.com>
To: gentoo-dev@lists.gentoo.org
Cc: "Aaron W. Swenson" <titanofold@gentoo.org>
Subject: Re: [gentoo-dev] Portage Git migration - Handling Pull Requests (Was: clean cut or git-cvsserver)
Date: Thu, 24 May 2012 17:01:43 -0500	[thread overview]
Message-ID: <1520922.mmjlWrWcEL@smorgbox> (raw)
In-Reply-To: <4FBEA181.30204@gentoo.org>

[-- Attachment #1: Type: text/plain, Size: 2356 bytes --]

On Thursday, May 24, 2012 05:00:49 PM Aaron W. Swenson wrote:
> This gets us into another topic altogether.
> 
> I do believe git pull-requests should go through Bugzilla. A
> pull-request is the equivalent to bump requests, patch fixes, and all
> sorts of stuff that we already handle through Bugzilla. Bugzilla also
> contains our history.

This discussion was sort of in the context of proxy-maintainers. A pull 
request isn't equivalent to a bump request or entirely overlapping with most 
bugs that go through bugzilla. A pull request is  just "here take my code", or 
is useful for code revewing just because it integrates with the git workflow. I 
suppose you would  have to define the sorts of things that should go through 
Bugzilla. I can't imagine it being reasonable to use github for most types of 
bump requests.

> If they happen to be needing to be pulled from github, that's fine.
> Definitely convenient for the contributor.
> 
> We'll also need to clearly document how the pull-request is to be
> generated. (I vote for requiring signed pull-requests. [1])
> 
> github should not be our central point of contact. We have b.g.o for
> that. github should be on the fringes as a tool that we can use if we
> want to.

This reminds me of Linus' old Google talk on Git in which He said something 
along the lines of: "Many companies using Git internally don't know they're 
using git - they're using it whether they like it or not". There are ALREADY 
Gentoo projects using github and even pull requests. It doesn't really matter 
because everything more or less comes back to one point in the end. It's the 
repo that's the history of the project, not bugzilla. I've "filed" more bugs 
over IRC and had them fixed in the tree within 60 seconds than I have gone 
through bugzilla formalisms. FWIW, I think having the entire tree in one big 
repo is a bad idea to begin with.

But ok it's a good point. Github isn't a good central point of contact. People 
have to use their discression. It's just uncommon these days for a project as 
big as Gentoo to have ultra-centralized corporate-style procedures where 
everything happens exclusively though official channels. And anyway you couldn't 
"enforce" that sort of thing if you tried.

> [1]
> http://git-blame.blogspot.com.ar/2012/01/using-signed-tag-in-pull-requests.h
> tml

-- 
Dan Douglas

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

  parent reply	other threads:[~2012-05-24 22:02 UTC|newest]

Thread overview: 163+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-23 12:42 [gentoo-dev] Portage Git migration - clean cut or git-cvsserver Michael Weber
2012-05-23 12:54 ` Johannes Huber
2012-05-23 13:14   ` Ian Whyman
2012-05-23 13:31   ` Matthew Thode
2012-05-23 14:39   ` Alexey Shvetsov
2012-05-23 14:43     ` Anthony G. Basile
2012-05-23 16:33       ` Rich Freeman
2012-05-23 13:25 ` Andreas K. Huettel
2012-05-23 13:28   ` Aaron W. Swenson
2012-05-23 13:34     ` Fabio Erculiani
2012-05-23 14:17 ` [gentoo-dev] " Michael Palimaka
2012-05-23 14:54 ` [gentoo-dev] " justin
2012-05-23 16:32 ` Sergei Trofimovich
2012-05-23 16:33 ` Michał Górny
2012-05-23 16:42   ` Andreas K. Huettel
2012-05-23 16:45   ` Alexey Shvetsov
2012-05-24 19:51   ` Marc Schiffbauer
2012-05-23 16:47 ` Robin H. Johnson
2012-05-23 16:58   ` Alexey Shvetsov
2012-05-23 17:19     ` Robin H. Johnson
2012-05-23 17:22       ` Alexey Shvetsov
2012-05-23 17:32         ` Rich Freeman
2012-05-23 17:35           ` Alexey Shvetsov
2012-05-24 10:02             ` Kent Fredric
2012-05-24 10:16               ` Alexey Shvetsov
2012-05-23 17:38           ` Robin H. Johnson
2012-05-23 17:38         ` Chí-Thanh Christopher Nguyễn
2012-05-23 18:37           ` Rafael Goncalves Martins
2012-05-23 19:37           ` Arun Raghavan
2012-05-23 19:40             ` Nirbheek Chauhan
2012-05-23 19:47             ` Fabio Erculiani
2012-05-23 19:51             ` hasufell
2012-05-23 20:00             ` Ezequiel Garcia
2012-05-23 20:32               ` Rich Freeman
2012-05-24 10:08                 ` Kent Fredric
2012-05-24  6:34               ` Pacho Ramos
2012-05-23 20:05             ` Alexey Shvetsov
2012-05-23 20:25             ` William Hubbs
2012-05-23 20:37               ` Michał Górny
2012-05-23 20:55                 ` William Hubbs
2012-05-24  4:49             ` Matt Turner
2012-05-23 16:58   ` Justin
2012-05-23 20:34     ` Michael Weber
2012-05-23 16:59   ` Matt Turner
2012-05-23 17:06     ` Alexey Shvetsov
2012-05-23 20:39       ` Michael Weber
2012-05-23 17:09   ` Alexey Shvetsov
2012-05-23 21:14   ` Dan Douglas
2012-05-23 21:48     ` Michael Weber
2012-05-24 10:17       ` Kent Fredric
2012-05-24 14:40         ` Michał Górny
2012-05-24 15:02           ` Ralph Sennhauser
2012-05-24 15:32             ` Rich Freeman
2012-05-25  1:21               ` Alec Warner
2012-05-25  1:32                 ` Kent Fredric
2012-05-25  6:12                   ` Ulrich Mueller
2012-05-25  6:47                     ` Kent Fredric
2012-05-25 11:07                       ` Alec Warner
2012-05-24 15:40             ` Michał Górny
2012-05-24 17:13             ` Kent Fredric
2012-05-24 17:52               ` Ian Stakenvicius
2012-05-24 18:08                 ` Ciaran McCreesh
2012-05-24 18:17                 ` Kent Fredric
2012-05-24 18:33                 ` Markos Chandras
2012-05-24 18:37                 ` Dan Douglas
     [not found]                   ` <4FBE8223.2010406@gentoo.org>
2012-05-24 19:57                     ` Dan Douglas
2012-05-24 21:00                       ` [gentoo-dev] Portage Git migration - Handling Pull Requests (Was: clean cut or git-cvsserver) Aaron W. Swenson
2012-05-24 21:14                         ` Alexey Shvetsov
2012-05-24 21:20                           ` Kent Fredric
2012-05-24 21:27                             ` Alexey Shvetsov
2012-05-24 21:16                         ` Kent Fredric
2012-05-24 22:01                         ` Dan Douglas [this message]
2012-05-25  0:58                           ` Rich Freeman
2012-05-25  1:40                           ` Maxim Kammerer
2012-05-24  5:56     ` [gentoo-dev] Portage Git migration - clean cut or git-cvsserver Michał Górny
2012-05-24  6:04       ` Dan Douglas
2012-05-24  6:33         ` [gentoo-dev] " Duncan
2012-05-24  6:51           ` Dan Douglas
2012-05-24  9:16             ` Vítor Brandão
2012-05-24 11:43             ` Duncan
2012-05-24 12:05               ` Dirkjan Ochtman
2012-05-24 13:03                 ` Ciaran McCreesh
2012-05-24 13:37                 ` Kent Fredric
2012-05-24 13:51                   ` Michael Weber
2012-05-24  1:19 ` [gentoo-dev] " Mark Wright
2012-05-24 16:17   ` Ultrabug
2012-05-26 16:18 ` Alexey Shvetsov
2012-05-30  1:55   ` Rich Freeman
2012-05-30  9:38     ` [gentoo-dev] " Duncan
2012-05-30 10:16       ` Dirkjan Ochtman
2012-05-30 11:32         ` Rich Freeman
2012-05-30 13:06           ` Duncan
2012-05-30 18:33             ` Robin H. Johnson
2012-05-30 20:31               ` Dirkjan Ochtman
2012-05-31 12:04                 ` Aaron W. Swenson
2012-05-31 12:16                   ` Peter Stuge
2012-05-31 12:21                   ` Dirkjan Ochtman
2012-05-31 16:50                   ` Robin H. Johnson
2012-06-01 20:49                   ` Michael Weber
2012-06-02  0:52                     ` Rich Freeman
2012-06-02  4:04                       ` Robin H. Johnson
2012-06-02 10:59                         ` Rich Freeman
2012-06-02 12:03                           ` Dirkjan Ochtman
2012-06-02 14:19                             ` Rich Freeman
2012-05-31 16:49                 ` Robin H. Johnson
2012-05-31 17:48                   ` Rich Freeman
2012-05-31 19:13                     ` Robin H. Johnson
2012-05-31 19:29                       ` Rich Freeman
2012-05-31 19:54                       ` William Hubbs
2012-05-31 20:26                         ` Duncan
2012-05-31 20:57                           ` William Hubbs
2012-06-01 12:57                             ` Duncan
2012-06-01 13:25                               ` Nicolas Sebrecht
2012-06-01 15:08                                 ` Andreas K. Huettel
2012-06-01 15:39                                 ` Rich Freeman
2012-06-01 15:56                                   ` Kent Fredric
2012-06-01 16:18                                     ` W. Trevor King
2012-06-01 15:45                               ` William Hubbs
2012-06-01 16:33                                 ` Robin H. Johnson
2012-06-01 16:55                                   ` Kent Fredric
2012-06-01 18:28                                   ` Rich Freeman
2012-06-01 19:22                                   ` William Hubbs
2012-05-31 22:04                           ` Kent Fredric
2012-05-31 19:18                     ` William Hubbs
2012-05-31 19:23                       ` Ciaran McCreesh
2012-05-31 22:04                         ` William Hubbs
2012-05-31 22:28                           ` Peter Stuge
2012-06-01  3:59                           ` Michał Górny
2012-06-02  0:14                           ` James Cloos
2012-05-31 19:33                       ` Michał Górny
2012-05-31 19:52                         ` Alexey Shvetsov
2012-05-31 21:52                           ` Kent Fredric
2012-06-01  2:49                             ` Rich Freeman
2012-06-01  4:55                               ` Kent Fredric
2012-06-01 10:54                                 ` Rich Freeman
2012-06-01 11:23                                   ` Kent Fredric
2012-06-01 11:44                                     ` Michał Górny
2012-06-01 12:42                                     ` Rich Freeman
2012-06-01 12:48                                       ` Alexey Shvetsov
2012-06-01 14:37                                       ` Kent Fredric
2012-06-01 15:12                                         ` Andreas K. Huettel
2012-06-01 15:25                                           ` Kent Fredric
2012-06-01 15:36                                             ` Andreas K. Huettel
2012-06-01 15:53                                               ` Rich Freeman
2012-06-01 16:12                                                 ` Dirkjan Ochtman
2012-06-01 16:14                                                 ` Kent Fredric
2012-06-01 16:34                                                 ` Robin H. Johnson
2012-06-01 16:56                                                   ` Kent Fredric
2012-06-01 17:36                                                     ` Robin H. Johnson
2012-06-01 15:39                                             ` Michał Górny
2012-06-01 15:28                                           ` Rich Freeman
2012-05-31 19:58                         ` Rich Freeman
2012-05-31 20:09                           ` Michał Górny
2012-05-31 20:27                             ` Michael Orlitzky
2012-06-01  4:05                               ` Michał Górny
2012-06-01 10:49                                 ` Rich Freeman
2012-05-31 21:06                           ` William Hubbs
2012-05-31 22:00                           ` Kent Fredric
2012-06-01  8:16                   ` Dirkjan Ochtman
2012-06-01  8:40                     ` Kent Fredric
2012-05-30 16:53           ` Tobias Klausmann
2012-05-30 16:58             ` Aaron W. Swenson
2012-05-30 18:12               ` Justin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1520922.mmjlWrWcEL@smorgbox \
    --to=ormaaj@gmail.com \
    --cc=gentoo-dev@lists.gentoo.org \
    --cc=titanofold@gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox