Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(313)

Issue 5414001: Remove more broken urlparse.urljoin examples. (Closed)

Created:
10 years ago by davidjames
Modified:
9 years, 4 months ago
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

Remove more broken urlparse.urljoin examples. urlparse.urljoin('scheme:foo/bar/', 'baz') == 'baz', so it doesn't work the way I expected. Instead, let's use regular string joins. TBR, because the tree is broken as a result. BUG=chromium-os:5311 TEST=Examine example Packages file, run buildbot. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=2df2dd5

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -4 lines) Patch
M chromite/lib/binpkg.py View 4 chunks +3 lines, -4 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
diandersAtChromium
10 years ago (2010-12-02 01:05:44 UTC) #1
LGTM

Powered by Google App Engine
This is Rietveld 408576698