Bug 1630 - Info when installing mpd mentions Mandrake instead of Mageia!
Summary: Info when installing mpd mentions Mandrake instead of Mageia!
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 1
Hardware: i586 Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard:
Keywords: validated_update
: 2558 2559 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-06-06 10:14 CEST by Margot Lawrence
Modified: 2011-11-08 07:57 CET (History)
10 users (show)

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


Attachments
Patch to remove config lines that generate errors. (659 bytes, patch)
2011-08-01 00:32 CEST, Dave Hodgins
Details | Diff
Patch to remove unsupported option from startup script (510 bytes, patch)
2011-08-01 00:40 CEST, Dave Hodgins
Details | Diff

Description Margot Lawrence 2011-06-06 10:14:15 CEST
Description of problem:
When installing mpd with urpmi, this information appears:

[More information on package mpd-0.16.2-2.mga1.tainted.i586
This package of MPD comes with service capabilities and will automatically start
at boot.

You now have to edit your /etc/mpd.conf file, check the different options to
adapt them to your needs (pay attention to the password section). You can
leave log_file, error_file, db_file, and state_file options as they are. But
you MUST edit music_directory and playlist_directory to tell MPD where to grab
your music files, and don't forget to make them available to the user "mpd".
If you use passwords, please change permissions of this file to 600
(passwords are in clear plain text).

If you DON'T want MPD to start at boot (to let users have different
configuration, for example), just disable the service via the Mandrake Control
Center, or by running "chkconfig --del mpd" as root.


Note: if your msec level is 3 or above (this should be the case for any machine
connected to the internet), the service will fail to start. In order to
make it start at boot, you need to do this :

* get root privileges edit /etc/security/console.perms
near the end of the file you'll see a line looking like this :
<console>  0600 <sound>      0600 root.audio

just change the permissions to 0660 to obtain this line : 
<console>  0660 <sound>      0660 root.audio

Save the file and exit your editor.

MPD has now the right to access sound devices, as all users from the audio
group.]


Perhaps this was missed during the cleanup because it mentions 'Mandrake' rather than 'Mandriva'?


Steps to Reproduce:
1. Use urpmi to install mpd 
2. Read the info message
3.
Manuel Hiebel 2011-07-10 16:28:20 CEST

CC: (none) => shikamaru
Source RPM: mpd-0.16.2-2.mga1.tainted.src.rpm => mpd

Comment 1 Sandro CAZZANIGA 2011-07-14 18:16:19 CEST
thanks for the report, it's fixed in core/updates_testing for mageia 1 and core/release for cauldron.

Status: NEW => RESOLVED
CC: (none) => cazzaniga.sandro
Resolution: (none) => FIXED

Comment 2 Manuel Hiebel 2011-07-14 18:44:06 CEST
assigned to qa, any update announcement ?

Status: RESOLVED => REOPENED
Resolution: FIXED => (none)
Assignee: bugsquad => qa-bugs

Comment 3 Manuel Hiebel 2011-07-14 18:51:52 CEST
sorry wrong status, and  it's the package mpd-0.16.2-3.mga1

Status: REOPENED => ASSIGNED

Comment 4 Dave Hodgins 2011-07-14 21:30:40 CEST
What about the tainted version?

CC: (none) => davidwhodgins

Comment 5 Samuel Verschelde 2011-07-21 13:49:50 CEST
ping Kharec :
* we still need an advisory text
* the tainted version is missing

CC: (none) => stormi

Comment 6 Margot Lawrence 2011-07-30 10:26:05 CEST
PING?

As the bug was originally reported on the mpd-0.16.2-2.mga1.tainted.i586 package, I am looking for the update in the Mageia/distrib/1/i586/media/tainted/updates_testing/ repo.

