[Toybox] [PATCH] Fix file.c build and fix sense of fstat test.

enh enh at google.com
Mon Feb 29 16:25:10 PST 2016


what's checked in now still doesn't work in the regular file case though...


Subject: [PATCH] Fix file "cannot open".

We don't want to _always_ fall through.
---
 toys/pending/file.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

On Mon, Feb 29, 2016 at 1:15 PM, Rob Landley <rob at landley.net> wrote:
> Already checked these two in before I saw this. (I checked in the
> architecture name list redo last night because the changes to file were
> turning into a rathole, but "I tested this and it worked" only means
> something if you haven't changed it since. Sigh...)
>
> It should be working now. And I can make test cases that check /dev is a
> directory, /dev/zero is a char device, /dev/loop0 is a block device (as
> standard of one as we're likely to get anyway), mkfifo a fifo, ln -s
> makes a symlink...
>
> Huh,
>
> $ file /usr/lib/firefox/firefox.sh
> /usr/lib/firefox/firefox.sh: POSIX shell script, ASCII text executable
>
> According to posix that should reply "commands text". Once again, posix
> is completely ignored in practice.
>
> Sigh,
>
> Rob



-- 
Elliott Hughes - http://who/enh - http://jessies.org/~enh/
Android native code/tools questions? Mail me/drop by/add me as a reviewer.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-file-cannot-open.patch
Type: text/x-patch
Size: 993 bytes
Desc: not available
URL: <http://lists.landley.net/pipermail/toybox-landley.net/attachments/20160229/97cbd49c/attachment-0005.bin>


More information about the Toybox mailing list