Skip Menu |

This queue is for tickets about the Archive-Tar CPAN distribution.

Report information
The Basics
Id: 58636
Status: open
Worked: 30 min
Priority: 0/
Queue: Archive-Tar

People
Owner: BINGOS [...] cpan.org
Requestors: mcermak [...] redhat.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



CC: Tomas Hoger <thoger [...] redhat.com>
Subject: New Bug: Archive::Tar->extract_archive() does not return proper exit code
Date: Wed, 23 Jun 2010 10:05:37 -0400 (EDT)
To: bug-Archive-Tar [...] rt.cpan.org
From: Martin Cermak <mcermak [...] redhat.com>
Hello, I would like to report a bug against Archive::Tar: Archive::Tar->extract_archive() does not return proper exit code There is a Red Hat bugzilla here: https://bugzilla.redhat.com/show_bug.cgi?id=606813. Proposed patch you can find here: https://bugzilla.redhat.com/attachment.cgi?id=426194 Further details: # perl -v This is perl, v5.10.1 (*) built for x86_64-linux-thread-multi # uname -a Linux a6 2.6.32-33.el6.x86_64 #1 SMP Thu Jun 3 13:00:03 EDT 2010 x86_64 x86_64 x86_64 GNU/Linux Martin
RT-Send-CC: thoger [...] redhat.com
Hi, Thanks for upstreaming this report (and the patch). Applied and released to CPAN as Archive-Tar-1.62 Also has been updated in blead perl Cheers.
Subject: Re: [rt.cpan.org #58636] New Bug: Archive::Tar->extract_archive() does not return proper exit code
Date: Tue, 29 Jun 2010 10:00:34 +0200
To: bug-Archive-Tar [...] rt.cpan.org
From: Tomas Hoger <thoger [...] redhat.com>
Hi! On Mon, 28 Jun 2010 17:21:28 -0400 BINGOS via RT wrote: Show quoted text
> Thanks for upstreaming this report (and the patch). > > Applied and released to CPAN as Archive-Tar-1.62 > > Also has been updated in blead perl
Did you see any issue with iter fix part? It seems to have been omitted intentionally. Any comments on the return value concerns raised in Red Hat bugzilla bug? Is extract_archive expected to stop extracting when it hits first problem (secure extract mode failure, or failure to create symlink if it already exists) as it does with this patch, or should it rather try to extract as much as possible as previous versions used to do and stash false return value until it tried all files in the archive? Thank you! -- Tomas Hoger / Red Hat