Re: [PATCH v2] untar: do not exit with error when created directory already exists.

2015-11-23 Thread Pavel Pisa
Hello Chris, On Monday 23 of November 2015 23:58:20 Chris Johns wrote: > On 24/11/2015 2:41 AM, Gedare Bloom wrote: > > Thanks, patch applied to 4.11 and master > > Thanks Pavel and Gedare. A nice result. > > Can the tickets be closed if still open? there should be open new ticket for untar code

Re: [PATCH v2] untar: do not exit with error when created directory already exists.

2015-11-23 Thread Chris Johns
On 24/11/2015 2:41 AM, Gedare Bloom wrote: > Thanks, patch applied to 4.11 and master Thanks Pavel and Gedare. A nice result. Can the tickets be closed if still open? Chris ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/

Re: [PATCH v2] untar: do not exit with error when created directory already exists.

2015-11-23 Thread Gedare Bloom
Thanks, patch applied to 4.11 and master On Mon, Nov 23, 2015 at 4:16 AM, Pavel Pisa wrote: > Hello Chris and Gedare, > > I have updated fix of untar problem which has been > introduced during 4.11 development cycle. > The file in a way of created directory is unlinked > per Chris request. > > Co

Re: [PATCH v2] untar: do not exit with error when created directory already exists.

2015-11-23 Thread Pavel Pisa
Hello Chris and Gedare, I have updated fix of untar problem which has been introduced during 4.11 development cycle. The file in a way of created directory is unlinked per Chris request. Code is duplicated over all untar implementations but I would keep this minimal patch for now. It could be app

[PATCH v2] untar: do not exit with error when created directory already exists.

2015-11-23 Thread Pavel Pisa
The problem exists for both RTEMS untar implementations and their variants: Untar_FromMemory(), Untar_FromFile() and rtems_tarfs_load(). If filesystem object already exists at extracted directory path then if it is directory, creation is ignored. Attempt to delete/unlink object and make directory

Re: [PATCH v2] untar: do not exit with error when created directory already exists (fixes #2413).

2015-09-10 Thread Gedare Bloom
Do you have a test you can contribute? On Thu, Sep 10, 2015 at 3:56 AM, Pavel Pisa wrote: > Hello Gedare and others, > > On Thursday 10 of September 2015 05:00:27 Gedare Bloom wrote: >> Looks good. This should go on 4.11 and master? Someone can commit. > > please commit to both. > >> On Wed, Sep

Re: [PATCH v2] untar: do not exit with error when created directory already exists (fixes #2413).

2015-09-10 Thread Pavel Pisa
Hello Gedare and others, On Thursday 10 of September 2015 05:00:27 Gedare Bloom wrote: > Looks good. This should go on 4.11 and master? Someone can commit. please commit to both. > On Wed, Sep 9, 2015 at 6:09 PM, Pavel Pisa wrote: > > The problem exists for both RTEMS untar implementations and

Re: [PATCH v2] untar: do not exit with error when created directory already exists (fixes #2413).

2015-09-09 Thread Gedare Bloom
Looks good. This should go on 4.11 and master? Someone can commit. On Wed, Sep 9, 2015 at 6:09 PM, Pavel Pisa wrote: > The problem exists for both RTEMS untar implementations and their > variants: Untar_FromMemory(), Untar_FromFile() and rtems_tarfs_load(). > --- > cpukit/libfs/src/imfs/imfs_loa

[PATCH v2] untar: do not exit with error when created directory already exists (fixes #2413).

2015-09-09 Thread Pavel Pisa
The problem exists for both RTEMS untar implementations and their variants: Untar_FromMemory(), Untar_FromFile() and rtems_tarfs_load(). --- cpukit/libfs/src/imfs/imfs_load_tar.c | 12 +++- cpukit/libmisc/untar/untar.c | 18 +- 2 files changed, 28 insertions(+), 2