Re: [Bug-tar] [PATCH] tar: simplify code in system.c

bug-tar

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-tar] [PATCH] tar: simplify code in system.c

From:	Sergey Poznyakoff
Subject:	Re: [Bug-tar] [PATCH] tar: simplify code in system.c
Date:	Wed, 20 Feb 2013 15:09:27 +0300

Pavel Raiskup <address@hidden> ha escrit:

> Do not call wordsplit on multiple places when not necessary.
> 
> (run_decompress_program): Do not call wordsplit - rather reuse xexec.

That's wrong.  The intent of the function is to try all matching
decompressors and find the one which works. Xexec cannot be used here
because it exits if exec fails.

Regards,
Sergey

[Prev in Thread]

Current Thread

[Next in Thread]

Re: [Bug-tar] Allow tar to use command for compressing consisiting of multiple words., Sergey Poznyakoff, 2013/02/10
- [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.), Pavel Raiskup, 2013/02/20
  - [Bug-tar] [PATCH] tar: simplify code in system.c, Pavel Raiskup, 2013/02/20
    - Re: [Bug-tar] [PATCH] tar: simplify code in system.c, Sergey Poznyakoff <=
  - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.), Sergey Poznyakoff, 2013/02/20
    - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.), Pavel Raiskup, 2013/02/20
    - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.), Sergey Poznyakoff, 2013/02/20
    - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.), Pavel Raiskup, 2013/02/20
    - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.), Sergey Poznyakoff, 2013/02/20
    - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.), Paul Eggert, 2013/02/20
    - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.), Sergey Poznyakoff, 2013/02/20
    - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit, Eric Blake, 2013/02/20
    - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit, Joerg Schilling, 2013/02/21
    - Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit, Paul Eggert, 2013/02/21

Prev by Date: Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.)
Next by Date: Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.)
Previous by thread: [Bug-tar] [PATCH] tar: simplify code in system.c
Next by thread: Re: [Bug-tar] [PATCH 0/1] Notes for wordsplit (was: Allow tar to usecommand for compressing consisiting of multiple words.)
Index(es):
- Date
- Thread