Bug 16678 - cantata erroneously require mpd
Summary: cantata erroneously require mpd
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 5
Hardware: All Linux
Priority: Normal major
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: MGA5-64-OK MGA5-32-OK advisory
Keywords: validated_update
Depends on:
Blocks:
 
Reported: 2015-09-01 05:22 CEST by Johnny A. Solbu
Modified: 2016-07-05 17:47 CEST (History)
5 users (show)

See Also:
Source RPM: cantata
CVE:
Status comment:


Attachments

Description Johnny A. Solbu 2015-09-01 05:22:47 CEST
cantata is listed with mpd as a Requires, which is incorrect.
It is currently not possible to install cantata in order to controll a Remote mpd instance without also installing mpd on the local system.
It does _Not_ require mpd.

The mpd Requirement should be removed!

Reproducible: 

Steps to Reproduce:
Comment 1 Johnny A. Solbu 2015-09-01 05:26:57 CEST
Assigning to maintainer.

Version: 5 => Cauldron
Assignee: bugsquad => juan.baptiste
Whiteboard: (none) => MGA5TOO

Comment 2 Johnny A. Solbu 2016-06-22 02:25:03 CEST
Reassigning.

Assignee: juan.baptiste => geiger.david68210

Comment 3 David GEIGER 2016-06-22 08:09:26 CEST
Fixed now!

mpd is now removed from the required list.

Does this fix also be done for mga5?
Comment 4 Johnny A. Solbu 2016-06-22 09:18:35 CEST
(In reply to David GEIGER from comment #3)
> mpd is now removed from the required list.
> 
> Does this fix also be done for mga5?

Yes.
Comment 5 David GEIGER 2016-06-22 09:32:26 CEST
Assigning to QA now,


Advisory:
========================

cantata is listed with mpd as a Requires, which is incorrect.
It is currently not possible to install cantata in order to controll a Remote mpd instance without also installing mpd on the local system.
So this update removes mpd from the required list.

========================

Packages in 5/core/updates_testing:
========================
cantata-1.4.1-7.1.mga5.x86_64.rpm
cantata-1.4.1-7.1.mga5.i586.rpm

Source RPM: 
========================
cantata-1.4.1-7.mga5.src.rpm


How to test this update request:
========================
- Just ensure that mpd is no more pulled with cantata installation.

Assignee: geiger.david68210 => qa-bugs

Comment 6 David GEIGER 2016-06-22 09:33:06 CEST
Arr error! should be:

Source RPM: 
========================
cantata-1.4.1-7.1.mga5.src.rpm

CC: (none) => geiger.david68210

Comment 7 Otto Leipälä 2016-06-22 09:34:57 CEST
This would break cantata installs totally in non remote systems please change it to recommends. If user installs now cantata it can't play anything if you don't manually install mpd.

CC: (none) => ozkyster

Comment 8 David GEIGER 2016-06-22 09:43:42 CEST
Any objection Johnny A. Solbu?

Assignee: qa-bugs => geiger.david68210

Comment 9 Otto Leipälä 2016-06-22 11:19:59 CEST
That would allow to use urpmi --no-recommends cantata to skip mpd install,but same time make sure that local/no remote systems still get mpd to enable playback so everybody is happy.
Comment 10 Johnny A. Solbu 2016-06-23 19:07:06 CEST
(In reply to David GEIGER from comment #8)
> Any objection Johnny A. Solbu?

No, :-)
Having a Recommends is fine.
Comment 11 David GEIGER 2016-06-23 19:18:57 CEST
ok thanks! let's go...
Comment 12 David GEIGER 2016-06-23 20:22:13 CEST
Re-Assigning to QA,


Advisory:
========================

cantata is listed with mpd as a Requires, which is incorrect.
It is currently not possible to install cantata in order to controll a Remote mpd instance without also installing mpd on the local system.
So this update removes mpd from the required list and replaces it as a Recommends instead.

========================

Packages in 5/core/updates_testing:
========================
cantata-1.4.1-7.2.mga5.x86_64.rpm
cantata-1.4.1-7.2.mga5.i586.rpm

Source RPM: 
========================
cantata-1.4.1-7.2.mga5.src.rpm


How to test this update request:
========================
- Just ensure that mpd is no more pulled with cantata installation using 
"urpmi --no-recommends cantata".

Assignee: geiger.david68210 => qa-bugs

Comment 13 claire robinson 2016-06-23 22:17:22 CEST
Has this been pushed in cauldron David?
Comment 14 David GEIGER 2016-06-23 22:19:34 CEST
Yes of course Claire :)
Comment 15 claire robinson 2016-06-23 23:07:03 CEST
Thanks :)

Version: Cauldron => 5
Whiteboard: MGA5TOO => (none)

Comment 16 James Kerr 2016-06-30 18:36:09 CEST
Testing on mga5-64

# urpmi --test cantata
To satisfy dependencies, the following packages are going to be installed:
(test only, installation will not be actually done)
  Package                        Version      Release       Arch    
(medium "Core Release (LAN1)")
  lib64fluidsynth1               1.1.6        5.mga5        x86_64  (recommended)
