-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 1/17/2011 12:45 PM, Jelmer Vernooij wrote: > Jelmer Vernooij has proposed merging lp:~jelmer/bzr/lp-qa-staging into lp:bzr. > > Requested reviews: > bzr-core (bzr-core) > > For more details, see: > https://code.launchpad.net/~jelmer/bzr/lp-qa-staging/+merge/46515 > > This adds support for lp://qastaging
from launchpadlib.uris import LPNET_SERVICE_ROOT except ImportError: LPNET_SERVICE_ROOT = 'https://api.launchpad.net/beta/' +try: + from launchpadlib.uris import QASTAGING_SERVICE_ROOT +except ImportError: + QASTAGING_SERVICE_ROOT = 'https://api.qastaging.launchpad.net/'
^- These sure look like things that should be done with getattr() rather than import hacks. Can we change these to:
from launchpadlib import uris
QASTAGING_SERVICE_ROOT = getattr(uris, 'QASTAGING_SERVICE_ROOT', 'https://api.qastaging.launchpad.net')
etc.
launchpadlib may be doing import time hacks, but I doubt it.
The rest looks good to me. I would make sure you manually tested it, but otherwise I'm happy.
merge: approve
John =:->
-----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (Cygwin) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAk011pAACgkQJdeBCYSNAANGrACfc/oUxjzdgBvnVnKj8Cs8VnIf ZuoAoIIxa+r1h36raJcEdpaKaxUIjz87 =thDm -----END PGP SIGNATURE-----
« Back to merge proposal
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 1/17/2011 12:45 PM, Jelmer Vernooij wrote: /code.launchpad .net/~jelmer/ bzr/lp- qa-staging/ +merge/ 46515
> Jelmer Vernooij has proposed merging lp:~jelmer/bzr/lp-qa-staging into lp:bzr.
>
> Requested reviews:
> bzr-core (bzr-core)
>
> For more details, see:
> https:/
>
> This adds support for lp://qastaging
from launchpadlib.uris import LPNET_SERVICE_ROOT SERVICE_ ROOT = 'https:/ /api.launchpad. net/beta/ ' SERVICE_ ROOT SERVICE_ ROOT = 'https:/ /api.qastaging. launchpad. net/'
except ImportError:
LPNET_
+try:
+ from launchpadlib.uris import QASTAGING_
+except ImportError:
+ QASTAGING_
^- These sure look like things that should be done with getattr() rather
than import hacks. Can we change these to:
from launchpadlib import uris
QASTAGING_ SERVICE_ ROOT = getattr(uris, 'QASTAGING_ SERVICE_ ROOT', /api.qastaging. launchpad. net')
'https:/
etc.
launchpadlib may be doing import time hacks, but I doubt it.
The rest looks good to me. I would make sure you manually tested it, but
otherwise I'm happy.
merge: approve
John
=:->
-----BEGIN PGP SIGNATURE----- enigmail. mozdev. org/
11pAACgkQJdeBCY SNAANGrACfc/ oUxjzdgBvnVnKj8 Cs8VnIf r1h36raJcEdpaKa xUIjz87
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAk0
ZuoAoIIxa+
=thDm
-----END PGP SIGNATURE-----