Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 442852 - media-sound/pulseaudio: keeps dir /var/run/pulse
Summary: media-sound/pulseaudio: keeps dir /var/run/pulse
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: keepdir-var-run_lock
  Show dependency tree
 
Reported: 2012-11-12 09:40 UTC by Alexander Tsoy
Modified: 2014-02-16 21:20 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
pulseaudio.init.d-5.patch (pulseaudio.init.d-5.patch,475 bytes, patch)
2012-12-25 18:00 UTC, Alexander Tsoy
Details | Diff
pulseaudio-3.0.ebuild.patch (pulseaudio-3.0.ebuild.patch,317 bytes, patch)
2012-12-25 18:01 UTC, Alexander Tsoy
Details | Diff
pulseaudio-3.0.ebuild.patch (pulseaudio-3.0.ebuild.patch,658 bytes, patch)
2012-12-26 09:57 UTC, Alexander Tsoy
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Tsoy 2012-11-12 09:40:38 UTC
1. pulseaudio ebuild should not do keepdir /var/run/pulse.
2. afaik user 'pulse', and groups 'pulse', 'pulse-access' are only needed if USE=system-wide. So enewuser and enewgroup shouldn't be done unconditionally.

Reproducible: Always
Comment 1 Alexander Tsoy 2012-11-12 11:07:25 UTC
Also home dir for user 'pulse' should be moved from /var/run/pulse
Comment 2 Pacho Ramos gentoo-dev 2012-11-12 20:28:47 UTC
this is not a regression over current stable
Comment 3 Arun Raghavan (RETIRED) gentoo-dev 2012-12-07 20:13:15 UTC
Fixed in 2.99.3.
Comment 4 Alexander Tsoy 2012-12-25 17:57:01 UTC
This bug is not completely fixed.

1. State dir should be created by init.d script. Something like this could be added to it:

checkpath -q -d -m 0755 -o pulse:pulse /var/run/pulse

2. This strings should be removed from ebuild. They are useless without deleted 'keepdir /var/run/pulse'.

        # Create the state directory
        use prefix || diropts -o pulse -g pulse -m0755
Comment 5 Alexander Tsoy 2012-12-25 18:00:10 UTC
Created attachment 333324 [details, diff]
pulseaudio.init.d-5.patch
Comment 6 Alexander Tsoy 2012-12-25 18:01:28 UTC
Created attachment 333328 [details, diff]
pulseaudio-3.0.ebuild.patch
Comment 7 Alexander Tsoy 2012-12-26 09:57:47 UTC
Created attachment 333374 [details, diff]
pulseaudio-3.0.ebuild.patch

Or maybe it would be better to create home dir for pulse user in /var/lib/pulse? Then pulseaudio state files will be persistent across reboots (not sure is this really needed). In this case patch for init.d script is not needed.
Comment 8 Alexander Tsoy 2012-12-30 21:50:11 UTC
(In reply to comment #7)
> In this case patch for init.d script is not needed.

Hmm.. No, this is not true. pid file is created in /var/run/pulse, so patch for init.d script is still needed.
Comment 9 Pacho Ramos gentoo-dev 2014-02-16 21:20:27 UTC
Fixed in 4.99.4