(medium "Core Updates Testing (LAN5)")
  cantata                        1.4.1        7.2.mga5      x86_64  
(medium "Tainted Release (LAN21)")
  lib64sidplay-builders0         2.1.1        20.mga5.tain> x86_64  (recommended)
  lib64sidplay2_1                2.1.1        20.mga5.tain> x86_64  (recommended)
  lib64sidutils0                 2.1.1        20.mga5.tain> x86_64  (recommended)
  mpd                            0.18.8       3.mga5.taint> x86_64  (recommended)

# urpmi --no-recommends cantata
  ftp://192.168.0.2/pub/mirror/Mageia/distrib/5/x86_64/media/core/updates_testing/cantata-1.4.1-7.2.mga5.x86_64.rpm
installing cantata-1.4.1-7.2.mga5.x86_64.rpm from /var/cache/urpmi/rpms

Which is the required behaviour. OK for mga5-64

CC: (none) => jim
Whiteboard: (none) => MGA4-64-OK

James Kerr 2016-06-30 18:37:48 CEST

Whiteboard: MGA4-64-OK => MGA5-64-OK

Comment 17 James Kerr 2016-06-30 19:08:23 CEST
I have no idea how cantata is supposed to be used, but is changing all of the dependencies to suggests correct? I've removed the OK for mga5-64. This will need to be tested by someone who understands what this application is supposed to do and how to use it.

Whiteboard: MGA5-64-OK => (none)

Comment 18 James Kerr 2016-06-30 19:23:51 CEST
Please ignore comment #17

I've now understood (at the umpteenth time of reading) how mpd relates to cantata. 

I've restored the OK for mga5-64

Whiteboard: (none) => MGA5-64-OK

Comment 19 Johnny A. Solbu 2016-06-30 19:28:55 CEST
(In reply to James Kerr from comment #17)
> I have no idea how cantata is supposed to be used, but is changing all of
> the dependencies to suggests correct? I've removed the OK for mga5-64. This
> will need to be tested by someone who understands what this application is
> supposed to do and how to use it.

The alternative is to Remove the Requires/Recommends entirely.

Cantata is a tool to remotely access an mpd server, for playing music on said server. It does the same as the old Sonata program does, for those who remember that one. The only noticeable difference is that Cantata on initial launch will complain that it can't access the local mpd server, as the local mpd server is the default.

Cantata is an mpd client, just as mpc is. But mpc doesn't require mpd installed, even if it's usewless without an mpd server. But the server doesn't need to be local.

True, most users probably use it to control the mpd server on the same local machine. but it is not required in any way to have mpd installed on the local machine in order to use it. I use it to control an mpd server on an old MDV-2010.2 system, which is connected to my stereo system. So it is really anoying to have mpd installed as a requirement on the local system, in order to controll the remote mpd server.

An analogy would be to require apache in order to install Firefox. Firefox is for most users useless without a webservert to connect to. but you don't need to have a webserver installed localy in order to install a webbrowser.
Comment 20 James Kerr 2016-06-30 19:59:26 CEST
@Johnny-
Thanks for the excellent explanation. It confirms and enhances the understanding that I had struggled to reach.
Comment 21 James Kerr 2016-06-30 20:59:46 CEST
Testing on mga5-32

# urpmi --test cantata
To satisfy dependencies, the following packages are going to be installed:
(test only, installation will not be actually done)
  Package                        Version      Release       Arch    
(medium "Core Release (LAN1)")
  libfluidsynth1                 1.1.6        5.mga5        i586    (recommended)
(medium "Core Updates Testing (LAN5)")
  cantata                        1.4.1        7.2.mga5      i586    
(medium "Tainted Release (LAN21)")
  libsidplay-builders0           2.1.1        20.mga5.tain> i586    (recommended)
  libsidplay2_1                  2.1.1        20.mga5.tain> i586    (recommended)
  libsidutils0                   2.1.1        20.mga5.tain> i586    (recommended)
  mpd                            0.18.8       3.mga5.taint> i586    (recommended)


# urpmi --no-recommends cantata
   ftp://192.168.0.2/pub/mirror/Mageia/distrib/5/i586/media/core/updates_testing/cantata-1.4.1-7.2.mga5.i586.rpm
installing cantata-1.4.1-7.2.mga5.i586.rpm from /var/cache/urpmi/rpms 

Which is the desired behaviour. OK for mga5-32

Whiteboard: MGA5-64-OK => MGA5-64-OK MGA5-32-OK

Comment 22 James Kerr 2016-06-30 21:03:21 CEST
This update is now validated. The Advisory needs to be uploaded to SVN. The packages can then be pushed to updates.

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

Dave Hodgins 2016-07-04 09:14:53 CEST

CC: (none) => davidwhodgins
Whiteboard: MGA5-64-OK MGA5-32-OK => MGA5-64-OK MGA5-32-OK advisory

Comment 23 Mageia Robot 2016-07-05 17:47:53 CEST
An update for this issue has been pushed to the Mageia Updates repository.

http://advisories.mageia.org/MGAA-2016-0096.html

Status: NEW => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.