Bug 11156

Summary:

Patches containing long lines are split by patchwork

Product:

[Yocto Project Subprojects] Patchwork/Patchtest

Reporter:

Leonardo Sandoval Gonzalez <leonardo.sandoval.gonzalez>

Component:

Patchwork

Assignee:

Jose Lamego <jose.a.lamego>

Status:

RESOLVED WONTFIX

QA Contact:

Severity:

normal

Priority:

Medium+

Version:

2.3

Target Milestone:

2.3 M4

Hardware:

x86

OS:

Multiple

Whiteboard:

OS type for building Yocto:

---

Type of Regression:

---

Verified:

Documentation change:

No (bug/feature does not impact docs)

Attachments:

Description	Flags
Email received by pw from the mailing list	none

Description Leonardo Sandoval Gonzalez 2017-03-14 18:17:07 UTC

I sent a series [1] which contains a patch [2] that removes a long long line. When patchtest tries to test it, it fails on the merge because the original patch's long line is truncated by patchwork while creating the series internally.

[1] https://patchwork.openembedded.org/series/5803/#
[2] https://patchwork.openembedded.org/patch/138105/

Comment 1 Jose Lamego 2017-03-17 18:06:13 UTC

Created attachment 3654 [details]
Email received by pw from the mailing list

Comment 2 Jose Lamego 2017-03-17 18:12:04 UTC

I've attached the patch message as it is sent by the OE-core mailing list and then received by patchwork, were it shows that the line splitting is already present.

I checked the mailing list and the line in original patch was correct, so the splitting happens somewhere at the ML, not patchwork.

I'm closing this bug now, but feel free to reopen and re-phrase if needed.

Comment 3 Leonardo Sandoval Gonzalez 2017-03-17 18:15:22 UTC

(In reply to comment #2)
> I've attached the patch message as it is sent by the OE-core mailing list
> and then received by patchwork, were it shows that the line splitting is
> already present.
> 
> I checked the mailing list and the line in original patch was correct, so
> the splitting happens somewhere at the ML, not patchwork.
> 
> I'm closing this bug now, but feel free to reopen and re-phrase if needed.

Jose, try this

git pw mbox <id> | git am

where id is the pw series id.

Comment 4 Jose Lamego 2017-03-17 18:31:34 UTC

(In reply to comment #3)

> 
> Jose, try this
> 
> git pw mbox <id> | git am
> 
> where id is the pw series id.

Yes, the mbox includes the (wrongly) split line. The issue here is that patchwork received it that way from the ML. To verify this, look at your inbox for the email sent by the OE-core mailing list.