[oe] [meta-oe PATCH 1/2] python-idna: Avoid removing '+x' from `.egg-info/` directory

Khem Raj raj.khem at gmail.com
Wed Mar 6 01:50:24 UTC 2019


Hi Maxime,

I think we need to understand it a bit better, eg. should we need to 
change the mode for directories or just files. I Think it deserves a new 
patch, I am going to apply Otavio's patch and then you can try if that
does not work then lets fix it by changing perms for files.

On 3/1/19 8:38 AM, Maxime BORGES wrote:
> For reference:
> 
> https://github.com/openembedded/meta-openembedded/pull/92#issue-216348464
> 
> ------------------------------------------------------------------------
> *De :* Khem Raj <raj.khem at gmail.com>
> *Envoyé :* vendredi 1 mars 2019 16:56:31
> *À :* Otavio Salvador; Maxime BORGES
> *Cc :* Burton, Ross; Otavio Salvador; Meta-OpenEmbedded Mailing listing
> *Objet :* Re: [oe] [meta-oe PATCH 1/2] python-idna: Avoid removing '+x' 
> from `.egg-info/` directory
> Otavio
> 
> Can you resend this patch series, its missing in patchwork, so
> hopefully, it will catch it this time.
> 
> On Mon, Feb 25, 2019 at 11:05 AM Khem Raj <raj.khem at gmail.com> wrote:
>>
>> On Mon, Feb 25, 2019 at 10:38 AM Otavio Salvador
>> <otavio.salvador at ossystems.com.br> wrote:
>> >
>> > On Mon, Feb 25, 2019 at 2:03 PM Burton, Ross <ross.burton at intel.com> wrote:
>> > >
>> > > The original patch doesn't explain what it's doing and is clearly
>> > > going to remove executable bits from directories, which clearly isn't
>> > > right.
>> > >
>> > > I'd say remove it, and *then* if it breaks again fix properly.
>> >
>> > I am with Ross here. It is a really ugly patch and if need, we need a
>> > comment on the code and a proper reason to add it back.
>> >
>>
>> +Maxime for information on the original patch


More information about the Openembedded-devel mailing list