public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Michael Mol <mikemol@gmail.com>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] Re: [OT/NIT] Re: Re: [gentoo-commits] gentoo-x86 commit in profiles: ChangeLog package.mask
Date: Wed, 24 Apr 2013 07:47:42 -0400	[thread overview]
Message-ID: <5177C65E.6070404@gmail.com> (raw)
In-Reply-To: <20130424112129.25885.qmail@stuge.se>

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

On 04/24/2013 07:21 AM, Peter Stuge wrote:
> Jeroen Roovers wrote:
>> Er, you can't be seriously suggesting we will drop repoman checks
>> with the migration to git? I don't see how that would benefit anyone.
> 
> I would argue that repoman and/or corresponding checks should be run
> by a CI system hooked up to the Gerrit instance that developers push to.
> 
> Anything else is IMO waste of developers' time and minds.

I was thinking something similar, actually, except you'd need something
like this:

1. dev pushes to Gerrit
2. Gerrit does processing
3. Gerrit pushes to tree, if tests pass.

It would necessitate needing a separate mechanism to be able to undo
changes to tree that broke things Gerrit missed, or it would necessitate
"undo" steps being pushed through Gerrit. Both have their disadvantages.

It might also result in only being able to push one changeset at a time,
due to this scenario:

1. Dev A pulls from tree
2. Dev A merges those changes with his local copy of tree
3. Dev A pushes to Gerrit
4. Gerrit begins tests on changeset A
5. Dev B pulls from tree (or perhaps he pulled earlier)
6. Gerrit is still testing changeset A
7. Dev B merges those changes with his local copy of tree
8. Gerrit finishes testing, pushes to tree
9. Dev B pushes to Gerrit
10. Gerrit runs tests on changeset B
11. Gerrit finishes tests, pushes to tree
13. Gerrit's push to tree fails, since tree with changeset A isn't in
changeset B's ancestry.

Though you might be able to get around that by using git's email
features to email diffs, allowing Gerrit to pipeline them (unless one
diff set fails to apply after another has been applied, of course).




[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 555 bytes --]

  parent reply	other threads:[~2013-04-24 11:47 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20130419091632.D01152171D@flycatcher.gentoo.org>
2013-04-19 13:30 ` [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in profiles: ChangeLog package.mask Alexis Ballier
2013-04-20 17:28   ` Jeroen Roovers
2013-04-21 12:53   ` Ben de Groot
2013-04-21 14:11     ` Denis Dupeyron
2013-04-21 14:23       ` Rich Freeman
2013-04-21 14:39         ` Denis Dupeyron
2013-04-21 14:38       ` Chí-Thanh Christopher Nguyễn
2013-04-21 14:50         ` Denis Dupeyron
2013-04-21 15:07           ` Chí-Thanh Christopher Nguyễn
2013-04-21 15:17             ` Alexis Ballier
2013-04-22 11:43         ` Ben de Groot
2013-04-22 17:13           ` Michał Górny
2013-04-23 16:16             ` Ben de Groot
2013-04-22 12:07       ` Ben de Groot
2013-04-21 14:59     ` Alexis Ballier
2013-04-22 11:56       ` Ben de Groot
2013-04-22 14:00         ` Alexis Ballier
2013-04-23  3:58         ` Ryan Hill
2013-04-23 16:24           ` Ben de Groot
2013-04-24 10:59             ` Duncan
2013-04-30  2:06             ` Ryan Hill
2013-04-21 15:05     ` [OT/NIT] " Alexis Ballier
2013-04-21 15:32       ` Tom Wijsman
2013-04-21 15:43         ` Alexis Ballier
2013-04-21 16:00         ` Peter Stuge
2013-04-21 17:07           ` Tom Wijsman
2013-04-21 18:47             ` Peter Stuge
2013-04-21 18:57       ` Michał Górny
2013-04-22 12:00       ` Ben de Groot
2013-04-22 13:40         ` Alexis Ballier
2013-04-22 22:46           ` [gentoo-dev] Re: [OT/NIT] " Duncan
2013-04-23 18:00             ` Jeroen Roovers
2013-04-23 18:20               ` Rich Freeman
2013-04-23 18:37                 ` Matt Turner
2013-04-23 19:11                   ` Rich Freeman
2013-04-23 19:21                     ` Matt Turner
2013-04-23 19:25                     ` Ian Stakenvicius
2013-04-23 19:43                       ` Rich Freeman
2013-04-23 18:38               ` Matt Turner
2013-04-24 11:18               ` Duncan
2013-04-24 11:21               ` Peter Stuge
2013-04-24 11:25                 ` Peter Stuge
2013-04-24 11:47                 ` Michael Mol [this message]
2013-04-24 13:25                   ` Rich Freeman
2013-04-24 20:04                     ` Alex Xu
2013-04-24 22:26                       ` Rich Freeman
2013-04-24 23:23                 ` Patrick Lauer
2013-04-24 23:50                   ` Peter Stuge
2013-04-23 19:31             ` Michał Górny
2013-04-23 19:50               ` Jeroen Roovers
2013-04-23 20:27             ` Tom Wijsman
2013-04-23 21:12               ` Zac Medico

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=5177C65E.6070404@gmail.com \
    --to=mikemol@gmail.com \
    --cc=gentoo-dev@lists.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