From: "Marco Costalba" <mcostalba@gmail.com>
To: "Jan Hudec" <bulb@ucw.cz>
Cc: "Karl Hasselström" <kha@treskal.com>, git@vger.kernel.org
Subject: Re: [QGIT RFC] Unit tests for QGit
Date: Fri, 29 Aug 2008 09:01:48 +0200 [thread overview]
Message-ID: <e5bfff550808290001r168c7edfre926f1bc735f3ad0@mail.gmail.com> (raw)
In-Reply-To: <20080828220124.GF15520@efreet.light.src>
On Fri, Aug 29, 2008 at 12:01 AM, Jan Hudec <bulb@ucw.cz> wrote:
>
> So, now there is a test infrastructure plus test case to reproduce this
> switching between stgit and non-stgit branch in
> git://repo.or.cz/qgit4/bulb.git (http://repo.or.cz/r/qgit4/bulb.git) with
> whoping 9 commits. Should I send out a patch series, or do you prefer just
> pulling?
>
I prefer to pull.
> Now in my opinion the code could use some refactoring rather than just fixing
> the bugs (my long term intent is to add features like topgit support,
> push/pull/merge and other things git-gui can do and such). I'd start with
> the Git initialization sequence. I'll write tests for the new code, but as
> I expect it to have significantly different interface from the old one, I'll
> not try to write tests for the current one.
>
We have following possibilities:
- Pull the code directly in qgit master as soon as there are some new
commits in your branch
- Pull the code in public qgit repo but under a different branch,
let's call' it "next" ;-)
- Waiting for your code has stabilized a bit (testing infrastructure
is very young and for what I have seen from revision history code is
still very 'fluid'), then pull the branch in qgit master directly.
These are my two cents:
Option one could be a little bit misleading for people pulling from
qgit repo to get current qgit sources + just fixes.
Option two is doable but is an additional step with an additional
maintainer burden, probably the current number of contributors to qgit
is not enough to justify such a complex development model.
Perhaps option three it seems the more balanced, also looking at
projects git related and with similar size of qgit, as example StGit
itself. When let's say Karl has ready a block of patches he sends them
all as a series and are applied to StGit master branch.
The only modification I would suggest is that I can pull from you repo
directly instead of asking you to send patches to the git list.
I leave up to you when to ask for a pull request, I only ask you to
consider that qgit public repo is pulled also by people not interested
in the latest development, and they only want a stable qgit, so please
ask for a pull when you think stuff is stable enough.
Comments?
Marco
next prev parent reply other threads:[~2008-08-29 7:02 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-08 21:13 [QGIT RFC] Unit tests for QGit Jan Hudec
2008-08-08 23:00 ` Benjamin Sergeant
2008-08-10 7:55 ` Jan Hudec
2008-08-17 8:57 ` Marco Costalba
2008-08-17 14:15 ` Jan Hudec
2008-08-17 15:46 ` Marco Costalba
2008-08-17 19:58 ` Jan Hudec
2008-08-17 20:30 ` Marco Costalba
2008-08-18 18:00 ` Jan Hudec
2008-08-19 14:53 ` Marco Costalba
2008-08-27 20:18 ` Jan Hudec
2008-08-28 11:29 ` Marco Costalba
2008-08-28 15:31 ` Karl Hasselström
2008-08-28 18:54 ` Marco Costalba
2008-08-28 22:01 ` Jan Hudec
2008-08-29 7:01 ` Marco Costalba [this message]
2008-08-28 22:18 ` Karl Hasselström
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=e5bfff550808290001r168c7edfre926f1bc735f3ad0@mail.gmail.com \
--to=mcostalba@gmail.com \
--cc=bulb@ucw.cz \
--cc=git@vger.kernel.org \
--cc=kha@treskal.com \
/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;
as well as URLs for NNTP newsgroup(s).