← Back to team overview

debcrafters-packages team mailing list archive

[Bug 2056595] Re: Lot of SyntaxWarning messages for rhythmbox

 

The "SyntaxWarning" is complaining about the incorrectly escaped string
literal.  Backslash followed by a bracket char, i.e. "\(", is not a
valid to python string literal escape sequence.

It was the regular expression library needed to escape the bracket char.

To do it correctly, change line#171 from 
translation = re.sub('\(..\)', '', translation, flags=re.DOTALL)
to
translation = re.sub('\\(..\\)', '', translation, flags=re.DOTALL)

-- 
You received this bug notification because you are a member of
Debcrafters packages, which is subscribed to rhythmbox in Ubuntu.
https://bugs.launchpad.net/bugs/2056595

Title:
  Lot of SyntaxWarning messages for rhythmbox

Status in rhythmbox package in Ubuntu:
  Confirmed

Bug description:
  During recent updates on Ubuntu 24.04 I see Lot of SyntaxWarning
  messages for rhythmbox

  /usr/lib/rhythmbox/plugins/alternative-toolbar/alttoolbar_plugins.py:171: SyntaxWarning: invalid escape sequence '\('
    translation = re.sub('\(..\)', '', translation, flags=re.DOTALL)
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/AstrawebParser.py:64: SyntaxWarning: invalid escape sequence '\/'
    url = re.split('(\/display[^"]*)', entry)[1]
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/AstrawebParser.py:66: SyntaxWarning: invalid escape sequence '\/'
    title = re.split('(\/display[^>]*)([^<]*)', entry)[2][1:].strip()
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/AstrawebParser.py:97: SyntaxWarning: invalid escape sequence '\/'
    lyrics = re.split('(<font face=arial size=2>)(.*)(<\/font><br></td><td*)', result)[2]
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/JetlyricsParser.py:50: SyntaxWarning: invalid escape sequence '\.'
    m = re.search('<a href=\'(http://jetlyrics\.com/viewlyrics\.php\?id=[0-9]*)\'>', result)
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/JlyricParser.py:49: SyntaxWarning: invalid escape sequence '\.'
    m = re.search('<div class=\'title\'><a href=\'(/artist/[^\.]*\.html)\'>', result)
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/TerraParser.py:39: SyntaxWarning: invalid escape sequence '\w'
    pattern = re.compile("&(#?\w+?);")
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/WinampcnParser.py:87: SyntaxWarning: invalid escape sequence '\['
    lrcplaintext = re.sub('\[.*?\]', '', lrcplaintext)
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/lyrics.py:44: SyntaxWarning: invalid escape sequence '\('
    LYRIC_TITLE_STRIP=["\(live[^\)]*\)", "\(acoustic[^\)]*\)", "\([^\)]*mix\)", "\([^\)]*version\)", "\([^\)]*edit\)", "\(feat[^\)]*\)"]
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/lyrics.py:44: SyntaxWarning: invalid escape sequence '\('
    LYRIC_TITLE_STRIP=["\(live[^\)]*\)", "\(acoustic[^\)]*\)", "\([^\)]*mix\)", "\([^\)]*version\)", "\([^\)]*edit\)", "\(feat[^\)]*\)"]
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/lyrics.py:44: SyntaxWarning: invalid escape sequence '\('
    LYRIC_TITLE_STRIP=["\(live[^\)]*\)", "\(acoustic[^\)]*\)", "\([^\)]*mix\)", "\([^\)]*version\)", "\([^\)]*edit\)", "\(feat[^\)]*\)"]
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/lyrics.py:44: SyntaxWarning: invalid escape sequence '\('
    LYRIC_TITLE_STRIP=["\(live[^\)]*\)", "\(acoustic[^\)]*\)", "\([^\)]*mix\)", "\([^\)]*version\)", "\([^\)]*edit\)", "\(feat[^\)]*\)"]
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/lyrics.py:44: SyntaxWarning: invalid escape sequence '\('
    LYRIC_TITLE_STRIP=["\(live[^\)]*\)", "\(acoustic[^\)]*\)", "\([^\)]*mix\)", "\([^\)]*version\)", "\([^\)]*edit\)", "\(feat[^\)]*\)"]
  /usr/lib/x86_64-linux-gnu/rhythmbox/plugins/lyrics/lyrics.py:44: SyntaxWarning: invalid escape sequence '\('
    LYRIC_TITLE_STRIP=["\(live[^\)]*\)", "\(acoustic[^\)]*\)", "\([^\)]*mix\)", "\([^\)]*version\)", "\([^\)]*edit\)", "\(feat[^\)]*\)"]

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/rhythmbox/+bug/2056595/+subscriptions