Bug 11156 - Patches containing long lines are split by patchwork
Summary: Patches containing long lines are split by patchwork
Status: RESOLVED WONTFIX
Alias: None
Product: Patchwork/Patchtest
Classification: Yocto Project Subprojects
Component: Patchwork (show other bugs)
Version: 2.3
Hardware: x86 Multiple
: Medium+ normal
Target Milestone: 2.3 M4
Assignee: Jose Lamego
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2017-03-14 18:17 UTC by Leonardo Sandoval Gonzalez
Modified: 2017-12-06 17:26 UTC (History)
0 users

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments
Email received by pw from the mailing list (11.75 KB, message/rfc822)
2017-03-17 18:06 UTC, Jose Lamego
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
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.