[Bug 477542] Review Request: mpdscribble - A mpd client which submits information about tracks being played to Last.fm

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=477542





--- Comment #22 from Peter Lemenkov <lemenkov@xxxxxxxxx>  2009-10-28 05:10:19 EDT ---
One small issue still remains (fortunately, the last) - a path for the pid-file
is set to /var/run/$prog/$prog.pid, but you didn't list /var/run/${name} in the
%files section.

Creating extra directory in /var/run, for storing pid-file, is an obvious
solution for permissions issue, when main application runs from different user.
Since mpdscribble runs from root, it doesn't make any sense.

So, I advice you to change variable $pidfile in the init-script to 
/var/run/$prog.pid

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]