Graig <[EMAIL PROTECTED]> wrote:
> I've found that the only way that ptx will correctly use the contents of the
> ignore-file is if that file is formatted in the Unix style with just an octal
> 012 (LF) between lines. If it is formated in the DOS style with octal 015
> (CR) and 012 (LF) between lines, the results are as though the ignore-file
> wasn't specified at all.
>
> Attached is a small zip file that demonstrates the problem. The contents are:
> SplForPtx.txt - sample input for the ptx command
> ptx_ignore.w.txt - an ignore-file formatted in dos mode
> ptx_ignore.u.txt - an ignore-file formatted in unix mode
> MakeSplPtx.bat - a script that will create output using the two different
> ignore files
> ShopTips.w.ptx - output when using the dos formatted ignore-file
> ShopTips.u.ptx - output when using the unix formatted ignore-file
Thanks for reporting that.
This is due to the silly binary/text distinction that developers
everywhere have come to love.
In this case, maybe we can make it easier for the folks using Cygwin.
A lot of the coreutils already do this by calling e.g., fopen with "rb".
But ptx uses open, so here's a proposed patch:
diff --git a/src/ptx.c b/src/ptx.c
index 67b70c6..a5cea72 100644
--- a/src/ptx.c
+++ b/src/ptx.c
@@ -523,7 +523,7 @@ swallow_file_in_memory (const char *file_name, BLOCK *block)
if (using_stdin)
file_handle = STDIN_FILENO;
else
- if ((file_handle = open (file_name, O_RDONLY)) < 0)
+ if ((file_handle = open (file_name, O_RDONLY | O_TEXT)) < 0)
error (EXIT_FAILURE, errno, "%s", file_name);
/* If the file is a plain, regular file, allocate the memory buffer all at
Does that solve your problem?
Adding O_TEXT is a no-op on Unix systems.
_______________________________________________
Bug-coreutils mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/bug-coreutils
