Re: [PATCH] tests: adjust PATH to include /usr/sbin for filefrag-using tests

2011-11-27 Thread Bob Proulx
Erik Auerswald wrote: > Bernhard Voelker wrote: > > Bob Proulx wrote: > > > I am sure that you like me always add /usr/local/sbin:/usr/sbin:/sbin > > > to your PATH as a normal non-root user. This shields us from most of > > > those problems. And sometimes hides the problem. > > That is what I d

Re: [PATCH] tests: adjust PATH to include /usr/sbin for filefrag-using tests

2011-11-27 Thread Bob Proulx
Bernhard Voelker wrote: > Again, I don't think it is related to sudo. It is /bin/login. You didn't use sudo to run those tests? Okay. That wasn't immediately clear. > Am I missing something? What's the connection to sudo here? The connection to sudo apparently a very early misunderstanding in

[PATCH] tests: add a syntax check for last week's global change

2011-11-27 Thread Jim Meyering
Someone should have dinged me ;-) Last week I made a global change but forgot to add a matching syntax-check rule. >From 5b3e538b7fc193f8e54b16aeb99b48f28744c1db Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Sun, 27 Nov 2011 15:36:06 +0100 Subject: [PATCH] tests: add a syntax check for last w

Re: New coreutil suggestion - tableize

2011-11-27 Thread Guillaume Kielwasser
Thanks Pádraig for your instructive input! > Hmm, I wonder would it be worth augmenting `column -t` from util-linux > to add a --border option and also detect number cols and auto right align? > I guess a --header option would be useful too. I have to admit I missed the '-t' option of column over

[PATCH] maint: update comment gcc-4.6.2 still botches -Wsuggest-attribute=pure

2011-11-27 Thread Jim Meyering
FYI, Worth updating, because if you follow the advice provoked by this warning option (i.e., you declare test.c's advance function with the pure attribute), gcc will then compile test in such a way that the program infloops. >From 040bc8b50fe36120ed3fec10575df2590b081eca Mon Sep 17 00:00:00 2001