"bzr cat lp:foo/bar" fails

[Nazo]

using bzr 1.6 on Ubuntu intrepid

$ bzr ls lp:ubufox
...
lp:ubufox/content/overlay.js
...
$ bzr cat lp:ubufox/content/overlay.js
bzr: ERROR: Invalid url supplied to transport: "lp:ubufox/content/overlay.js": Invalid branch identifier: 'ubufox/content/overlay.js'

I think this may be design bug.

[James Westby (james-w)]

Hi,

I can confirm this problem. I agree that it is a bit of design
problem in the launchpad directory service, but I don't think
fixing it should be too hard.

Thanks,

James

[Martin Pool (mbp)]

One way to handle it would be to have less dwim and instead say 'bzr
cat -d lp:ubufox content/overlay.js'. There is another bug asking for
-d support across all commands.

--
Martin <http://launchpad.net/~mbp/>

[Jelmer Vernooij (jelmer)]

It seems like Launchpad shouldn't raise PermissionDenied when a user tries to access a path inside of a branch but rather NoSuchFile.

The following patch to bzr fixes the issue (but could mask other bugs, so is incorrect):

=== modified file 'bzrlib/controldir.py'
--- bzrlib/controldir.py 2011-12-14 12:25:34 +0000
+++ bzrlib/controldir.py 2011-12-16 17:01:05 +0000
@@ -738,6 +738,8 @@
                 return result, urlutils.unescape(a_transport.relpath(url))
             except errors.NotBranchError, e:
                 pass
+ except errors.PermissionDenied:
+ pass
             try:
                 new_t = a_transport.clone('..')
             except errors.InvalidURLJoin: