Re: [PATCH] in place option for unexpand

2009-09-21 Thread Pádraig Brady
Pádraig Brady wrote: Sami Kerola wrote: Good weekend, If you consider in-line option is useful for unexpand I could wrote another patch for expand command which will do the same thing. Thanks for this. I'm wondering about usefulness to other coreutils. Perhaps fmt,fold,nl,tac,tr would

Re: [PATCH] in place option for unexpand

2009-04-20 Thread Sami Kerola
On Fri, Mar 27, 2009 at 15:20, Sami Kerola kerol...@iki.fi wrote: Well I changed rename to copy that is part of coreutils and use the call in move_mode. I did not send new patch because I want to be sure my employer has signed papers stating that they have no right to my contributions to

Re: [PATCH] in place option for unexpand

2009-04-20 Thread Pádraig Brady
Sami Kerola wrote: On Fri, Mar 27, 2009 at 15:20, Sami Kerola kerol...@iki.fi wrote: Well I changed rename to copy that is part of coreutils and use the call in move_mode. I did not send new patch because I want to be sure my employer has signed papers stating that they have no right to my

Re: [PATCH] in place option for unexpand

2009-03-27 Thread Pádraig Brady
Sami Kerola wrote: Mean while I've continued with a first patch. I am having a hard time with cp_option_init. My source file is created by mkstemp which causes permissions to be 600. What ever I do with preserve permissions bits I always seem to get source file permissions in place. Am I such

Re: [PATCH] in place option for unexpand

2009-03-27 Thread Sami Kerola
2009/3/27 Pádraig Brady p...@draigbrady.com: Sami Kerola wrote: Mean while I've continued with a first patch. I am having a hard time with cp_option_init. My source file is created by mkstemp which causes permissions to be 600. What ever I do with preserve permissions bits I always seem to

Re: [PATCH] in place option for unexpand

2009-03-26 Thread Sami Kerola
On Sun, Mar 22, 2009 at 10:58, Jim Meyering j...@meyering.net wrote: Sami Kerola wrote: On Sun, Mar 22, 2009 at 08:22, Jim Meyering j...@meyering.net wrote: Pádraig Brady wrote: Sami Kerola wrote: Good weekend, If you consider in-line option is useful for unexpand I could wrote another

Re: [PATCH] in place option for unexpand

2009-03-22 Thread Jim Meyering
Pádraig Brady wrote: Sami Kerola wrote: Good weekend, If you consider in-line option is useful for unexpand I could wrote another patch for expand command which will do the same thing. Thanks for this. I'm wondering about usefulness to other coreutils. Perhaps fmt,fold,nl,tac,tr would also

[PATCH] in place option for unexpand

2009-03-21 Thread Sami Kerola
...@iki.fi Date: Sat, 21 Mar 2009 18:23:04 +0100 Subject: [PATCH] in place option for unexpand Cc: kerol...@iki.fi --- src/unexpand.c | 116 +-- 1 files changed, 95 insertions(+), 21 deletions(-) diff --git a/src/unexpand.c b/src/unexpand.c index

Re: [PATCH] in place option for unexpand

2009-03-21 Thread Pádraig Brady
Sami Kerola wrote: Good weekend, If you consider in-line option is useful for unexpand I could wrote another patch for expand command which will do the same thing. Thanks for this. I'm wondering about usefulness to other coreutils. Perhaps fmt,fold,nl,tac,tr would also benefit in addition to