From: "Gábor Boskovits" <boskovits@gmail.com>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: Guix-devel <guix-devel@gnu.org>
Subject: Re: Patchwork + automated checking and testing of patches
Date: Tue, 6 Nov 2018 16:13:36 +0100 [thread overview]
Message-ID: <CAE4v=pgx4bgxMG0+eaJ8AXYmBSSCyB1K9dH3g2o+ZZTjtGxUaw@mail.gmail.com> (raw)
In-Reply-To: <87sh0e2v7v.fsf@gnu.org>
Hello,
Ludovic Courtès <ludo@gnu.org> ezt írta (időpont: 2018. nov. 6., K, 14:33):
>
> Hello!
>
> Christopher Baines <mail@cbaines.net> skribis:
>
> > Ludovic Courtès <ludo@gnu.org> writes:
>
> [...]
>
> >> Back when we tried, it had a couple of shortcomings:
> >>
> >> 1. It would not automatically detect which patches have been merged;
> >>
> >> 2. It would not present patch series correctly.
> >>
> >> From what you write it looks like #1 has been fixed, but the web
> >> interface suggests that #2 isn’t quite fixed yet, is that correct?
> >
> > On the detecting merged patches, that's definately working for some
> > patches though. I don't fully understand what criteria it's using
> > though, as it's comparing the commits that come through to the master
> > branch, and I bet it's possible to confuse it a bit by tweaking patches
> > before pushing them.
> >
> > Regarding patch series, I don't know much about the specifics of this,
> > and I don't know much about Patchwork, but just comparing a few patches
> > on the older version [1], and the newer version [2], it looks like it's
> > better. Take this patch [3], it's part of a series, but you can't
> > tell. However, with this patch [4], you can see the series and related
> > patches towards the top of the page, and also a link to download the
> > whole series as an mbox. How does this look to you?
>
> It looks better than the old Patchwork version, but I think
> <https://patchwork.cbaines.net/project/guix-patches/list/> still leaves
> a bit to be desired. For instance, it shows every message, even for
> series.
>
> At this point I find <https://issues.guix.info/search?query=is%3Aopen>
> clearer and less cluttered, though it needs some improvements (e.g.,
> most-recent-first sorting, listing all the open issues, etc.), which I
> think should be relatively easy to do. Also, it is directly linked to
> Debbugs and its notion of issues and associated properties (bug number,
> owner, status, tags, etc.), which is good IMO.
I have noticed that issues.guix.info goes to 504 quite often, Ricardo said it
was because we only have one backend. So it might be a good idea to get
some infrastructure up there. WDYT?
>
> WDYT?
>
> Thanks,
> Ludo’.
>
g_bor
next prev parent reply other threads:[~2018-11-06 15:13 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-10-31 10:43 Patchwork + automated checking and testing of patches Christopher Baines
2018-10-31 13:34 ` Tobias Geerinckx-Rice
2018-10-31 13:53 ` Christopher Baines
2018-11-01 15:22 ` Ludovic Courtès
2018-11-01 18:55 ` Christopher Baines
2018-11-06 13:26 ` Ludovic Courtès
2018-11-06 15:13 ` Gábor Boskovits [this message]
2018-11-06 18:52 ` Ricardo Wurmus
2018-11-07 18:40 ` Christopher Baines
2018-11-07 22:00 ` Ludovic Courtès
2018-11-19 19:32 ` Christopher Baines
2018-11-22 9:07 ` Ludovic Courtès
2018-12-02 22:45 ` Chris Marusich
2018-12-03 0:51 ` Christopher Baines
2018-12-08 21:27 ` Chris Marusich
2019-02-01 12:53 ` Christopher Baines
2019-02-04 21:20 ` Ludovic Courtès
2019-02-08 12:04 ` Christopher Baines
2019-02-08 18:54 ` Björn Höfling
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='CAE4v=pgx4bgxMG0+eaJ8AXYmBSSCyB1K9dH3g2o+ZZTjtGxUaw@mail.gmail.com' \
--to=boskovits@gmail.com \
--cc=guix-devel@gnu.org \
--cc=ludo@gnu.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.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.