Quickly

should be able to specify target distribution in ubuntu-application (unable to upload to PPA if not on Ubuntu)

Bug #608191 reported by Umang Varma
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Quickly
Fix Released
Undecided
Umang Varma
quickly (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

After Martin Pitt fixed a bug on Debian I was able to try to upload to a PPA (http://bugs.debian.org/589794). It seems Launchpad will not accept a package whose distribution is not karmic/lucid/maverick/etc.

Since python-mkdebian (on Debian) will now give us a changelog with unstable as the distribution, I suggest any one of the following:

1. ubuntu-template checks if the distribution is unstable and changes it to the current ubuntu distribution if it is
2. quickly allows a --distribution option to commands like share (this is very easy to implement as we will just need to run `dch -D <distribution> --force-distribution ""` and the user will not be prompted to edit the changelog)

I am for the second option. If this is agreed upon, I could try to implement it.

From what I can see, all other hurdles in getting a Debian package uploaded to a PPA on Launchpad from a Debian computer have been cleared with http://bugs.debian.org/589794 and http://bugs.debian.org/589802 being fixed.

Related branches

lp:~umang/quickly/608191-distribution
lp:~umang/quickly/608191-config-dist
Didier Roche-Tolomelli: Approve
Diff: 54 lines (+20/-1)
2 files modified
data/templates/ubuntu-application/configure.py (+14/-1)
data/templates/ubuntu-application/internal/packaging.py (+6/-0)
lp:ubuntu/maverick/quickly
Revision history for this message
Umang Varma (umang) wrote :

I've made a --distribution option. Please do look to see if this seems like the right way to go.

Changed in quickly:
assignee: nobody → Umang Varma (umang)
status: New → In Progress
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

I would say, let's wait for launchpad to accept other distribution than just ubuntu for now and we will look at this branch then. Thanks a lot for working on this!

Revision history for this message
Umang Varma (umang) wrote :

As discussed on IRC, I'll try to make such an option for python-mkdebian.

If and when that is approved, I'll have that switch passed to python-mkdebian. However this will not be configured with a switch to quickly share, but by configuring quickly using `quickly configure target_distribution lucid` (for lucid).

Revision history for this message
Umang Varma (umang) wrote :

Fix for lp:608589 has been merged. I'm going ahead with this bug.

Revision history for this message
Umang Varma (umang) wrote : Re: [Bug 608191] Re: should be able to specify target distribution in ubuntu-application (unable to upload to PPA if not on Ubuntu)

Just wondering, like with the --ppa option, should there be two ways to
specify the target_distribution: one with quickly configure and one with
a --distribution option for quickly share or quickly release?

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Not sure TBH. Like ppa is more "just try for once to push to my buddy ppa", I think --distribution should still be discouraged and kept in the configure, but not switch.

I'll review your patch on Monday. Thanks a lot for your contribution there!

Revision history for this message
Umang Varma (umang) wrote :

On 07/24/2010 06:58 PM, Didier Roche wrote:
> Not sure TBH. Like ppa is more "just try for once to push to my buddy
> ppa", I think --distribution should still be discouraged and kept in the
> configure, but not switch.

That makes sense. Agreed.

> I'll review your patch on Monday. Thanks a lot for your contribution
> there!

Sure. Thanks.

Didier Roche-Tolomelli (didrocks)
Changed in quickly:
status: In Progress → Fix Committed
Changed in quickly (Ubuntu):
status: New → Triaged
Didier Roche-Tolomelli (didrocks)
Changed in quickly:
status: Fix Committed → Fix Released
Revision history for this message
Colin Watson (cjwatson) wrote :

quickly (0.6-0ubuntu1) maverick; urgency=low

  * New upstream release: (LP: #638130)
   + quickly core:
    - enhance API with real case usage: nautilus extension
    - move bzr call to use internal API (Jelmer Vernooij)
    - support user defined template paths (Barry Warsaw) (LP: #585617)
    - quickly quickly now uses import command functionality from the origin
      template
    - fix copyright file (Kyle Nitzsche) (LP: #610096)
   + ubuntu-application-template and derivatives:
    - push the tarball to launchpad instead of the .changes (LP: #594256)
    - move bzr call to use internal API (Jelmer Vernooij)
    - add apache license (Monty Taylor)
    - update and improvment of the tutorial (Ryan Macnish)
    - "$ quickly add indicator" to add indicator support to your project
      (Owais Lone)
    - add yelp dep for tutorial (Umang Varma) (LP: #610026)
    - some fixes to the tutorial (Henrik H.) (LP: #608421)
    - don't add changelog for quickly package command (Umang Varma)
      (LP: #587538)
    - add quickly configure distribution <distro> to release to another
      distribution than your current one (Umang Varma) (LP: #608191)
    - Provide a modeline in generated source (Michael Terry) (LP: #606285)
  * debian/control:
    - bump python-distutils-extra build-dep
    - new core breaks older ubuntu-application template (package command)
      (LP: #635624)
    - quickly-ubuntu-template now depends on yelp (LP: #610026)

 -- Didier Roche <email address hidden> Tue, 14 Sep 2010 17:12:23 +0200

Changed in quickly (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.