Merge lp://staging/~mnordhoff/loggerhead/relative-links into lp://staging/loggerhead

Proposed by Matt Nordhoff
Status: Merged
Approved by: Martin Albisetti
Approved revision: 419
Merged at revision: 416
Proposed branch: lp://staging/~mnordhoff/loggerhead/relative-links
Merge into: lp://staging/loggerhead
Diff against target: 101 lines (+19/-14)
5 files modified
loggerhead/apps/branch.py (+10/-5)
loggerhead/apps/config.py (+2/-2)
loggerhead/apps/transport.py (+2/-2)
loggerhead/controllers/download_ui.py (+2/-2)
loggerhead/templates/atom.pt (+3/-3)
To merge this branch: bzr merge lp://staging/~mnordhoff/loggerhead/relative-links
Reviewer Review Type Date Requested Status
Martin Albisetti Approve
Review via email: mp+24767@code.staging.launchpad.net

Commit message

Generate relative links where possible (Michael Hudson, Matt Nordhoff)

Description of the change

Quoting https://code.edge.launchpad.net/~mwhudson/loggerhead/relative-links/+merge/15298:

> This branch generates relative links apart from in redirects.

*This* branch fixes the issues I noticed when reviewing that branch many moons ago, namely that absolute links are necessary in some parts of the Atom feed. The original branch fixed one HTTP redirect to be absolute; this branch fixes the rest too.

There may be other issues. I dunno.

To post a comment you must log in.
Revision history for this message
Martin Albisetti (beuno) :
review: Approve
420. By Matt Nordhoff

NEWS

421. By Matt Nordhoff

Adjust NEWS

Revision history for this message
Robert Collins (lifeless) wrote :

Looks good enough to eat, to me.

Revision history for this message
Glen Mailer (glenjamin) wrote :

This appears to have broken the "To get this branch, use:" part, which i suspect will need to be special cased.

Updating diff...

An updated diff will be available in a few minutes. Reload to see the changes.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches