Bug 3399

Summary:	kdesvn requires cervisia that is for cvs
Product:	Mageia	Reporter:	Angelo Naselli <anaselli>
Component:	RPM Packages	Assignee:	QA Team <qa-bugs>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	Normal	CC:	davidwhodgins, juan.baptiste, sysadmin-bugs, tmb
Version:	1	Keywords:	Triaged, validated_update
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	kdesvn	CVE:
Status comment:

Description Angelo Naselli 2011-11-20 23:31:57 CET

Installing kdesvn, cervisia is offered as requirement.
Cervisia is kde graphical CVS client, so i found odd this requirement.

Anyway "googling" a bit i found:
http://kdesvn.alwins-world.de/ticket/25
in which it seems that only cervisia icons are required.

With the help of mikala we found:
urpmq --wget -l kdesvn | grep png | wc -l
258
icons concerning kdesvn, maybe that dependency should be removed
or cervisia package splitted to get only the icon dependency.

Comment 1 Manuel Hiebel 2011-11-21 19:07:17 CET

Hi, thanks for reporting this bug.
Assigned to the package maintainer.

Keywords: (none) => Triaged
Assignee: bugsquad => juan.baptiste

Comment 2 Juan Luis Baptiste 2011-11-21 21:32:45 CET

Ok I'll look into this tonight.

Status: NEW => ASSIGNED

Comment 3 Juan Luis Baptiste 2011-11-22 01:23:41 CET

Ok, 1.5.5-3 on cauldron removes requires for cervisia and adds suggests for kdiff3 and kompare as discussed on IRC today.

Should this fix be sent to updates_testing ?

Comment 4 Angelo Naselli 2011-11-22 09:56:29 CET

Juan, imo yes. Let me know when it is there, thanks.

Comment 5 Juan Luis Baptiste 2011-11-22 18:04:46 CET

Ok, pushed to updates_testing, it will be available soon.

Comment 6 Angelo Naselli 2011-11-23 21:59:33 CET

Tested, the bug is fixed as far as i can say. 
Please reassign the bug to qa team to with an advisory test to make it available for updates.

Comment 7 Juan Luis Baptiste 2011-11-25 05:47:32 CET

Advisory:

A not required dependency for kdesvn Subversion client was found, which was forcing the installation of unneeded packages. This update remove those unneeded dependencies.

Comment 8 Remco Rijnders 2011-11-25 06:44:11 CET

Hi Juan Luis,

If I might be so free, I'd reword the advisory a bit: 

An unrequired dependency for the kdesvn subversion client was found, which resulted in the installation of unneeded packages. This update remove those
unneeded dependencies.

Assignee: juan.baptiste => qa-bugs
Source RPM: kdesvn-1.5.5-2.mga1.src.rpm => kdesvn

Remco Rijnders 2011-11-25 06:44:32 CET

CC: (none) => juan.baptiste

Comment 9 Juan Luis Baptiste 2011-11-25 07:32:28 CET

(In reply to comment #8)
> Hi Juan Luis,
> 
> If I might be so free, I'd reword the advisory a bit: 
> 
> An unrequired dependency for the kdesvn subversion client was found, which
> resulted in the installation of unneeded packages. This update remove those
> unneeded dependencies.

It's ok for me :)

Comment 10 Dave Hodgins 2011-11-26 05:17:26 CET

Testing complete in i586 for the srpm
kdesvn-1.5.5-2.1.mga1.src.rpm

I uninstalled cervisia, then installed kdesvn.  It did not
include cervisia.

I then confirmed kdesvn works by starting it from the Development/Tools
menu entry, and used Subversion/General/Checkout a repository to
get a copy of http://tortoisesvn.googlecode.com/svn/trunk/ to
a new directory on my hard drive.

Juan, are you using i586 or x86-64?

CC: (none) => davidwhodgins

Comment 11 Angelo Naselli 2011-11-26 11:52:40 CET

Dave (comment #10) my test was done using x86_64 arch.

Comment 12 Dave Hodgins 2011-11-26 23:09:11 CET

(In reply to comment #11)
> Dave (comment #10) my test was done using x86_64 arch.

Thanks. Validating the update.

Could someone from the sysadmin team push the srpm
kdesvn-1.5.5-2.1.mga1.src.rpm
from Core Updates Testing to Core Updates.

Advisory: An unrequired dependency for the kdesvn subversion client
was found, which resulted in the installation of unneeded packages.
This update remove those unneeded dependencies.

https://bugs.mageia.org/show_bug.cgi?id=3399

Keywords: (none) => validated_update
CC: (none) => sysadmin-bugs

Comment 13 Thomas Backlund 2011-11-29 21:44:55 CET

Update pushed.

Status: ASSIGNED => RESOLVED
CC: (none) => tmb
Resolution: (none) => FIXED