Bug 6834 - Reconcile filename/path differences in RPM and deb
: Reconcile filename/path differences in RPM and deb
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Platform Support
: 7.0
: PC Linux (other)
: P5 normal (vote)
: ---
Assigned To: Matt Wise
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-01-28 11:08 UTC by Mark Miksis
Modified: 2009-09-08 09:31 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
OSDetect.pm patch (731 bytes, patch)
2008-01-30 11:39 UTC, Mark Miksis
Details | Diff
Fix broken symlinks (927 bytes, patch)
2008-01-31 14:56 UTC, Mark Miksis
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Miksis 2008-01-28 11:08:39 UTC
Matt may already be working on this, but just to be sure...

There are still a bunch of unresolved filename/path differences between the RPM and deb as described and discussed at http://forums.slimdevices.com/showthread.php?p=253519#post253519.  IMO, it will be much messier to try to address this post-7.0.

CCing Dean to request a 7.0 milestone.
Comment 1 Blackketter Dean 2008-01-28 11:16:55 UTC
Does this prevent us from shipping 7.0 on these platforms?
Comment 2 Mark Miksis 2008-01-28 11:19:39 UTC
(In reply to comment #1)
> Does this prevent us from shipping 7.0 on these platforms?
> 

No, they both work as-is.  It's just messy and will complicate support.  Fixing it after the masses have upgraded to 7.0 will confuse people even more.
Comment 3 Blackketter Dean 2008-01-28 11:21:29 UTC
Matt: what do you think?  
Comment 4 Matt Wise 2008-01-28 13:59:50 UTC
I'll look into making some of the changes tonight and tomorrow... most of them are simple. I agree that making the changes after-release is a little confusing ... but its not the end of the world either way. 
Comment 5 Matt Wise 2008-01-29 07:26:37 UTC
I've made several of the changes that the users have requested... 

-) cachedir is now /var/lib/squeezecenter/cache
-) presdir is used instead of prefsfile, and its /var/lib/squeezecenter/prefs
-) executable name is now squeezecenter-server ... the squeezecenter_safe wrapper is still in place. 
-) plugins directory is now under /var/lib/squeezecenter/Plugins and symlinked to the old /usr location

I'll post a link to an RPM with the changes for testing before I commit them. I dont have an active Ubuntu/Debian box right now to test the RPM on myself! 

http://www.slimdevices.com/downloads/nightly/latest/7.0/squeezecenter_7.0-16880_all.deb
Comment 6 Mark Miksis 2008-01-29 09:16:45 UTC
I don't have a Debian test platform either, but if you post the various files (control, rules, etc), I'll take a look at them.  

The only item not mentioned in your list is the location of the executables.  Based on feedback in the referenced thread, I suggest we change the RPM so they both use /usr/sbin.
Comment 7 Blackketter Dean 2008-01-29 22:21:04 UTC
Can I grab this from the unstable feed for my machine?
Comment 8 Matt Wise 2008-01-30 08:27:32 UTC
Sorry Dean, I didn't put the file into the unstable repo because I was afraid it might really break. I've got my Ubuntu box up and I'm testing it right now. Found one typo, fixed that, and am just walking through things to make sure they work properly. 
Comment 9 Matt Wise 2008-01-30 09:37:31 UTC
Looks like we're all set now... turns out there's a bug in slimserver.pl that will not automatically create a new cachedir anywhere except /var/squeezecenter/cache. So we actually have to make the directory manually in the Debian package install, and then it works just fine. I've checked in my changes and am rebuilding the nightlies now. 
Comment 10 Mark Miksis 2008-01-30 09:41:39 UTC
(In reply to comment #9)
> Looks like we're all set now... turns out there's a bug in slimserver.pl that
> will not automatically create a new cachedir anywhere except
> /var/squeezecenter/cache. 

Could this be because /var/cache/squeezecenter is still hardcoded in OSDetect.pm?  I'm not sure what else that might break.

Also, have you decided to leave the deb binaries in /usr/sbin and the RPM binaries in /usr/libexec?  It's not too big a deal to me, just wanted to make sure it's not an oversight...
Comment 11 Mark Miksis 2008-01-30 11:39:26 UTC
Created attachment 2778 [details]
OSDetect.pm patch

Suggested patch to OSDetect.pm
Comment 12 Mark Miksis 2008-01-31 14:56:47 UTC
Created attachment 2788 [details]
Fix broken symlinks

Untested, but I assume this is the fix to the broken symlinks reported in the Beta list.
Comment 13 Matt Wise 2008-02-01 09:35:00 UTC
I just checked those changes in ... I had fixed them already, but forgot to check them in and they got overwritten last night. Oops! I'll rebuild the Debian packages now...

Comment 14 Blackketter Dean 2008-02-01 17:43:14 UTC
Matt, can this be closed? 
Comment 15 Matt Wise 2008-02-04 10:45:41 UTC
I think this is finally fixed... I had to make some changes to osdetect, and the rules file for everything to play well together. 
Comment 16 Chris Owens 2008-03-07 09:04:44 UTC
This bug is being closed since it was resolved for a version which is now released!  Please download the new version of SqueezeCenter (formerly SlimServer) at http://www.slimdevices.com/su_downloads.html

If you are still seeing this bug, please re-open it and we will consider it for a future release.