> -----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). > Thanks, > > > //Peter //Peter -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
