Quickly should warn or display obvious errors if a python module is not found

Bug #482915 reported by Philip Peitsch
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Quickly
Fix Released
High
Unassigned
quickly (Ubuntu)
Fix Released
High
Didier Roche-Tolomelli

Bug Description

Binary package hint: quickly

This is related to bug 482913.

It would be nice if quickly made a bigger deal about python modules not being found. Currently it will happily upload the package to a ppa, in spite of the fact the package is essentially broken upon installation (that is, some of the dependencies need to be manually installed). It would be handy if perhaps quickly prompted the user to confirm this was desired behaviour.

The output for the section I'm referring to looks like:

Searching packages which provide required Python modules:
   desktopcouch.records.record ... python-desktopcouch-records
   desktopcouch.records.server ... python-desktopcouch-records
   feedparser ... [not found]
   gobject ... python-gobject
   gtk ... python-gtk2
   pynotify ... python-notify

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

Thanks for your bug report and your interest in Quickly.

We want to reduce on Quickly 0.4 the verbose mode when building a package we currently have. We can't exit on errors as imports can be made on conditions.

So, we should obviously warn and ask from user to confirm that they want to continue the current process.

Changed in quickly (Ubuntu):
status: New → Triaged
importance: Undecided → High
Changed in quickly:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Alex Goretoy (a1g) wrote :

im getting the same thing, it says this when i run quickly package. not sure why that is, seeing as vte should be included with all ubuntu version, but i may be wrong about that. I haven't checked the built package on a diff machine yet, but i don't think vte will be a big problem, not sure how much more of problem will be if other modules cant be found. I've attached the output of quickly run to this comment. Other than that the package seems to install and run on my machine fine.

vte ... [not found]

also there's another error, not sure if this needs a new bug to be submitted, I'll just append it to this for now.

(setup.py:25389): libgnomevfs-WARNING **: Unable to create ~/.gnome2 directory: Permission denied

and this:

WARNING: the following files are not recognized by DistUtilsExtra.auto:
  Copyright
  courtesyflush.desktop.in
  debian/changelog
  debian/compat
  debian/control
  debian/copyright
  debian/pycompat
  debian/python-module-stampdir/courtesyflush
  debian/rules
  po/courtesyflush.pot

Changed in quickly (Ubuntu):
status: Triaged → In Progress
assignee: nobody → Didier Roche (didrocks)
Revision history for this message
Alex Goretoy (a1g) wrote :

thanks for your great work my friend

Changed in quickly:
status: Triaged → Fix Committed
Changed in quickly:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (6.7 KiB)

This bug was fixed in the package quickly - 0.3.90-0ubuntu1

---------------
quickly (0.3.90-0ubuntu1) lucid; urgency=low

  * New release:
    Quickly core:
    - check now that --template has one argument
    - -- enables to give options to templates (unknown options and arguments
      are still given to templates but -- specify explicity what to give to
      templates commands like -h, --version...)
    - check and enable recreation of the credential if user deletes it on
      Launchpad
    - Use realpath so that we can symlink to the binary in trunk.
      (Jonathan Lange)
    - project_path_not_found is raised when data_path_not_found should be
      (originated from Philip Peitsch's patch). (LP: #497688)
    - importing command in one template from another template is now possible
      support as well "all" keyword to import all commands from a template
      into another one (take care of overridden commands too)
      (LP: #452306, #487301)
    - add an apport hook to get installed templates (LP: #411127)
    - enable option completion for templates
    - handle now version upgrade, launching script for templates and handling
      versionning on them
    - change getstarted wording (LP: #486180) - Grant
    - Quickly ship now "version" tag instead of "format" tag
    - add some templatetools to check X display
    - if a commands exit with 4, do not show "ERROR:" (case of wrong command
      usage, for instance)
    - Rewrote importing module in both bin/quickly and ubuntu-project binary
    - Add get_camel_case_name() and get_sentence_name() to templatetools
    - Refactor some code in pre_create hook()
    - Now dashes and spaces support in project name! Dealing with python and
      packaging issues for you. (LP: #493130)
    - Provide Quickly API (LP: #426480)
    - Fix some issues in the man page
    ubuntu-application Template:
    - Enabling upload to team or user ppa, as well as any ppa.
      Use --ppa ppaname or --ppa team/ppaname
      .quickly file can be used as well with ppa = ppaname or
      ppa = team/ppaname (LP: #470192)
      ppa completion is also available
    - Change BSD header by full license
    - Add MIT license
    - Fixes an error when launchpad display name contains unicode character
      (Petar Vasić)
    - Fix typo in quickly help save (Petar Vasić)
    - Adds logo.png as logo property in about dialog (LP: #478389)
      (Petar Vasić)
    - Remove lp-project-change command and add configure command.
      configure lp-project [project_name] to set/reset lp project binded to
      your Quickly project
      ppa <ppa name> to set a default ppa with your project
    - Fix ~/.selected-editor being ignored (LP: #484730)
    - Fix wrong shell completion when in a template (it wrongly proposed
      all commands followed by templates)
    - Check if we have a X display available before running "run" or launching
      GUI in "create" command.
    - add -- support in quickly run to pass options like --help, --version so
      that Quickly core don't take them.
    - enable preferences save call being called multiple times (Philip Peitsch)
    - Use realpath so that we can symlink to the binary in trunk.
 ...

Read more...

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

Other bug subscribers

Remote bug watches

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