GNOME Bugzilla – Bug 711091
gss-server tool names clashes with libkrb5
Last modified: 2014-06-22 16:03:04 UTC
On my system, there is a /usr/sbin/gss-server which is owned by the krb5-devel-1.11.3-2.fc19.x86_64 package. We should rename gst-streaming-server's command line utility to a name that doesn't clash or distros won't be happy.
Should probably be done before the first release, so let's mark it as blocker.
Created attachment 259022 [details] [review] half backed fix Half backed and untested fix. Changed also gss-transcoder to gst-gss-transcoder for congruency. Not sure about the name, open to suggestions here. I think source filenames might need to be changed too? not that it matters that much but for consistency I guess.
-- begin bikeshed -- Of course the 'g' in gss already stands for gst/gstreamer, and then other binaries in non-main modules do carry their own prefix as well, e.g. ges-launch ;) -- end bikeshed -- One could simply add the API version to the toolname, like we do with gstreamer binaries now.
Thought the idea of the version suffix was just to get over having more that one gst version cohabiting in the same system. I'm OK with your idea but I wonder if we aren't making a rule out of an ad-hoc exception? I think a more correct approach would be to work towards having every gst program sharing the same prefix and start by fixing this particular issue following that approach but if you decide otherwise I will just follow suit. Will wait for a confirmation on this and propose a new fix then (will probably go for gst-ss-/gst-sserver or smth).
It's Dave's call.
David? Can you take a look and suggest how to go forward with this one?
Please don't use "gst-gss-" as the prefix. I think the best direction is to switch to a gst- prefix, since that's what everything else uses. My first thought is to use 'gst-streaming-server' as the binary name.
Created attachment 259939 [details] [review] proposed fix Committing this one then. Change made only for gss-server for the time being.
Review of attachment 259939 [details] [review]: x