Re: [PATCH] Fix heavily broken relative URI resolution



On Thu, 2006-04-20 at 16:03 +0200, Alexander Larsson wrote:
> On Sun, 2006-03-19 at 17:06 +0100, Christian Neumair wrote:
> > 
> > > I don't know WHY many modules and applications use
> > > gnome_vfs_uri_resolve_relative, they usually shouldn't have to mess
> > > around with this and instead use a symlink resolution helper - maybe
> > > because the old docs claim it has something to do with relative path
> > > resolution, which is only right to a point where the relative URI only
> > > forms a path with segments that exclusively contain unreserved
> > > characters.
> 
> Yeah, we should be using a specific symlink resolver in some places
> instead of resolve_relative.

Actually, isn't this OK? I mean, the relative path should be escaped, so
it won't have any reserved characters.

For instance, this is what nautilus does in
nautilus_file_get_symbolic_link_target_uri():

      file_uri = nautilus_file_get_uri (file);
      escaped_name = gnome_vfs_escape_path_string
                   (file->details->info->symlink_name);

      target = gnome_vfs_uri_make_full_from_relative 
                        (file_uri, escaped_name);
      g_free (file_uri);
      g_free (escaped_name);
      return target;

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's a fast talking umbrella-wielding cyborg in a wheelchair. She's a 
green-fingered motormouth archaeologist who inherited a spooky stately manor 
from her late maiden aunt. They fight crime! 




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]