SyntaxWarning with obviously buggy code

Bug #1870947 reported by Zygmunt Krynicki
26
This bug affects 4 people
Affects Status Importance Assigned to Milestone
dstat (Debian)
Fix Released
Unknown
dstat (Ubuntu)
Confirmed
Undecided
Unassigned

Bug Description

Updating dstat to version 0.7.4-6

I see this:

/usr/share/dstat/dstat_mysql_keys.py:41: SyntaxWarning: 'str' object is not callable; perhaps you missed a comma?
  if op.debug > 1: print('%s: exception' (self.filename, e))
/usr/share/dstat/dstat_squid.py:48: SyntaxWarning: 'str' object is not callable; perhaps you missed a comma?
  if op.debug > 1: print('%s: exception' (self.filename, e))

Without even looking deeper that's an obviously missing comma, after "exception'", on both lines.

Tags: focal
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Confirmed. I get the same warnings when installing dstat 0.7.4-6 on Ubuntu Focal.

Changed in dstat (Ubuntu):
status: New → Confirmed
tags: added: focal
Changed in dstat (Debian):
status: Unknown → New
Revision history for this message
Pranshu J (pmj642) wrote :

I would like to send a pull request of the fix. How can I do that?

Revision history for this message
Tom Reynolds (tomreyn) wrote :

https://packaging.ubuntu.com/html/fixing-a-bug.html probably still applies.

However, expanding the version numbers for recent releases on https://launchpad.net/ubuntu/+source/dstat shows, in "Publishing details", shows:
"Copied from debian sid in Primary Archive for Debian GNU/Linux."

So this is a direct copy from Debian - in such cases it's good to check whether this very bug has already been fixed in Debian, in which case another sync to Ubuntu may be in order. Or whether you can contribute the fix to Debian, who have kindly provided the package, then have Ubuntu sync the updated package again. See the right-hand column on https://packages.debian.org/sid/dstat for a link to the Debian bug tracker and additional info on the package state in Debian.

https://wiki.ubuntu.com/SyncRequestProcess

Changed in dstat (Debian):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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