Re: [PATCH] CVS ${HOME} should be an absolute path
- From: Vincent Legoll <vincent legoll gmail com>
- To: Stephen Kennedy <stevek gnome org>
- Cc: meld-list <meld-list gnome org>
- Subject: Re: [PATCH] CVS ${HOME} should be an absolute path
- Date: Fri, 3 Apr 2009 11:12:18 +0200
On Fri, Apr 3, 2009 at 9:09 AM, Stephen Kennedy <stevek gnome org> wrote:
> On Sat, Mar 28, 2009 at 1:49 AM, Vincent Legoll
> <vincent legoll gmail com> wrote:
>> Was that a cut'n'paste mistake, or was it really intended
>> to os.path.join directory with ${HOME} here. I don't
>> understand how it could make sense. Please explain.
>
> Yes, the original code is confusing. Strangely it works ok though
> because of the strange behavior of os.path.join! os.path.join("foo",
> "/bar") == "/bar"
Yes that's what I found, but we shouldn't rely on corner cases, let our
code be straight clean and easily understandable.
--
Vincent Legoll
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]