[Toybox] [PATCH] Fix "ls -m".

2017-04-07 Thread enh
No new test; the existing test was already failing. --- toys/posix/ls.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) From e1b38296d521d56f4cc9a17dbbb73dfdafa532b6 Mon Sep 17 00:00:00 2001 From: Elliott Hughes Date: Fri, 7 Apr 2017 23:30:31 -0700 Subject: [PATCH] Fix "ls -m". No new

[Toybox] [PATCH] Speed up tail.test for shells without builtin printf.

2017-04-07 Thread enh
The test only needs echo, and Android currently doesn't use mksh's builtin printf; the toybox printf is fuller featured and less buggy, but a lot slower to call in a loop like this (and mksh doesn't fall back to the printf on the path if given something it can't handle, so we can't have the best of

Re: [Toybox] [PATCH] Make renice.test work without awk.

2017-04-07 Thread enh
oops. accidental "git commit -a" of debugging "printf"s. will come back to losetup properly at some point. On Fri, Apr 7, 2017 at 9:54 AM, Rob Landley wrote: > On 04/03/2017 10:58 AM, enh wrote: > > > > Also try to only kill processes we created. > > --- > > tests/losetup.test | 6 ++ > > t

[Toybox] [PATCH] Fix "make test_split".

2017-04-07 Thread enh
The test shouldn't assume there's a toybox binary on the path. --- tests/split.test | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) From d832e3d9dbe1defe724f42e9acb61a43f7013a8c Mon Sep 17 00:00:00 2001 From: Elliott Hughes Date: Fri, 7 Apr 2017 23:00:54 -0700 Subject: [PATCH] Fix "make tes

Re: [Toybox] Stat, wrong blockcount for F2FS filesystems?

2017-04-07 Thread enh
yeah, there was a stat bug that was fixed. some filesystems modify the block value before passing it back. drosen fixed sdcardfs to look at this in "ANDROID: sdcardfs: Get the blocksize from the lower fs": int f2fs_getattr(struct vfsmount *mnt, struct dentry *dentry, struct kstat *stat) { struct i

Re: [Toybox] Stat, wrong blockcount for F2FS filesystems?

2017-04-07 Thread enh
thanks. replied on the list, s/i/drosen/ :-) On Fri, Apr 7, 2017 at 6:55 PM, enh wrote: > yeah, there was a stat bug that was fixed. some filesystems modify the > block value before passing it back. drosen fixed sdcardfs to look at this > in "ANDROID: sdcardfs: Get the blocksize from the lower f

Re: [Toybox] [PATCH] Remove mount.test awk dependency.

2017-04-07 Thread Rob Landley
On 04/07/2017 11:50 AM, enh wrote: > Yeah, but that's a big job for the future. I am root and can spin up a > new emulator instance and throw it away afterwards at very little cost > (for x86/x86-64). So it's still useful in the usual "half an eye is > better than no eye" sense. Oh sure, I'm just

Re: [Toybox] Stat, wrong blockcount for F2FS filesystems?

2017-04-07 Thread darken
Wrong train of thought, toybox stat doesn't do any blockcount calculation, but just gets the value from `st_blocks`. We are getting bad values from `/storage/emulated/0` which is FUSE mounted from `/data/media/0` and the blockcount values from `/data/media/0` are sane. So this seems to be FUSE re

Re: [Toybox] [PATCH] Make renice.test work without awk.

2017-04-07 Thread Rob Landley
On 04/03/2017 10:58 AM, enh wrote: > > Also try to only kill processes we created. > --- > tests/losetup.test | 6 ++ > tests/renice.test | 4 ++-- > 2 files changed, 8 insertions(+), 2 deletions(-) I applied the renice part of this, but not the losetup part of it. Rob _

Re: [Toybox] [PATCH] Remove mount.test awk dependency.

2017-04-07 Thread enh
Yeah, but that's a big job for the future. I am root and can spin up a new emulator instance and throw it away afterwards at very little cost (for x86/x86-64). So it's still useful in the usual "half an eye is better than no eye" sense. On Apr 7, 2017 09:47, "Rob Landley" wrote: On 04/03/2017 10

Re: [Toybox] [PATCH] Remove mount.test awk dependency.

2017-04-07 Thread Rob Landley
On 04/03/2017 10:35 AM, enh wrote: > > Parsing file(1) output isn't a good way to determine file system type > anyway. The mount tests (which there need to be s many more of, there's comments in mount.c about that) are one of the category of "tests really needing mkroot" because: A) Requires

[Toybox] Stat, wrong blockcount for F2FS filesystems?

2017-04-07 Thread darken
I'm using stat to display filesize and allocated size in my Android app. I calculate the allocated size by multiplying the block count (%b) with 512 Byte. I found out in https://github.com/landley/toybox/issues/41 that the blocksize for the blockcount calculation is hardcoded to 512. So this shou