Re: [ccache] ccache problem with CCACHE_BASEDIR

2012-07-28 Thread Eric Blau
On 2012-07-28 11:24, Joel Rosdahl wrote: Eric, I've now applied your patch with some tweaks in 2df269a3121889ebcdfa5d98dfb4d675f690e039 (see http://git.samba.org/?p=ccache.git;a=commit;h=2df269a3121889ebcdfa5d98dfb4d675f690e039). Thanks! Thanks for fixing this up the patch and applying it. I

Re: [ccache] ccache problem with CCACHE_BASEDIR

2012-07-20 Thread Eric Blau
On 2012-07-20 13:09, Joel Rosdahl wrote: Thanks. Two things: 1. The patch canonicalizes the "from" parameter, but I think that the "to" parameter should be canonicalized as well, for two reasons: a) realpath() expands symlinks, and comparing (see common_dir_prefix_length) an expanded path with a

Re: [ccache] ccache problem with CCACHE_BASEDIR

2012-07-19 Thread Eric Blau
On 2012-06-11 07:54, Eric Blau wrote: On 2012-06-10 16:09, Joel Rosdahl wrote: > My suggestion is that get_relative_path essentially should stay the > same and continue to assume that the paths are well-formed and > canonical. make_relative_path (the only caller of get_relative_path)

Re: [ccache] ccache problem with CCACHE_BASEDIR

2012-06-11 Thread Eric Blau
On 2012-06-10 16:09, Joel Rosdahl wrote: My suggestion is that get_relative_path essentially should stay the same and continue to assume that the paths are well-formed and canonical. make_relative_path (the only caller of get_relative_path) should canonicalize current_working_dir and path by ca