Ubuntu
pylint package

pylint-gui throws exception when running file or package

Bug #989252 reported by Timo Vanwynsberghe on 2012-04-26

This bug affects 2 people

Affects		Status	Importance	Assigned to	Milestone
	pylint	Unknown	Unknown	auto-www.logilab.org #92584
	pylint (Ubuntu)	Fix Released	Undecided	Unassigned

Bug Description

Open a file or package in pylint-gui and press the "run" button, nothing happens and following exception is printed in the terminal:

Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 551, in __bootstrap_inner
    self.run()
  File "/usr/lib/python2.7/threading.py", line 504, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/usr/lib/python2.7/dist-packages/pylint/gui.py", line 439, in lint_thread
    lint_obj = pylint.lint.Run(args=[module], reporter=reporter, exit=False)
  File "/usr/lib/python2.7/dist-packages/pylint/lint.py", line 874, in __init__
    linter.check(args)
  File "/usr/lib/python2.7/dist-packages/pylint/lint.py", line 496, in check
    self.check_astng_module(astng, walker, rawcheckers)
  File "/usr/lib/python2.7/dist-packages/pylint/lint.py", line 566, in check_astng_module
    checker.process_module(astng)
  File "/usr/lib/python2.7/dist-packages/pylint/checkers/format.py", line 191, in process_module
    self.process_tokens(tokenize.generate_tokens(readline))
  File "/usr/lib/python2.7/dist-packages/pylint/checkers/format.py", line 224, in process_tokens
    self.new_line(tok_type, line, line_num, junk)
  File "/usr/lib/python2.7/dist-packages/pylint/checkers/format.py", line 197, in new_line
    self.check_lines(line, line_num)
  File "/usr/lib/python2.7/dist-packages/pylint/checkers/format.py", line 325, in check_lines
    self.add_message('C0301', line=i, args=(len(line), max_chars))
  File "/usr/lib/python2.7/dist-packages/pylint/checkers/__init__.py", line 97, in add_message
    self.linter.add_message(msg_id, line, node, args)
  File "/usr/lib/python2.7/dist-packages/pylint/utils.py", line 280, in add_message
    self.reporter.add_message(msgid, (path, module, obj, line or 1, col_offset or 0), msg)
  File "/usr/lib/python2.7/dist-packages/pylint/reporters/guireporter.py", line 24, in add_message
    module, obj, line = location[1:]
ValueError: too many values to unpack

ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: pylint 0.25.0-1ubuntu1
ProcVersionSignature: Ubuntu 3.2.0-23.36-generic 3.2.14
Uname: Linux 3.2.0-23-generic x86_64
NonfreeKernelModules: nvidia
ApportVersion: 2.0.1-0ubuntu5
Architecture: amd64
Date: Thu Apr 26 23:18:35 2012
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Beta amd64 (20120421)
PackageArchitecture: all
ProcEnviron:
LANGUAGE=nl_BE:nl
TERM=xterm
PATH=(custom, user)
LANG=nl_BE.UTF-8
SHELL=/bin/bash
SourcePackage: pylint
UpgradeStatus: No upgrade log present (probably fresh install)

Tags:

Revision history for this message

Timo Vanwynsberghe (timovwb) wrote on 2012-04-26:

Dependencies.txt Edit (843 bytes, text/plain; charset="utf-8")

Revision history for this message

Timo Vanwynsberghe (timovwb) wrote on 2012-04-27:

pylint-gui-fix.patch Edit (673 bytes, text/plain)

Revision history for this message

Timo Vanwynsberghe (timovwb) wrote on 2012-04-27:

Pylint bugreport: http://www.logilab.org/ticket/92584

Revision history for this message

Ubuntu Foundations Team Bug Bot (crichton) wrote on 2012-04-27:

The attachment "pylint-gui-fix.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags:

added: patch

Revision history for this message

Hans Joachim Desserud (hjd) wrote on 2012-04-29:

I haven't looked at the patch, but I can confirm I get the same error message when attempting to run pylint-gui on a file.

Changed in pylint (Ubuntu):
status:	New → Confirmed

Revision history for this message

Hans Joachim Desserud (hjd) wrote on 2012-04-29:

(on Ubuntu 12.04. Sorry I didn't include this in the comment above)

Logan Rosen (logan) on 2012-05-06

no longer affects:

pylint

Revision history for this message

MestreLion (mestrelion) wrote on 2012-11-22:

Is it possible for Ubuntu to release a new version containing this patch, for 12.04 precise and up?

- This patch has the same content of what was applied upstream:
https://bitbucket.org/logilab/pylint/changeset/b68f274aaab72edcb4cced3d8f4d60bd9d1f0353

- Upstream already released debian packages for 0.25.2 and 0.26, but none of them made into debian's unstable yet (only 0.26, in experimental)

- This is a severe usability issue: pyling-gui simple does not work at all without this patch.

If it is possible to release an "0.25.1-1~0ubuntu1" , what would the procedure be?

If not, how can I apply the patch myself in a way that future releases of pylint (if any) will install cleanly, without any dpkg conflicts?

Thanks!

Revision history for this message

Pander (pander) wrote on 2015-06-14:

Does this still needs to be fixed?

Revision history for this message

Hans Joachim Desserud (hjd) wrote on 2015-06-16:

>Does this still needs to be fixed?

Thanks for the reminder. :) I checked now and I am no longer able to trigger this issue with pylint 1.1.0-1 on Ubuntu 14.04 or later. This newer version presumably contains the upstream fix, so I'm closing this as fix released.

However, pylint 0.25.0-1ubuntu1 on Ubuntu 12.04 is still affected and this is a rather serious issue. If someone need a fix for the bug for 12.04, please perform as much as possible of the SRU Procedure [1] to bring the need to a developer's attention. I may be able to help out a bit here with preparing a debdiff with the attached patch. :)

[1]: https://wiki.ubuntu.com/StableReleaseUpdates#Procedure