Bug 6622

Summary:	[xfce4-weather-plugin] Fix licence key / new forecast provider
Product:	Mageia	Reporter:	Raphael Groner <raphgro>
Component:	RPM Packages	Assignee:	Jani Välimaa <jani.valimaa>
Status:	RESOLVED FIXED	QA Contact:
Severity:	major
Priority:	Normal	Keywords:	PATCH, UPSTREAM
Version:	Cauldron
Target Milestone:	---
Hardware:	i586
OS:	Linux
URL:	https://bugzilla.xfce.org/show_bug.cgi?id=8105
Whiteboard:
Source RPM:	xfce4-weather-plugin	CVE:
Status comment:

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