Re: ✗ patchtest: failure for 11 packages upgrade

[Hongxu Jia <hongxu.jia@windriver.com>]

[-- Attachment #1: Type: text/plain, Size: 3175 bytes --]

On 2017年02月08日 11:30, Patchwork wrote:
> == Series Details ==
>
> Series: 11 packages upgrade
> Revision: 1
> URL   : https://patchwork.openembedded.org/series/5200/
> State : failure
>
> == Summary ==
>
>
> Thank you for submitting this patch series to OpenEmbedded Core. This is
> an automated response. Several tests have been executed on the proposed
> series by patchtest resulting in the following failures:
>
>
>
> * Issue             Added patch file is missing Upstream-Status in the header [test_upstream_status_presence]
>    Suggested fix    Add Upstream-Status: <status> to the header of meta/recipes-devtools/elfutils/elfutils-0.168/debian/hurd_path.patch (possible values: Pending, Submitted, Accepted, Backport, Denied, Inappropriate)

The hurd_path.patch is cherry picked from debian which does
not has "Upstream-Status: <status>" and "Signed-off-by", so I
think we could ignore this warning.

Here is the upstream source of debian patch:
http://ftp.de.debian.org/debian/pool/main/e/elfutils/elfutils_0.168-0.2.debian.tar.xz

You may suggest to add it to SRC_URI rather than picking up manually,
but debian frequently upgrades miner version "-0.2", and which does not
sync up with upstream elfutils, that's why I manually cherry pick.

> * Issue             Series does not apply on top of target branch [test_series_merge_on_head]
>    Suggested fix    Rebase your series on top of targeted branch
>    Targeted branch  master (currently at 68da6c9884)

Previously, Zhengtried to upgrade bash failed (git am "bash: 4.3.30 -> 
4.4.eml" failed)
the reason is encoding issue, if invoking "git send-email", it set UTF-8 
by default
---
            Content-Type: text/plain; charset=UTF-8
            Content-Transfer-Encoding: 8bit
---
And the utf-8 encoding changed the content of 
meta/recipes-extended/bash/bash/fix-run-intl.patch,
which caused "git am" failed.

Manually send the patch as attachment could workaround the issue.
But the patchwork still report apply failure, I guess it applied [PATCH 
12/13]
before [PATCH 11/13], or could not detect attachment of [PATCH 11/13].

Hi Ross or any one concerns,

Could you apply the patch manually, to see if it is OK?
(I tested to save as the eml file and invoke git am to apply, everything 
is ok)

//Hongxu

> * Issue             A patch file has been added, but does not have a Signed-off-by tag [test_signed_off_by_presence]
>    Suggested fix    Sign off the added patch file (meta/recipes-devtools/elfutils/elfutils-0.168/debian/hurd_path.patch)
>
>
>
> If you believe any of these test results are incorrect, please reply to the
> mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
> Otherwise we would appreciate you correcting the issues and submitting a new
> version of the patchset if applicable. Please ensure you add/increment the
> version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
> [PATCH v3] -> ...).
>
> ---
> Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
> Test suite:     http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe
>


[-- Attachment #2: Type: text/html, Size: 4653 bytes --]