public inbox for gentoo-scm@lists.gentoo.org
 help / color / mirror / Atom feed
From: Donnie Berkholz <dberkholz@gentoo.org>
To: Alec Warner <antarus@gentoo.org>
Cc: Mike Auty <ikelos@gentoo.org>, gentoo-scm@lists.gentoo.org
Subject: Re: [gentoo-scm] repoman patch for git support (from drobbins)
Date: Sun, 16 Nov 2008 09:20:38 -0800	[thread overview]
Message-ID: <20081116172037.GF23210@comet> (raw)
In-Reply-To: <b41005390811160134y6bffbc1u639181f9379f20b2@mail.gmail.com>

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

On 01:34 Sun 16 Nov     , Alec Warner wrote:
> Change the workflow[1].
> 
> Right now you pay the cost of running tree-wide tests every time you
> commit.  This incentivizes developers to commit less often (to avoid
> paying the tax of tree-wide tests).
> 
> In CVS commiting less is a problem:
> 
> 1. Make Changes to a number of files.
> 2. Script your commits.
> 3. Run Script
> 4....N: Script commits one file at a time.
> 4.5: A race condition between changes you have commited to CVS versus
> uncommited changes occurs when CVS is synced to Osprey.  This race
> condition can often cause tree oddness.
> 5.  All Changes commited.
> 5.5: All changes synced to Osprey.
> 
> I am unsure if repoman category and repoman tree-wide commits avoid
> this race condition.
> 
> A new scheme would be:
> 
> 1. Make Changes to a number of files.
> 2. Category or Treewide Repoman commit.
> 3. Run taxing tree-wide tests.
> 4. git commit -a (?)
> 5. Done!
> 
> Tell me if I'm horribly wrong or missing something, it is late here.

Are you saying we could use the index here? Another option (that you may 
or may not be suggesting) could be to make a new 'repoman push' 
operation that actually does the hard checks, and make 'repoman commit' 
just do a local commit and only very quick checks.

-- 
Thanks,
Donnie

Donnie Berkholz
Developer, Gentoo Linux
Blog: http://dberkholz.wordpress.com

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

  reply	other threads:[~2008-11-16 17:20 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-07  1:12 [gentoo-scm] repoman patch for git support (from drobbins) Jeremy Olexa
2008-11-07  0:55 ` Robin H. Johnson
2008-11-11 10:22   ` Fabian Groffen
2008-11-11  9:36     ` Robin H. Johnson
2008-11-12  1:55     ` Zac Medico
2008-11-15 16:52       ` Mike Auty
2008-11-15 22:32         ` Zac Medico
2008-11-16  5:04           ` Donnie Berkholz
2008-11-15 22:44         ` Alec Warner
2008-11-15 23:07           ` Mike Auty
2008-11-16  4:53             ` Donnie Berkholz
2008-11-16  6:41               ` Zac Medico
2008-11-16  6:47                 ` Donnie Berkholz
2008-11-16  7:10                   ` Zac Medico
2008-11-16  9:34               ` Alec Warner
2008-11-16 17:20                 ` Donnie Berkholz [this message]
2008-11-16 21:49                   ` Alec Warner
2008-11-16 18:05                 ` Zac Medico
2008-11-16 21:55                   ` Alec Warner
2008-11-16 11:29               ` Mike Auty

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=20081116172037.GF23210@comet \
    --to=dberkholz@gentoo.org \
    --cc=antarus@gentoo.org \
    --cc=gentoo-scm@lists.gentoo.org \
    --cc=ikelos@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