Re: Patch to fix data loss with `tail -F' (bug 6612)

2008-10-06 Thread Jos Backus
On Mon, Oct 06, 2008 at 07:14:02AM +0200, Jim Meyering wrote: Sorry it couldn't make it into 7.0. No need to apologize, Jim. I'm just glad to know this will be fixed - thanks again for working on this. I do apologize for not being able to provide a better patch. The change I have in mind isn't

Re: Patch to fix data loss with `tail -F' (bug 6612)

2008-10-05 Thread Jos Backus
I see that coreutils-7.0 has been released without this change. To recap, I don't think I'm able to provide a more satisfactory patch. Is the consensus that I should just continue to apply my patch locally? That works for me; after all, it does fix my problem, introduces no known regressions that

Re: Patch to fix data loss with `tail -F' (bug 6612)

2008-10-05 Thread Jim Meyering
Jos Backus [EMAIL PROTECTED] wrote: I see that coreutils-7.0 has been released without this change. To recap, I don't think I'm able to provide a more satisfactory patch. Is the consensus that I should just continue to apply my patch locally? That works for me; after all, it does fix my

Re: Patch to fix data loss with `tail -F' (bug 6612)

2008-09-30 Thread Jim Meyering
Jos Backus [EMAIL PROTECTED] wrote: On Mon, Sep 29, 2008 at 10:37:31AM +0200, Andreas Schwab wrote: Jos Backus [EMAIL PROTECTED] writes: The `date' output shows up in the tail output. Also, this change only stops reading from the old file once the new file has some content. At that time

Re: Patch to fix data loss with `tail -F' (bug 6612)

2008-09-30 Thread Jos Backus
Hi Jim, On Tue, Sep 30, 2008 at 10:37:54AM +0200, Jim Meyering wrote: Jos Backus [EMAIL PROTECTED] wrote: On Mon, Sep 29, 2008 at 10:37:31AM +0200, Andreas Schwab wrote: Jos Backus [EMAIL PROTECTED] writes: The `date' output shows up in the tail output. Also, this change only

Re: Patch to fix data loss with `tail -F' (bug 6612)

2008-09-29 Thread Andreas Schwab
Jos Backus [EMAIL PROTECTED] writes: The `date' output shows up in the tail output. Also, this change only stops reading from the old file once the new file has some content. At that time the old file is read until EOF and closed before starting on the new file. What if the old file keeps

Re: Patch to fix data loss with `tail -F' (bug 6612)

2008-09-29 Thread Jos Backus
On Mon, Sep 29, 2008 at 10:37:31AM +0200, Andreas Schwab wrote: Jos Backus [EMAIL PROTECTED] writes: The `date' output shows up in the tail output. Also, this change only stops reading from the old file once the new file has some content. At that time the old file is read until EOF and

Re: Patch to fix data loss with `tail -F' (bug 6612)

2008-09-28 Thread Jos Backus
Hi Jim, Thanks for your feedback so far. You wrote: I think the trick to doing this right is as follows: When a tailed file (name) disappears, and a subsequent open attempt shows the new file is nonexistent or empty, continue tailing the old file descriptor. Only once

Re: Patch to fix data loss with `tail -F'

2008-08-31 Thread Jos Backus
On Mon, Aug 18, 2008 at 08:13:09PM -0700, Jos Backus wrote: https://savannah.gnu.org/patch/index.php?6612 Please let me know if you see any issues with this change; I'd like to see it adopted for the next coreutils release. *bump* -- Jos Backus jos at catnook.com

Re: Patch to fix data loss with `tail -F'

2008-08-31 Thread Jos Backus
On Sun, Aug 31, 2008 at 08:07:54PM +0200, Jim Meyering wrote: Jos Backus [EMAIL PROTECTED] wrote: https://savannah.gnu.org/patch/index.php?6612 Please let me know if you see any issues with this change; I'd like to see it adopted for the next coreutils release. Thank you! You're

Patch to fix data loss with `tail -F'

2008-08-18 Thread Jos Backus
https://savannah.gnu.org/patch/index.php?6612 Please let me know if you see any issues with this change; I'd like to see it adopted for the next coreutils release. -- Jos Backus jos at catnook.com ___ Bug-coreutils mailing list Bug-coreutils@gnu.org