Comment 0 for bug 629646

Revision history for this message
menthurae (menthurae) wrote :

-----------------------------------
Issue Description
-----------------------------------

In relation to "weather-applet" and "libgweather" - over the past several months, the Australian Bureau of Meteorology (BOM) has updated its website to deliver a HTML forecast page for a number of locations around Australia.

Currently - the <zone> tag from "Locations.xml" is used within the "weather-bom.c" to retrieve information from the appropriate bom.gov.au URL which if successful is then parsed in the "bom_parse" function.

With the new HTML forecast pages for various locations however, the data can not be successfully parsed and instead the message "Forecast not currently available for this location." shows up on the Forecast tab of weather-applet.

-----------------------------------
Area of Interest
-----------------------------------

Within "weather-bom.c" - the following line of code points to URLs (depending on location) that now have HTML forecast data rather than plain text...

    url = g_strdup_printf ("http://www.bom.gov.au/cgi-bin/wrap_fwo.pl?%s.txt",
      loc->zone + 1);

-----------------------------------
Suggested Fix
-----------------------------------

I suggest/propose that the URL above be changed to point to plain text documents on the BOM FTP server (ftp://ftp2.bom.gov.au/anon/gen/fwo/) with the appropriate weather station zone.

e.g. for Sydney (Zone IDN10064) - currently the code would point to:

http://www.bom.gov.au/cgi-bin/wrap_fwo.pl?IDN10064.txt

However this is now a HTML forecast, which causes the issue noted above... by pointing the code to:

ftp://ftp2.bom.gov.au/anon/gen/fwo/IDN10064.txt

The plain text forecast can be used without any modification to the other functions/aspects of the code.

-----------------------------------
Patches - Untested!
-----------------------------------

Please see the attached "weather-bom.c" which includes the suggested fix (i.e. patched version). Note it will need to be/should be tested by someone who knows how to recompile the libgweather source.

---------------------------------------------------------------------------------------------------------
System information
---------------------------------------------------------------------------------------------------------
Description: Ubuntu 10.04.1 LTS
Release: 10.04

libgweather-common:
  Installed: 2.30.0-0ubuntu1
  Candidate: 2.30.0-0ubuntu1
  Version table:
 *** 2.30.0-0ubuntu1 0
        500 http://ftp.iinet.net.au/linux/ubuntu/ lucid/main Packages
        100 /var/lib/dpkg/status
---------------------------------------------------------------------------------------------------------