On 01/20/20 11:15, Peter Kjellerstedt wrote:
-----Original Message-----
From: Yu, Mingli <mingli...@windriver.com>
Sent: den 20 januari 2020 04:04
To: Peter Kjellerstedt <peter.kjellerst...@axis.com>; Paul Eggleton
<paul.eggle...@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] ✗ patchtest: failure for Open file with O_CLOEXEC
to avoid fd leak (rev3)
On 01/20/20 10:28, Peter Kjellerstedt wrote:
-----Original Message-----
From: openembedded-core-boun...@lists.openembedded.org <openembedded-
core-
boun...@lists.openembedded.org> On Behalf Of Yu, Mingli
Sent: den 20 januari 2020 02:34
To: Paul Eggleton <paul.eggle...@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] ✗ patchtest: failure for Open file with
O_CLOEXEC
to avoid fd leak (rev3)
On 01/20/20 09:29, Paul Eggleton wrote:
Hi Mingli
On Sunday, 19 January 2020 6:41:49 PM NZDT Yu, Mingli wrote:
On 01/19/20 13:32, Patchwork wrote:
== Series Details ==
Series: Open file with O_CLOEXEC to avoid fd leak (rev3)
Revision: 3
URL : https://patchwork.openembedded.org/series/22123/
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 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 9d7a521971)
Have already rebased on the master branch of
http://git.yoctoproject.org/cgit/cgit.cgi/pseudo/, why still issue
out
the above issue? Any hint?
It's assuming that patches sent to this list are for OE-Core (which
they
generally are). I wasn't aware but apparently the documented place to
send pseudo patches is this list, so I think in this case you can just
ignore patchtest.
Okay, many thanks Paul for your hint!
Thanks,
Cheers,
Paul
But it is probably a good idea to include a "[pseudo]" tag in the
subject
when sending patches related to pseudo to this list, as the expectation
otherwise is that they are for OE-Core.
Thanks Peter! And already add the "pseudo_util.c" tag in the subject in
the v3, please check the mail with the subject "[OE-core] [PATCH v3]
pseudo_util.c: Open file with O_CLOEXEC to avoid fd leak".
That is not the same. The "pseudo_util.c:" is part of the commit subject
and will end up in the Git repository. A "[pseudo]" tag is for anyone
seeing the mail (possibly including patchwork and patchtest), letting
them know that this commit is about pseudo, not OE-Core (similar to the
"[PATCH v3]" tag, which is also information about the commit).
Got it! V4 is incoming.
Thanks,
Thanks,
//Peter
//Peter
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
