Bug 5623 - Incorrect SLIMSERVER_HOME in rpm package
: Incorrect SLIMSERVER_HOME in rpm package
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: RPM
: 7.0
: PC Fedora
: P2 normal (vote)
: ---
Assigned To: Mark Miksis
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-09-30 06:39 UTC by Peter Oliver
Modified: 2009-01-29 09:47 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Oliver 2007-09-30 06:39:42 UTC
SlimServer_trunk_v2007-09-30.noarch.rpm delivers the following /etc/sysconfig/slimserver:

# Edit this to suit your setup
SLIMSERVER_USER="slimserver"
SLIMSERVER_HOME="PREFIX"
SLIMSERVER_CFG="/etc/slimserver.conf"
SLIMSERVER_LOG="/var/log/slimserver"
SLIMSERVER_ARGS="--daemon --prefsfile=$SLIMSERVER_CFG --logdir=$SLIMSERVER_LOG"

Here, the string PREFIX should have been replaced with /usr/local/slimserver.  This results in SlimCenter failing to start.
Comment 1 KDF 2007-09-30 13:17:32 UTC
please consider using the yum repo for Fedora.  
Comment 2 Peter Oliver 2007-09-30 14:16:17 UTC
Is there a repository other that the one at http://repos.slimdevices.com/yum/slimserver-testing/?  Unless I'm missing something, this only contains slimserver 6.5.0, but the bug is new in version 7.
Comment 3 KDF 2007-09-30 14:33:36 UTC
good point.  unfortunately, the bog standard noarch rpm really isn't designed for fedora.  It uses a redhat build script, but fedora began requiring different elements.  Hence, the yum version of the rpm.  Both the PREFIX and /var/log/slimserver issues are being dealt with in the build scripts, but something in fedora makes it work differently.

I'll see if I can flag someone at SD about setting up a yum build of SC7.
Comment 4 Chris Owens 2007-10-02 12:33:16 UTC
cc'ing Andy to see how difficult it would be to have a regularly-generated yum package.
Comment 5 Mark Miksis 2007-10-31 11:59:59 UTC
I just found this bug today...

(In reply to comment #3)
> It uses a redhat build script, but fedora began requiring
> different elements.  Hence, the yum version of the rpm.  Both the PREFIX and
> /var/log/slimserver issues are being dealt with in the build scripts, but
> something in fedora makes it work differently.

I'm not sure what you mean by this.  The original objective of the Fedora RPM
was to remove the CPAN directory, get
the perl modules from other repos using "Requires:" and provide a spec file back to SD
that could be used to host a slimserver yum repo.

Sadly, we never finished because:
1) We both ran out of time to contribute.
2) Using external repos for perl modules became a big mess because the
required modules weren't available from a single place (and many
weren't available anywhere).  SD has also given up
on this for Debian.

The naming of the svn directory as "platforms/fedora" is perhaps unfortunate.  Our work
had nothing to do with Fedora vs. Red Hat differences.  The RPM should work
on both.

Please don't use the stuff in http://repos.slimdevices.com/yum/slimserver-testing/.  The 
only reason I left it there was in case anyone wanted to continue what we started.

In any case, I'm not sure I understand this bug.  The mechanism of using "PREFIX" and patching
it during %post seems like an odd way to do things, but afaict it works.  My guess is that the
%post script failed for some reason.  Were there any errors during RPM install?  Maybe you don't have sed or grep for some reason?

For information on the new RPM and yum see:
http://forums.slimdevices.com/showthread.php?t=39595 and
http://forums.slimdevices.com/showthread.php?t=39789
 


Comment 6 Chris Owens 2007-11-01 10:48:15 UTC
Fletch can we assign this to you so it doesn't keep showing up in our triage meetings? :)  You can always assign it back to me or on to someone else if you want/need.
Comment 7 Mark Miksis 2007-11-01 10:50:34 UTC
Sure.  In fact, I expect the new RPM to fix the following.  Feel free to assign them to me.
1906
3865
4118
4718
5389
5620
5623
5624
5736
Comment 8 Mark Miksis 2007-12-26 10:58:16 UTC
I was never able to reproduce this bug as described, but I'm confident that it doesn't exist in the new SqueezeCenter RPM.  Please see http://forums.slimdevices.com/showthread.php?t=41217 and http://wiki.slimdevices.com/index.cgi?SqueezeCenterRPM for more information.  Marking as FIXED.