Ignore whitespace before and after git tags

Offen

#645 erstellt am 08.1.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 45/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: git, python
Bereich: cli, tooling

Rechercherichtung

Beginne mit dem git-am-Einstiegspunkt von pwclient und reproduziere den gemeldeten Patch unter Verwendung des im Issue gezeigten Reviewed-by-Tags mit vorangestelltem und nachgestelltem Whitespace. Ermittle, ob die Tag-Analyse zu patchwork oder pwclient gehört, und überprüfe anschließend, dass git-am den Tag beibehält und dass Tags ohne umgebenden Whitespace weiterhin funktionieren.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

NOTE: Experienced on old 2.2.0, maybe it has been fixed. Also I'm not sure if it is patchwork or pwclient issue.

Space before git tags (e.g. Reviewed-by: Li Wang <liwang@redhat.com>) causes pwclient git-am to ignore them.

See:
https://lore.kernel.org/ltp/CAEemH2dU6XP+Er75eERgo+p7TyMgEjMJY7tjAGA15NOKFrZM+A@mail.gmail.com/
https://patchwork.ozlabs.org/project/ltp/patch/20251010113014.309433-1-pvorel@suse.cz/#3595268

$ cat ~/.pwclientrc
[ozlabs]
url = https://patchwork.ozlabs.org/xmlrpc/
username: ...
password: ...

$ pwclient git-am 2148087 -p ozlabs
Applying patch #2148087 using "git am"
Description: tst_cmd.h: Convert the rest of comments to kerneldoc
Applying: tst_cmd.h: Convert the rest of comments to kerneldoc

$ git log -1
commit 980c30ad8042de6694be88866a0d668aa8748bc9 (HEAD -> xxx)
Author: Petr Vorel <pvorel@suse.cz>
Date:   Fri Oct 10 13:30:14 2025 +0200

    tst_cmd.h: Convert the rest of comments to kerneldoc
    
    Move docs from the private function implementations (functions with
    trailing underscore) to functions used into functions used in the new
    API.
    
    This avoids to document private functions.
    
    Signed-off-by: Petr Vorel <pvorel@suse.cz>
    Reviewed-by: Andrea Cervesato <andrea.cervesato@suse.com>

=> There is no Reviewed-by: Li Wang <liwang@redhat.com>.

Vorherrschende Sprache: Python
Sterne: 317
Forks: 91
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus getpatchwork/patchwork

Broken "documentation for this release" links in the release notes Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

getpatchwork/patchwork#668 · Erstellt am 03.8.2026
[v3.1.2] Submitter search doesn't autocomplete Offen

bug web-ui

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100

getpatchwork/patchwork#546 · 2 Kommentare · Erstellt am 30.7.2023
Patchwork mangling binary file deletion patch Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 52/100

getpatchwork/patchwork#664 · Erstellt am 28.7.2026
slight privacy inconsistency: /api/patches (HTML) lists people to anonymous users but /api/people doesn't Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

getpatchwork/patchwork#663 · Erstellt am 23.7.2026
Showing a diff between two versions of the same series Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100

getpatchwork/patchwork#636 · 2 Kommentare · Erstellt am 06.8.2025

Alle Issues in getpatchwork/patchwork