[PATCH 3 of 6 V2 RFC] localrepo: use the vfs join method to implement the localrepo join method

[Angel Ezquerra]

On Tue, Dec 30, 2014 at 10:26 PM, Pierre-Yves David
<pierre-yves.david at ens-lyon.org> wrote:
>
>
> On 12/26/2014 03:46 AM, Angel Ezquerra wrote:
>>
>> # HG changeset patch
>> # User Angel Ezquerra <angel.ezquerra at gmail.com>
>> # Date 1419360518 -3600
>> #      Tue Dec 23 19:48:38 2014 +0100
>> # Node ID f03bef0e2051c76f7dcfd5f580f58037e9dac50a
>> # Parent  3229dbe4251c8fed502c6c7914e03350dd1c5f5d
>> localrepo: use the vfs join method to implement the localrepo join method
>>
>> This will make it possible to customize the behavior of the join method by
>> changing the vfs class (e.g. by using the recently introduced "unionvfs"
>> class).
>>
>> Note that we could have modified the VFS join methods to accept a set of
>> optional paths in the same way thta the localrepo join method does.
>> However it
>> seemed simpler to simply call os.path.join before calling self.vfs.join.
>
>
> This patch wants to:
> - come first, without and RFC flag,
> - also use the vfs.reljoin() method recently introduced.
>
> Otherwise I'm super happy to see it.
>
> --
> Pierre-Yves David

OK, I just sent a new patch series containing this patch, modified as
you suggested, and a follow up patch that changes
localrepository.wjoin to also use self.vfs.reljoin rather than
os.path.join.

Cheers,

Angel