Bug 6622 - [xfce4-weather-plugin] Fix licence key / new forecast provider
Summary: [xfce4-weather-plugin] Fix licence key / new forecast provider
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: i586 Linux
Priority: Normal major
Target Milestone: ---
Assignee: Jani Välimaa
QA Contact:
URL: https://bugzilla.xfce.org/show_bug.cg...
Whiteboard:
Keywords: PATCH, UPSTREAM
Depends on:
Blocks:
 
Reported: 2012-06-29 15:45 CEST by Raphael Groner
Modified: 2012-06-29 21:52 CEST (History)
0 users

See Also:
Source RPM: xfce4-weather-plugin
CVE:
Status comment:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Raphael Groner 2012-06-29 15:45:04 CEST 
Description of problem:
Upstream has a patch to make the weather plugin work again, using another forecast provider.

Version-Release number of selected component (if applicable):


How reproducible:
yes

Steps to Reproduce:
1. place the weather plugin on a panel
2.
3.

Upstream:
https://bugzilla.xfce.org/show_bug.cgi?id=8105
Manuel Hiebel 2012-06-29 17:40:47 CEST

Keywords: (none) => PATCH, UPSTREAM
Assignee: bugsquad => jani.valimaa
Source RPM: (none) => xfce4-weather-plugin

Comment 1 Jani Välimaa 2012-06-29 17:52:13 CEST 
We already have the patch [1], patch was added about 7 months ago. For me, plugin is working OK.

[1] http://svnweb.mageia.org/packages/cauldron/xfce4-weather-plugin/current/SOURCES/xfce4-weather-plugin-0.7.4-working-XOAP-license-key.patch?revision=169949&view=markup
Comment 2 Raphael Groner 2012-06-29 18:04:14 CEST 
(In reply to comment #1)
> We already have the patch [1], patch was added about 7 months ago. For me,
> plugin is working OK.
> 
> [1]
> http://svnweb.mageia.org/packages/cauldron/xfce4-weather-plugin/current/SOURCES/xfce4-weather-plugin-0.7.4-working-XOAP-license-key.patch?revision=169949&view=markup

Sorry, but I can't confirm. The upstream patch is a different one. Harald fixed the plugin with  getting data from another forecast provider. Please look into the upstream bug report.

Futher, using the old provider, there's a licencing issue.

I am using Cauldron (Mageia3 branch), maybe that information helps.
Comment 3 Jani Välimaa 2012-06-29 18:13:03 CEST 
I'll test the new provider patch. IIRC I tested it before with no luck.
Comment 4 Raphael Groner 2012-06-29 19:00:20 CEST 
(In reply to comment #3)
> I'll test the new provider patch. IIRC I tested it before with no luck.

It should work with latest git. I used latest git clone to test.
Comment 5 Jani Välimaa 2012-06-29 19:03:53 CEST 
(In reply to comment #4)
> (In reply to comment #3)
> > I'll test the new provider patch. IIRC I tested it before with no luck.
> 
> It should work with latest git. I used latest git clone to test.

Yes, just build it with latest git snapshot. I guess I'll switch to git snapshot + Harald's patch untill the new version (including Harald's work) is released.
Comment 6 Jani Välimaa 2012-06-29 19:13:27 CEST 
Pushed latest git snapshot with Harold's patch [1]. Please test.

[1] xfce4-weather-plugin-0.7.4-6.git20120629.1
Comment 7 Raphael Groner 2012-06-29 21:52:26 CEST 
(In reply to comment #6)
> Pushed latest git snapshot with Harold's patch [1]. Please test.
> 
> [1] xfce4-weather-plugin-0.7.4-6.git20120629.1

It works. Approved. Thanks for the fast fix.
Raphael Groner 2012-06-29 21:52:37 CEST

Status: NEW => RESOLVED
Resolution: (none) => FIXED
Note You need to log in before you can comment on or make changes to this bug.