Merge lp:~mwhudson/loggerhead/customizable-branch-location into lp:loggerhead

[Michael Hudson-Doyle]

[Michael Hudson-Doyle (mwhudson)]

Sorry no time to write this, see if you can figure it out :)

[Martin Albisetti (beuno)]

I... don't quite understand.
Also, this looks maybe wrong?

  === modified file 'loggerhead/apps/filesystem.py'
  --- loggerhead/apps/filesystem.py^I2009-01-23 19:00:35 +0000
  +++ loggerhead/apps/filesystem.py^I2009-02-18 10:36:48 +0000
  @@ -32,7 +32,7 @@
               is_root = False
           branch_app = BranchWSGIApp(
               branch, name, {'cachepath': sql_dir}, self.root.graph_cache,
  - is_root=is_root)
  + is_root=is_root, served_url='lp:loggerhead')
           return branch_app.app

[lp:~mwhudson/loggerhead/customizable-branch-location]

273. By Michael Hudson-Doyle on 2009-02-19

oops :)

[Michael Hudson-Doyle (mwhudson)]

>I... don't quite understand.

Sorry. The idea is to be able to customize the location shown in the "To get this branch, use:
bzr branch http://localhost:8080" text -- for example on launchpad we should use a lp:foo url.

> Also, this looks maybe wrong?

Quite right!

[Martin Albisetti (beuno)]

Ah!
Now it makes sense.
Pending that fix of removing "lp...", it's good to go.
A very good idea :)

review: Approve