Am I looking in the wrong place, or did someone forget to produce an update for the package against which the bug was actually reported?
Comment 7 Dave Hodgins 2011-07-30 18:56:52 CEST
As per comment 4 from 2 weeks ago, this rpm cannot go to updates without a
matching release number tainted version.
Comment 8 Samuel Verschelde 2011-07-30 18:58:37 CEST
Kharec should have done it but for a reason unknown to me he still hasn't done it (and now has left for holidays). I will take care of it.
Comment 9 Samuel Verschelde 2011-07-30 19:41:32 CEST
Kharec, although it was written in the updates policy, whose link we gave to you on IRC, you have bumped the release and not added any subrel for this update :/

I pushed the package to tainted/updates_testing without touching the release so that it keeps the same as the one in core/updates_testing, but future updates of this package should use subrels.

To QA Team and to the reporter : please test the packages in i586, core and tainted, making sure the bug is fixed and the software still works.
Comment 10 Margot Lawrence 2011-07-30 23:23:26 CEST
Tested package in tainted/updates_testing - bug is fixed, thank you.
Comment 11 Samuel Verschelde 2011-07-30 23:38:39 CEST
Thanks :)

Now we need tests on x86_64 for the package in core and that in tainted
Comment 12 Dave Hodgins 2011-08-01 00:32:05 CEST
Created attachment 698 [details]
Patch to remove config lines that generate errors.

Each of the three lines deleted by the patch stop mpd from
starting, until they are removed.
Comment 13 Dave Hodgins 2011-08-01 00:40:12 CEST
Created attachment 699 [details]
Patch to remove unsupported option from startup script

In addition to this patch, the line
service mpd createdb
should be removed from the postinstall scriptlet.

The description shown to the user during installation should
also warn them, that on the first startup, mpd will display
the message
"Starting MPD: Failed to load database: Database corrupted"
which should be ignored.

After adding the line
<console>  0660 <sound>      0660 root.audio
to /etc/security/console.perms, I had to reboot before mpd
could access the sound device.

I used gmpc to test mpd.
Comment 14 Derek Jennings 2011-08-02 15:24:38 CEST
Worked OK on x86_64 after I had applied the patch to mpd.conf described in Comment 13.

I did not have to do anything to console.perms.  mpd was able to use the sound device without any modification. User mpd was a member of the audio group.

CC: (none) => derekjenn

Comment 15 Dave Hodgins 2011-08-28 22:04:33 CEST
*** Bug 2558 has been marked as a duplicate of this bug. ***

CC: (none) => jani.valimaa

Comment 16 Dave Hodgins 2011-08-28 22:05:31 CEST
*** Bug 2559 has been marked as a duplicate of this bug. ***
Comment 17 Samuel Verschelde 2011-09-01 23:40:02 CEST
Adding also jq and cjw in CC, as they worked on this package.

To packagers : could you have a look at the patches in comment #13 and comment #14 (and to the bugs mentioned in comment #15 and comment #16) ? mpd has several issues currently and it would be nice to fix them before we push an update for it to mageia 1.

CC: (none) => cjw, jquelin

Comment 18 Samuel Verschelde 2011-09-10 01:24:57 CEST
Assigning back to Kharec as several issues block the update.

Status: ASSIGNED => NEW
CC: (none) => qa-bugs
Assignee: qa-bugs => cazzaniga.sandro

Comment 19 Jerome Quelin 2011-09-17 18:14:40 CEST
i confirm that new versions of mpd do not support anymore neither the --create-db flag, nor the following config options:
 - error_file
 - mixer_device
 - mixer_control

i did not spot this even if i'm using mpd, since i have a personal (non-system wide) mpd configuration. and i already removed those options quite some time ago...

dave's patches had almost everything correct - except that it missed the call create-db in %post in the spec, and the mention of error_file in README + logrorate config. however, i kind of disagree that they are duplicates of 2558 and 2559: there was indeed 3 bugs, and it's better to have one bug per problem to track what's being done.

anyway, mpd-0.16.3-3.mga2 is on its way to bs (cauldron currently) with the fix. 
it will have to be sent to updates afterwards...
Comment 20 Manuel Hiebel 2011-09-27 17:53:52 CEST
assign back to bugsquad (kharac has no more time for packaging)

