[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openembedded-core
Subject:    Re: [OE-core] [PATCH] Use shutil.move when os.rename fails
From:       "Devendra Tewari" <devendra.tewari () gmail ! com>
Date:       2021-04-28 23:02:32
Message-ID: 74CCF8C8-CE6E-43AA-95F8-F38FB64B656B () gmail ! com
[Download RAW message or body]

Understood - carrying patches is a burden for sure. I'll maintain patches for my own \
narrow use cases at my end, and perhaps submit them upstream.

Thanks,
Devendra

> On 28 Apr 2021, at 18:42, Richard Purdie <richard.purdie@linuxfoundation.org> \
> wrote: 
> On Wed, 2021-04-28 at 07:00 -0300, Devendra Tewari wrote:
> > The gist of it is that Docker uses the overlay2 storage driver by default, 
> > previously it used aufs. Both drivers have the same limitation in rename, 
> > where file rename across layers is handled by moving, but directory rename
> > across layers results in error EXDEV. It is up to user space to handle the error.
> 
> I'm still struggling to see how you're going to patch all the various pieces of
> software out there used during a build to handle EXDEV correctly.
> 
> We have handled some cases of this for a long time where we're moving files
> over to different parts of a filesystem, e.g. /tmp/ to WORKDIR or vice versa.
> 
> This new docker case triggers for renames even within a directory, e.g. 
> /some/path/a -> /some/path/b which is not something that would often trigger
> this.
> 
> I need to say clearly now that we are not carrying/taking patches to patch
> every bit of other software to try and avoid this.
> 
> Cheers,
> 
> Richard
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#151070): https://lists.openembedded.org/g/openembedded-core/message/151070
Mute This Topic: https://lists.openembedded.org/mt/81698791/4454766
Group Owner: openembedded-core+owner@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [openembedded-core@marc.info]
-=-=-=-=-=-=-=-=-=-=-=-



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic