[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [patch] add some formatting to webdav browsing

From: Ben Collins <bcollins_at_debian.org>
Date: 2002-02-19 18:26:13 CET

On Tue, Feb 19, 2002 at 11:02:54AM -0600, cmpilato@collab.net wrote:
> Ben Collins <bcollins@debian.org> writes:
> > This patch actually adds some newlines to the html output. I was trying
> > to test some other changes, but it was pretty difficult to see when the
> > html returned was a single line :)
> >
> > Anyway, this also adds a ".." link to go to the parent directory.
> I love the intent of this patch. I, too, was a bit disappointed by
> the lack of newlines in the HTML output here. I do have to make a
> small complaint about your log message not mentioning and symbol names
> (see "Writing Log Entries" in the HACKING file).

Yeah, I know how to write changelogs :)

> > + if (resource->info->repos_path && resource->info->repos_path[1] != '\0')
> > + ap_fprintf(output, bb, " <li><a href=\"../\">..</a></li>\n");
> Also, I have a question about this portion of the code. I'm not
> currently building in shared object mode (I switch every couple of
> weeks depending on what I'm working on), so I haven't actually tried
> this code out. But I was wondering if the result someone sees in
> their browser when traversing down a few directories, then up the
> chain of '..' links, is something like:
> http://svn.collab.net/repos/svn/trunk
> or something like:
> http://svn.collab.net/repos/svn/trunk/subversion/test/libsvn_fs/../../..
> I would prefer the former, and imagine this is quite possible using
> svn_path_remove_last_component(resource->info->repos_path) to get the
> parent's path (and I suppose that would get appended to some base
> repository URL certainly stashed somewhere).

It's the former. I didn't see anyway of finding out the baseurl (the
rest of the code also uses relative links instead of absolute, so I
assumed it was ok to continue with that).

> Of course, your code may already give me my desired results, in which
> case you can simply reply with, "Don't knock it 'til you've tried it."
> :-)

Hey, it works, and is simple! :)

/       Ben Collins    --    Debian GNU/Linux    --    WatchGuard.com      \
`          bcollins@debian.org   --   Ben.Collins@watchguard.com           '
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:37:08 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.