Re: [Toybox] Add remaining pwd options

2013-01-13 Thread Felix Janda
Sorry for this repeated hair-splitting. On 01/12/13 at 11:33pm, Rob Landley wrote: On 01/10/2013 02:25:13 PM, Felix Janda wrote: On 01/02/13 at 12:41am, Rob Landley wrote: What I did was disable #3 in the case where cwd doesn't exist. So the new rule #3 is: 3) If cwd exists and

Re: [Toybox] Add remaining pwd options

2013-01-13 Thread Rob Landley
On 01/13/2013 04:34:56 AM, Felix Janda wrote: Sorry for this repeated hair-splitting. Eh, it happens. :) I'm constantly mucking about in areas I'm brand new to (or haven't got the background for, or last messed with so long ago I've forgotten important things, or while massively

Re: [Toybox] Add remaining pwd options

2012-12-31 Thread Felix Janda
On 12/30/12 at 05:47pm, Rob Landley wrote: On 12/30/2012 05:16:41 AM, Felix Janda wrote: On 12/30/12 at 04:43am, Rob Landley wrote: POSIX contains many surprises. In the section on environment variables it says that $PWD should be set if pwd -P was specified. What happens if an

Re: [Toybox] Add remaining pwd options

2012-12-30 Thread Rob Landley
On 12/29/2012 07:38:24 AM, Felix Janda wrote: POSIX says that pwd should behave the same as pwd -L. The current pwd -P should behave the same way as the previous version of pwd. It just returns the getcwd() output. pwd -L does just check whether the environment variable PWD is also a valid

Re: [Toybox] Add remaining pwd options

2012-12-30 Thread Rob Landley
On 12/30/2012 04:47:13 AM, Felix Janda wrote: Thanks for the various clarifications and making pwd -L check for dot and dot-dot as described in the standard. Looking at the POSIX man page toysh should set $PWD at some point, too. Right now we have toysh is hugely incomplete and I just

Re: [Toybox] Add remaining pwd options

2012-12-29 Thread Rob Landley
On 12/28/2012 03:24:17 PM, Felix Janda wrote: Hi, the first patch adds the -L and -P options to pwd as specified by POSIX. The test script again uses stat. This time in order to get inode numbers of directories. For future reference adding the test in the same commit as the changes

Re: [Toybox] Add remaining pwd options

2012-12-29 Thread Rob Landley
On 12/29/2012 07:38:24 AM, Felix Janda wrote: On 12/29/12 at 03:53am, Rob Landley wrote: On 12/28/2012 03:24:17 PM, Felix Janda wrote: Hi, the first patch adds the -L and -P options to pwd as specified by POSIX. The test script again uses stat. This time in order to get inode