public inbox for overseers@sourceware.org
 help / color / mirror / Atom feed
From: "mark at klomp dot org" <sourceware-bugzilla@sourceware.org>
To: overseers@sourceware.org
Subject: [Bug Infrastructure/30042] patchworks should automatically notice a pushed patch
Date: Tue, 24 Jan 2023 22:07:27 +0000	[thread overview]
Message-ID: <bug-30042-14326-3hXXZm0U8O@http.sourceware.org/bugzilla/> (raw)
In-Reply-To: <bug-30042-14326@http.sourceware.org/bugzilla/>

https://sourceware.org/bugzilla/show_bug.cgi?id=30042

Mark Wielaard <mark at klomp dot org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mark at klomp dot org

--- Comment #4 from Mark Wielaard <mark at klomp dot org> ---
(In reply to Tom Tromey from comment #3)
> (In reply to Carlos O'Donell from comment #1)
> 
> > The first problem we saw here was that it requires discipline from the
> > community to commit *exactly* what was posted and agreed upon in order to
> > make the checking less error prone.
> 
> This would work fine for [pushed] patches, which seems good enough for
> this ticket.

yeah, one step at a time. Siddhesh script could be run under gdbadmin or we
could do it from the buildbot since we build each gdb commit on push.

> For other patches, this seems like more work because it means there
> will have to be one final series sent, but patchworks is also bad about
> removing old series.

We could have a separate script or mail filter that checks for patches that
supersede a previous patch/series (check subject is the same modulo vX+1).

> > In either (1) or (2) I think this is up to the projects to run the scripts
> > that they need.
> 
> Are you saying this shouldn't be a sourceware PR?
> I can close it if so.

We could move it to the gdb product, but lets keep it open here to document the
steps we did so other projects can adopt them if they like.

-- 
You are receiving this mail because:
You are the assignee for the bug.

  parent reply	other threads:[~2023-01-24 22:07 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-24 14:29 [Bug Infrastructure/30042] New: " tromey at sourceware dot org
2023-01-24 15:02 ` [Bug Infrastructure/30042] " carlos at redhat dot com
2023-01-24 15:32 ` siddhesh at sourceware dot org
2023-01-24 21:55 ` tromey at sourceware dot org
2023-01-24 22:07 ` mark at klomp dot org [this message]
2023-01-25 17:22 ` siddhesh at sourceware dot org
2023-01-25 20:23 ` tromey at sourceware dot org
2023-01-30 14:19 ` pedro at palves dot net
2024-03-24 22:02 ` mark at klomp dot org

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=bug-30042-14326-3hXXZm0U8O@http.sourceware.org/bugzilla/ \
    --to=sourceware-bugzilla@sourceware.org \
    --cc=overseers@sourceware.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).