Merge lp:~jameinel/launchpadlib/notice into lp:~launchpad-pqm/launchpadlib/devel

Proposed by John A Meinel
Status: Merged
Merge reported by: Leonard Richardson
Merged at revision: not available
Proposed branch: lp:~jameinel/launchpadlib/notice
Merge into: lp:~launchpad-pqm/launchpadlib/devel
Diff against target: None lines
To merge this branch: bzr merge lp:~jameinel/launchpadlib/notice
Reviewer Review Type Date Requested Status
Paul Hummer (community) Approve
Review via email: mp+8463@code.launchpad.net
To post a comment you must log in.
Revision history for this message
John A Meinel (jameinel) wrote :

This is another minor patch which changes the formatting of the "this application is asking for authorization on your behalf".

When I got that message the first time, it was quite difficult to read. Especially since the URL is intermixed with the rest of the text.

This just changes the formatting to be more readable.

Unfortunately it is built on my win32 patch, since that is the only way I can checkout the sources to modify it on windows.

Revision history for this message
Leonard Richardson (leonardr) wrote :

> This is another minor patch which changes the formatting of the "this
> application is asking for authorization on your behalf".
>
> When I got that message the first time, it was quite difficult to read.
> Especially since the URL is intermixed with the rest of the text.
>
> This just changes the formatting to be more readable.
>
> Unfortunately it is built on my win32 patch, since that is the only way I can
> checkout the sources to modify it on windows.

Looks good, approved.*

Revision history for this message
Paul Hummer (rockstar) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'bootstrap.py'
2--- bootstrap.py 1970-01-01 00:00:00 +0000
3+++ bootstrap.py 2009-07-09 14:02:54 +0000
4@@ -0,0 +1,7 @@
5+#!/usr/bin/env python
6+"""A trivial thunk to call the real bootstrap without needing a symlink."""
7+import os
8+import sys
9+
10+sys.path.insert(0, os.path.join(os.path.dirname(__file__), '_bootstrap'))
11+import bootstrap
12
13=== removed symlink 'bootstrap.py'
14=== target was '_bootstrap/bootstrap.py'
15=== modified file 'src/launchpadlib/launchpad.py'
16--- src/launchpadlib/launchpad.py 2009-03-23 21:50:35 +0000
17+++ src/launchpadlib/launchpad.py 2009-07-09 14:47:46 +0000
18@@ -148,11 +148,11 @@
19 web_root = str(web_root_uri.ensureSlash())
20 authorization_url = credentials.get_request_token(web_root=web_root)
21 webbrowser.open(authorization_url)
22- print ("The authorization page (%s) should be opening in your "
23- "browser. After you have authorized this program to "
24- "access Launchpad on your behalf you should come back "
25- "here and press <Enter> to finish the authentication "
26- "process." % authorization_url)
27+ print "The authorization page:"
28+ print " (%s)" % authorization_url
29+ print "should be opening in your browser. After you have authorized"
30+ print "this program to access Launchpad on your behalf you should come"
31+ print "back here and press <Enter> to finish the authentication process."
32 sys.stdin.readline()
33 credentials.exchange_request_token_for_access_token(web_root)
34 return cls(credentials, service_root, cache, timeout, proxy_info)

Subscribers

People subscribed via source and target branches