From: Duy Nguyen <pclouds@gmail.com>
Date: Sun, 02 Feb 2014 02:23:32 GMT
Subject: Re: [PATCH v4 3/4] setup: Add 'abspath_part_inside_repo' function
Message-ID: <CACsJy8Drf5D273Tfaxpqjg==8jcB3iAaPKM4P0-ErF7mPCC1JQ@mail.gmail.com>
In-Reply-To: <CACsJy8A8hardH7EaopY2Xu5Ob50mew3pJdqAr6fJsqyyhLQYMg@mail.gmail.com>

On Sun, Feb 2, 2014 at 9:19 AM, Duy Nguyen <pclouds@gmail.com> wrote:
>> +       /* check whole path */
>> +       if (strcmp(real_path(path0), work_tree) == 0) {
>> +               *path0 = '\0';
>> +               return 0;
>> +       }
>
> I think this is already handled by the "check if work tree is already
> the prefix" block.

No, the "check if.." block does not do real_path() so we still need it
here. Sorry for the noise.
-- 
Duy