After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 711091 - gss-server tool names clashes with libkrb5
gss-server tool names clashes with libkrb5
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-streaming-server
git master
Other All
: Normal blocker
: 1.0.0
Assigned To: Reynaldo H. Verdejo Pinochet
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-10-29 17:32 UTC by Olivier Crête
Modified: 2014-06-22 16:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
half backed fix (3.11 KB, patch)
2013-11-05 15:58 UTC, Reynaldo H. Verdejo Pinochet
needs-work Details | Review
proposed fix (18.53 KB, patch)
2013-11-15 19:14 UTC, Reynaldo H. Verdejo Pinochet
committed Details | Review

Description Olivier Crête 2013-10-29 17:32:45 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.
Comment 1 Tim-Philipp Müller 2013-10-29 17:44:38 UTC
Should probably be done before the first release, so let's mark it as blocker.
Comment 2 Reynaldo H. Verdejo Pinochet 2013-11-05 15:58:03 UTC
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.
Comment 3 Tim-Philipp Müller 2013-11-05 19:09:41 UTC
-- 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.
Comment 4 Reynaldo H. Verdejo Pinochet 2013-11-05 20:34:35 UTC
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).
Comment 5 Tim-Philipp Müller 2013-11-05 20:51:41 UTC
It's Dave's call.
Comment 6 Reynaldo H. Verdejo Pinochet 2013-11-12 22:09:06 UTC
David? Can you take a look and suggest how to go forward
with this one?
Comment 7 David Schleef 2013-11-13 22:00:48 UTC
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.
Comment 8 Reynaldo H. Verdejo Pinochet 2013-11-15 19:14:43 UTC
Created attachment 259939 [details] [review]
proposed fix

Committing this one then. Change made only for gss-server
for the time being.
Comment 9 David Schleef 2013-11-17 05:17:49 UTC
Review of attachment 259939 [details] [review]:

x