break lines in workrave.desktop file for "ru" and "pl" languages

Bug #1176200 reported by Luis Mondesi
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Workrave
Fix Released
Medium
workrave (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

After seeing the following log entries in my log file, I opened the workrave.desktop file and indeed there are new-line chars (\n) in the lines for "pl" and "ru" languages.

The attached patch fixes the problem.

May 3 16:53:56 localhost pulseaudio[3788]: [pulseaudio] conf-parser.c: [/usr/share/applications/workrave.desktop:18] Missing '='.
May 3 16:53:56 localhost pulseaudio[3788]: [pulseaudio] module-augment-properties.c: Failed to parse .desktop file /usr/share/applications/workrave.desktop.

Tags: patch
Revision history for this message
Luis Mondesi (lemsx1) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "fixes new-line characters in workrave.desktop file" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Rajiv Shah (rajivshah3)
Changed in workrave (Ubuntu):
status: New → Confirmed
Revision history for this message
In , Rajiv Shah (rajivshah3) wrote :

--bug report by Luis Mondesi at the Ubuntu Launchpad Bug Tracker--
--https://bugs.launchpad.net/ubuntu/+source/workrave/+bug/1176200--

After seeing the following log entries in my log file, I opened the workrave.desktop file and indeed there are new-line chars (\n) in the lines for "pl" and "ru" languages.

The attached patch fixes the problem.

May 3 16:53:56 localhost pulseaudio[3788]: [pulseaudio] conf-parser.c: [/usr/share/applications/workrave.desktop:18] Missing '='.
May 3 16:53:56 localhost pulseaudio[3788]: [pulseaudio] module-augment-properties.c: Failed to parse .desktop file /usr/share/applications/workrave.desktop.

Changed in workrave:
importance: Unknown → Medium
status: Unknown → Confirmed
Revision history for this message
In , Rob Caelers (rob-caelers) wrote :

Fixed.

(commit c7928a1261716080228e4b5e180bd1c3bead636a)

Changed in workrave:
status: Confirmed → Fix Released
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks Luis, your patch is slightly incorrect (the translation files should be patched) but I'm going to do an upload with the upstream version

Changed in workrave (Ubuntu):
importance: Undecided → Low
status: Confirmed → In Progress
Changed in workrave (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package workrave - 1.10-0ubuntu3

---------------
workrave (1.10-0ubuntu3) saucy; urgency=low

  * debian/patches/git_fix_desktop_bugs.patch:
    - replace previous patch by an improved version from git upstream,
      should fix the parsing errors in the .desktop as well
      (lp: #1176200)
 -- Sebastien Bacher <email address hidden> Wed, 12 Jun 2013 16:26:14 +0200

Changed in workrave (Ubuntu):
status: Fix Committed → 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.