2007-03-19 07:37:05 |
Jonathan Lange |
bug |
|
|
added bug |
2007-03-19 15:10:37 |
David Allouche |
launchpad-bazaar: status |
Unconfirmed |
Confirmed |
|
2007-03-19 15:10:37 |
David Allouche |
launchpad-bazaar: importance |
Undecided |
Medium |
|
2007-03-19 15:10:37 |
David Allouche |
launchpad-bazaar: statusexplanation |
|
Right, the error messages should be improved.
It's not clear to me what would be the best way, since the current lp:// scheme just builds on the existing branch-reference redirection built into the launchpad webapp. Setting to Medium importance out of indecision. |
|
2007-11-25 23:57:26 |
Jonathan Lange |
launchpad-bazaar: assignee |
|
jml |
|
2007-11-25 23:57:26 |
Jonathan Lange |
launchpad-bazaar: milestone |
|
1.1.12 |
|
2007-11-27 04:26:06 |
Jonathan Lange |
launchpad-bazaar: milestone |
1.1.12 |
|
|
2009-10-20 11:20:33 |
Jonathan Lange |
tags |
|
launchpad |
|
2009-10-20 11:20:41 |
Jonathan Lange |
bzr: assignee |
Jonathan Lange (jml) |
|
|
2010-03-03 12:59:36 |
Jonathan Lange |
tags |
launchpad |
launchpad lp-directory |
|
2010-03-03 13:09:23 |
Jonathan Lange |
tags |
launchpad lp-directory |
launchpad lpurl |
|
2010-10-12 23:51:59 |
Robert Collins |
description |
After mucking around a bit with lp://, I came across these errors for things that I might have expected to work.
jml@rhino:~$ bzr branch lp:no-such-project
bzr: ERROR: Not a branch: http://code.launchpad.net/no-such-project/
jml@rhino:~$ bzr branch lp://subunit
bzr: ERROR: Invalid url supplied to transport: 'lp://subunit'
jml@rhino:~$ bzr branch lp:subunit/fix-stdout
bzr: ERROR: Not a branch: http://code.launchpad.net/subunit/fix-stdout/
jml@rhino:~$ bzr branch lp:~exarkun/subunit/fix-stdout
bzr: ERROR: Not a branch: http://code.launchpad.net/~exarkun/subunit/fix-stdout/
jml@rhino:~/Code/Subunit/trunk$ bzr push lp:subunit/new-branch
bzr: ERROR: Transport operation not possible: http does not support mkdir()
These errors (particularly the last one) should be cleaned. The new errors messages should refer users to 'bzr help launchpad'. |
We'd like to have clear error messages for lp://. JML suggested the following test cases:
jml@rhino:~$ bzr branch lp:no-such-project
bzr: ERROR: Not a branch: http://code.launchpad.net/no-such-project/
jml@rhino:~$ bzr branch lp://subunit
bzr: ERROR: Invalid url supplied to transport: 'lp://subunit'
jml@rhino:~$ bzr branch lp:subunit/fix-stdout
bzr: ERROR: Not a branch: http://code.launchpad.net/subunit/fix-stdout/
jml@rhino:~$ bzr branch lp:~exarkun/subunit/fix-stdout
bzr: ERROR: Not a branch: http://code.launchpad.net/~exarkun/subunit/fix-stdout/
jml@rhino:~/Code/Subunit/trunk$ bzr push lp:subunit/new-branch
bzr: ERROR: Transport operation not possible: http does not support mkdir()
We should add in ones for
branch lp:ubuntu/nopackage
branch lp:ubuntu/maverick/nopackage
push lp:ubuntu/natty/package-which-failed-import
log lp:ubuntu/apport/ChangeLog (or similar - a sub-path query into the repo)
The current design situation is this:
the lp: xmlrpc lookup handles aliased branches (that is product, product-series and package-without-distro urls by generating a link to +branches/, which is then resolved by the smart server over bzr+ssh (or http) using a backend translatePath API.
This API can return informative errors, but the codehosting server is having them squashed into a VFS model because BzrDir.open2.1 (and similar) delegate to the regular BzrDir ControlDir implementation.
If we install a custom BzrDir.open verb (and other similar ones) then we can directly call the translatePath API and only if it succeeds delegate to the regular BzrDir ControlDir implementation.
If translatePath returns an error we can wrap that in FileNotFound for returning to the bzr client; the bzr client should show that error (and we can make sure it does) to the user, including the custom message translatePath generates.
This may require some detailed knowledge of bzr internals <-> codehosting glue. |
|
2010-10-12 23:52:12 |
Robert Collins |
bug task added |
|
launchpad-code |
|
2010-10-20 01:40:07 |
Tim Penhey |
launchpad-code: status |
New |
Triaged |
|
2010-10-20 01:40:10 |
Tim Penhey |
launchpad-code: importance |
Undecided |
Medium |
|
2011-09-22 20:52:58 |
Robert Collins |
bzr: importance |
Medium |
Low |
|
2011-09-22 20:53:09 |
Robert Collins |
bzr: importance |
Low |
Medium |
|
2011-09-22 20:53:13 |
Robert Collins |
launchpad: importance |
Medium |
Low |
|
2017-11-08 22:56:38 |
Jelmer Vernooij |
tags |
launchpad lp-code lpurl |
check-for-breezy launchpad lp-code lpurl |
|
2018-06-02 22:31:48 |
Jelmer Vernooij |
bug task added |
|
brz |
|
2018-06-02 22:32:03 |
Jelmer Vernooij |
brz: status |
New |
Triaged |
|
2018-06-02 22:32:06 |
Jelmer Vernooij |
brz: importance |
Undecided |
Low |
|
2018-06-02 22:32:16 |
Jelmer Vernooij |
tags |
check-for-breezy launchpad lp-code lpurl |
launchpad lp-code lpurl |
|