Package: weather-util
Version: 1.5-1
Severity: normal

Hi,

when setting either --metric or --imperial and the temperature is below 0, the
parsing of the temperature fails, e.g.:

weather -i ID --headers=Temperature -m

results in:

Temperature: -6.0 F (-21.1 C)

instead of:

Temperature: -21.1 C

The same holds also for the --imperial option.

I would suggest to change the regex matching to fix this issue:

--- weather-util-1.5/weather.py 2010-03-19 14:29:34.000000000 +0100
+++ weather-util-1.5_new/weather.py     2012-03-11 16:52:49.000000000 +0100
@@ -85,7 +85,7 @@
       elif units == "metric": line = preamble + hpa + trailer
    # filter lines with both temperatures in the form of "X F (Y C)"
    dual_t = re.match(
-      "(.* )(\d*(\.\d+)? F) \((\d*(\.\d+)? C)\)(.*)",
+      "(.* )([-]?\d*(\.\d+)? F) \(([-]?\d*(\.\d+)? C)\)(.*)",
       line
    )
    if dual_t:



Best regards,
Stefan



-- System Information:
Debian Release: wheezy/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 3.2.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages weather-util depends on:
ii  python          2.7.2-10
ii  python-support  1.0.14

weather-util recommends no packages.

weather-util suggests no packages.



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to