CC: cazzaniga.sandro => (none)
Assignee: cazzaniga.sandro => bugsquad

Comment 21 Samuel Verschelde 2011-10-01 03:53:31 CEST
Assigning to new maintainer.

Assignee: bugsquad => jquelin

Comment 22 claire robinson 2011-10-15 12:03:34 CEST
Removing from QA list for the time being as there is nothing yet to test.

Please reassign to QA when ready :)

Thankyou

CC: qa-bugs => (none)

Comment 23 Jerome Quelin 2011-10-15 13:43:29 CEST
mpd-0.16.2-3.1.mga1 is available for mga1 in core/updates_testing with the changes mentioned in comment 19
==> qa, please review & push to updates

QA Contact: (none) => qa-bugs

Jerome Quelin 2011-10-15 13:43:49 CEST

Assignee: jquelin => qa-bugs

Comment 24 Derek Jennings 2011-10-15 16:40:35 CEST
mpd-0.16.2-3.1.mga1 validated on x86_64
Tested with gmpc client. All functions worked OK.

The only deviation I had to make from the standard configuration was to uncomment the line  
device                  "hw:0,0"
in the ALSA output.
Comment 25 Dave Hodgins 2011-10-17 21:59:44 CEST
Testing complete on i586 for the srpm
mpd-0.16.2-3.1.mga1.src.rpm.

I had to run paprefs, and on the network server tab, enable
"network access to local sound devices", to get the pulse
config working.

We still need the
mpd-0.16.2-3.1.mga1.tainted.src.rpm

Reassigning back to the developer until the tainted version is
available for testing.

CC: (none) => qa-bugs
Assignee: qa-bugs => jquelin

Comment 26 Jerome Quelin 2011-10-20 09:18:04 CEST
mpd-0.16.2-3.1.mga1 available in mga1 tainted/updates_testing

Assignee: jquelin => qa-bugs

Comment 27 Dave Hodgins 2011-10-22 05:17:49 CEST
The i586 testing of mpd, using pulseaudio is complete for the srpm
mpd-0.16.2-3.1.mga1.tainted.src.rpm

For some strange reason, in addition to enabling
network access in paprefs, I also had to set the
don't require authentication, to get it to work
this time.  Again used gmpc for the testing.

Note that paprefs is run as a regular usr.
Comment 28 Dave Hodgins 2011-10-31 03:12:09 CET
We still need an x86-64 test for mpd from tainted.
Comment 29 Derek Jennings 2011-11-01 15:08:23 CET
Tested mpd-0.16.2-3.1.mga1.tainted on x86_64

All works as expected apart from being unable to play or even add .m4a files to the database. lib64faad is installed and the same files play OK in vlc and clementine
Comment 30 claire robinson 2011-11-07 15:07:49 CET
Assigning maintainer, see comment 29 for why. Please reassign QA when you've looked into it. Thankyou.

Assignee: qa-bugs => jquelin
QA Contact: qa-bugs => (none)

Comment 31 Jerome Quelin 2011-11-07 16:14:10 CET
derek, is it working with mpd as shipped by mageia 1?
if not (and i suspect it isn't), then it's another bug, and please don't hold back the push of this update.
Comment 32 Derek Jennings 2011-11-08 01:05:34 CET
Confirmed that mpd-0.16.2-2.mga1.tainted does not play .m4a files either.
I am therefore validating this update and am raising a new bug to cover mpd.tainted not playing .m4a files.

Could someone from sysadmin please push mpd-0.16.2-3.1.mga1.src.rpm and mpd-0.16.2-3.1.mga1.tainted.src.rpm into core_updates and core_tainted_updates.

Advisory
--------
This update replaces Mandriva references with Mageia in information files as well as updating default mpd configuration.

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

Comment 33 Thomas Backlund 2011-11-08 07:57:56 CET
Update pushed.

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


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