Launchpad itself

Merge lp:~mwhudson/launchpad/git-http-imports-bug-438929 into lp:launchpad

  1. git-http-imports-bug-438929
  2. Merge into devel
Proposed by Michael Hudson-Doyle
Status: Merged
Approved by: Tim Penhey
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp:~mwhudson/launchpad/git-http-imports-bug-438929
Merge into: lp:launchpad
Diff against target: 12 lines (+1/-1)
1 file modified
lib/lp/code/browser/codeimport.py (+1/-1)
To merge this branch: bzr merge lp:~mwhudson/launchpad/git-http-imports-bug-438929
Reviewer Review Type Date Requested Status
Tim Penhey (community) Approve
Review via email: mp+20716@code.launchpad.net

Commit message

Allow the creation of http git imports.

Description of the change

Hi,

bzr-git and dulwich now support http urls for git imports, so we should allow people to create such imports.

No tests, but I tested locally and it works.

Cheers,
mwh

To post a comment you must log in.
Revision history for this message
Tim Penhey (thumper) wrote :

Good enough for me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/lp/code/browser/codeimport.py'
--- lib/lp/code/browser/codeimport.py 2010-01-20 03:19:44 +0000
+++ lib/lp/code/browser/codeimport.py 2010-03-07 20:23:27 +0000
@@ -216,7 +216,7 @@
216 description=_(216 description=_(
217 "The URL of the git repository. The HEAD branch will be "217 "The URL of the git repository. The HEAD branch will be "
218 "imported."),218 "imported."),
219 allowed_schemes=["git"],219 allowed_schemes=["git", "http", "https"],
220 allow_userinfo=False, # Only anonymous access is supported.220 allow_userinfo=False, # Only anonymous access is supported.
221 allow_port=True,221 allow_port=True,
222 allow_query=False,222 allow_query=False,