Merge lp:~cjwatson/brz/remove-lp-workaround into lp:brz

Proposed by Colin Watson
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~cjwatson/brz/remove-lp-workaround
Merge into: lp:brz
Diff against target: 13 lines (+0/-3)
1 file modified
breezy/git/__init__.py (+0/-3)
To merge this branch: bzr merge lp:~cjwatson/brz/remove-lp-workaround
Reviewer Review Type Date Requested Status
Jelmer Vernooij Approve
Review via email: mp+393710@code.launchpad.net

Commit message

Remove breezy.git workaround for bazaar.launchpad.net.

Description of the change

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) :
review: Approve
Revision history for this message
The Breezy Bot (the-breezy-bot) wrote :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'breezy/git/__init__.py'
--- breezy/git/__init__.py 2020-07-18 23:14:00 +0000
+++ breezy/git/__init__.py 2020-11-13 11:16:32 +0000
@@ -178,9 +178,6 @@
178 # GitHub requires we lie.178 # GitHub requires we lie.
179 # https://github.com/dulwich/dulwich/issues/562179 # https://github.com/dulwich/dulwich/issues/562
180 headers["User-Agent"] = user_agent_for_github()180 headers["User-Agent"] = user_agent_for_github()
181 elif host == "bazaar.launchpad.net":
182 # Don't attempt Git probes against bazaar.launchpad.net; pad.lv/1744830
183 raise brz_errors.NotBranchError(transport.base)
184 resp = transport.request('GET', url, headers=headers)181 resp = transport.request('GET', url, headers=headers)
185 if resp.status in (404, 405):182 if resp.status in (404, 405):
186 raise brz_errors.NotBranchError(transport.base)183 raise brz_errors.NotBranchError(transport.base)

Subscribers

People subscribed via source and target branches