Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.

2013-03-19 Thread Peter Bex
On Fri, Mar 15, 2013 at 07:58:02AM +0100, Florian Zumbiehl wrote: > Remove ##sys#expand-home-path as shell expansion has no place in a filesystem > API. I've moved the ~ behaviour to a new procedure in "utils" called "ep" and updated the NEWS file and docs. The signed off patch is attached. I'm

Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.

2013-03-19 Thread John Cowan
Peter Bex scripsit: > BTW: This behaviour isn't really compatible with the shell's, since it > disallows strings formatted like ~username from being expanded to that > user's homedir. I don't think that behavior is worth emulating. For one thing, machines are far less likely to be multi-user now

Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.

2013-03-19 Thread Mario Domenech Goulart
Hi, On Tue, 19 Mar 2013 16:44:24 +0100 Peter Bex wrote: > On Fri, Mar 15, 2013 at 07:58:02AM +0100, Florian Zumbiehl wrote: >> Remove ##sys#expand-home-path as shell expansion has no place in a filesystem >> API. > > I've moved the ~ behaviour to a new procedure in "utils" called "ep" and > upda

Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.

2013-03-19 Thread John Cowan
Mario Domenech Goulart scripsit: > b. Drop ##sys#expand-home-path. Do not implicitly expand ~ and >environment variables in pathnames. Provide a procedure to expand ~ >in pathnames. That's what this patch implements. I suggest: b2. Keep ##sys#expand-home-path, but make it the identi

Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.

2013-03-19 Thread Mario Domenech Goulart
Hi John, On Tue, 19 Mar 2013 22:01:45 -0400 John Cowan wrote: > Mario Domenech Goulart scripsit: > >> b. Drop ##sys#expand-home-path. Do not implicitly expand ~ and >>environment variables in pathnames. Provide a procedure to expand ~ >>in pathnames. That's what this patch implements.

Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.

2013-03-19 Thread John Cowan
Mario Domenech Goulart scripsit: > IMO, in this case we should just remove ##sys#expand-home-path, since it > is an internal procedure. Users should not rely on stability of the > internal API. By keeping ##sys#expand-home-path as an identity > procedure, we'd be just polluting the core and addi

Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.

2013-03-19 Thread Jim Ursetto
The following eggs in SVN use ##sys#expand-home-path, since this functionality is useful but was never exported as an official API: openssl osxattr posix-extras sql-de-lite sqlite3 vfs To be fair, three of those are mine. Jim On Mar 19, 2013, at 8:48 PM, Mario Domenech Goulart wrote: > Hi John