[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.
From: |
Felix |
Subject: |
Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path. |
Date: |
Fri, 15 Mar 2013 21:30:43 +0100 (CET) |
From: Peter Bex <address@hidden>
Subject: Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.
Date: Fri, 15 Mar 2013 10:01:42 +0100
> On Fri, Mar 15, 2013 at 09:36:01AM +0100, Christian Kellermann wrote:
>> * Florian Zumbiehl <address@hidden> [130315 07:58]:
>> > Remove ##sys#expand-home-path as shell expansion has no place in a
>> > filesystem
>> > API.
>>
>> I'd rather prefer a "no-shell-expansion" feature than breaking the
>> current behaviour. This way programmers can choose when they need
>> a "dumb" implementation (because the name is made up by user input)
>> or whether the old behaviour should work for them.
>
> A saner way of doing it is to have an explicit shell expansion procedure
> through which you can pass the path string. That way, we achieve clean
> separation of functionality.
>
>> Ripping it out like this is not an option.
>
> I agree we can't immediately do it, but we should strive towards a clean,
> safe API in the long run. Just keeping it around is not an option either.
> Adding more parameters or options is ugly, IMHO.
But let's not forget convenience...
cheers,
felix
- [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., (continued)
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Christian Kellermann, 2013/03/15
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Peter Bex, 2013/03/15
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Alaric Snell-Pym, 2013/03/15
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.,
Felix <=
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Peter Bex, 2013/03/15
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Jim Ursetto, 2013/03/15
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Felix, 2013/03/16
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Florian Zumbiehl, 2013/03/16
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Alaric Snell-Pym, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Mario Domenech Goulart, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Felix, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Mario Domenech Goulart, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Florian Zumbiehl, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Peter Bex, 2013/03/18