PostScript output is invalid on non-english locales

[Traumflug]

[Impact]
Low, Since the bug only replaces a decimal dot with a comma

[Development Fix]
https://launchpadlibrarian.net/103076603/0001-Fixed-command-line-batch-output-for-some-exporters.patch

[Stable Fix]
Probably, https://launchpadlibrarian.net/103076603/0001-Fixed-command-line-batch-output-for-some-exporters.patch

[Text Case]

1. Install PCB (sudo apt-get install pcb)
2. Using a german locale, export to a PostScript using the command line
Broken Behavior: decimal dots inside the file are replaced by commas.
Fixed Behavior:

[Regression Potential]
Low

[Original Report]When exporting or printing to PostScript from the command line, the resulting output is invalid. With a german locale, decimal dots inside the file are replaced by commas.

Fix is known.

$ lsb_release -rd
Description: Ubuntu 12.04 LTS
Release: 12.04

$ apt-cache policy pcb
pcb:
  Installiert: 20110918-4
  Kandidat: 20110918-4
  Versionstabelle:
 *** 20110918-4 0
        500 http://ch.archive.ubuntu.com/ubuntu/ precise/universe amd64 Packages
        100 /var/lib/dpkg/status

[Traumflug (mah-jump-ing)]

Fix attached, it's a one-liner. Taken as-is from pcb's git repository.

[Traumflug (mah-jump-ing)]

Now I hope somebody who knows how to forward this fix into the package update system picks up (or shows me how to do it myself).

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "0001-Fixed-command-line-batch-output-for-some-exporters.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.]

[Traumflug (mah-jump-ing)]

https://code.launchpad.net/~mah-jump-ing/ubuntu/precise/pcb/fix-for-988503/+merge/103688

[Traumflug (mah-jump-ing)]

The previous merge request accidently included a temporary file, a new merge request followed:

https://code.launchpad.net/~mah-jump-ing/ubuntu/precise/pcb/fix-for-988503-2/+merge/103767

[Launchpad Janitor (janitor)]

This bug was fixed in the package pcb - 20110918-5

---------------
pcb (20110918-5) unstable; urgency=low

  * Added 0001-Fixed-command-line-batch-output-for-some-exporters.patch from
    upstream git to fix command line batch output for some exporters.
    Thanks to Traumflug <email address hidden> (LP: #988503)
  * Bumped compat level to 9
  * debian/control: Updated Standards-Version to 3.9.3
  * debian/copyright: Updated copyright format & years.

 -- أحمد المحمودي (Ahmed El-Mahmoudy) <email address hidden> Tue, 08 May 2012 21:30:17 +0200

[Traumflug (mah-jump-ing)]

As of today, this package doesn't show up in my installation:

$ date && dpkg -s pcb
Di 5. Jun 01:04:53 CEST 2012
Package: pcb
Status: install ok installed
[...]
Version: 20110918-4
[...]

I'm subscribed to precise-updates, precise-proposed and precise-backports. What is going wrong?

[Fabrice Coutadeur (fabricesp)]

Hi,

I'm very sorry but I'm not able to reproduce the bug, even after switching to german locale (I'm a spanish user).
And I need to be sure that the fix works before uploading it.

Could you please indicate what is the value of your LANG variable?

Thanks,
Fabrice

[Traumflug (mah-jump-ing)]

To reproduce the bug, you have to use a locale which has something different than a dot for the decimal separator. Also, the bug is sometimes not obvious, as the output looks the same on the first page, but contains invalid PostScript for additional pages.

I've attached my environment, it's the default german environment.