Code review comment for lp:~leonardr/launchpadlib/multiversion

Revision history for this message
Francis J. Lacoste (flacoste) wrote :

On February 15, 2010, Leonard Richardson wrote:
> Leonard Richardson has proposed merging
> lp:~leonardr/launchpadlib/multiversion into lp:launchpadlib.
>
> Requested reviews:
> LAZR Developers (lazr-developers)
>
>
> This branch changes launchpadlib to take advantage of lazr.restful's
> 'version' constructor argument. Instead of a service root of
> http://api.launchpad.net/beta/, you now use a service root of
> http://api.launchpad.net/ and a version of "beta". This makes it much
> easier to specify which version of a web service you want to use.
>
> This branch also changes the default version of the web service to 'devel'.
> Here is the plan (not yet ratified by distro developers):
>
> 1. We get a launchpadlib release based on this branch into Lucid before the
> feature freeze. At this point anyone who writes launchpadlib scripts with
> beta versions of Lucid will be using the unstable 'devel' version of the
> web service.
>
> 2. Before Lucid is released, we make some changes to the Launchpad web
> service and freeze a '1.0' release.
>
> 3. We make a small launchpadlib release that does nothing but change the
> default web service version to '1.0'. Anyone who writes launchpadlib
> scripts with the released version of Lucid will be using the frozen '1.0'
> version of the web service.
>
> This is flacoste's plan; I would prefer to change the default web service
> version to '1.0' in this branch, and not mess with Lucid any more after
> this.
>
> Either way, by releasing a '1.0' version of the web service and tying it to
> Lucid, we will be able to retire the 'beta' version when we retire Karmic.
> If we don't get this done, we won't be able to retire the 'beta' version
> until we retire Lucid.
>

The default for production should still be beta, at least until this code is
deployed to production. Otherwise, anybody will get breakage when they try
launchpadlib on lpnet.

--
Francis J. Lacoste
<email address hidden>

« Back to merge proposal