RFC: Installing vdr to /usr(/local)/sbin

Message ID 1137703349.886.19.camel@bobcat.mine.nu
State New
Headers

Commit Message

Ville Skyttä Jan. 19, 2006, 8:42 p.m. UTC
As mentioned in a private mail to Klaus:

I tend to think /usr(/local)/sbin and manual page section 8 would be
more appropriate for vdr than /usr(/local)/bin and section 1.

Even though "sbin" and section 8 is generally for "system administration
commands", which vdr clearly is not, in my opinion vdr is even less a
"user command" which "bin" and section 1 more or less implies.  No need
to clutter normal users' PATH with executables they aren't really
supposed to run.

Additionally, the vast majority of daemon-like programs are installed to
sbin directories by all distributions I'm aware of.

So, what do people think, would something like the attached patch be a
good idea?  It might break some setups, so it clearly needs to be done
during a development cycle if ever.  While sort of cosmetic, I think
this would be a good thing, and in the series of making vdr feel "more
mature" like someone mentioned in a recent "don't run as root" thread.

Obviously, after applying the patch, vdr.1 should be renamed to vdr.8.
  

Comments

Udo Richter Jan. 20, 2006, 12:59 a.m. UTC | #1
Ville Skyttä wrote:
> Even though "sbin" and section 8 is generally for "system administration
> commands", which vdr clearly is not, in my opinion vdr is even less a
> "user command" which "bin" and section 1 more or less implies.  No need
> to clutter normal users' PATH with executables they aren't really
> supposed to run.

Well, that depends. The 'typical' use of VDR as recorder daemon, 
starting at boot time, would suggest this. However, I also have a 
secondary VDR on my notebook, relying on streamdev and xine-plugin, and 
in that case its really more an user process that is only started when 
needed.

On the other hand, I don't use the default make install anyway. ;)

Cheers,

Udo
  
Patrick Cernko Jan. 20, 2006, 5:42 a.m. UTC | #2
Ville Skyttä schrieb:
> As mentioned in a private mail to Klaus:
> 
> I tend to think /usr(/local)/sbin and manual page section 8 would be
> more appropriate for vdr than /usr(/local)/bin and section 1.
> 
> So, what do people think, would something like the attached patch be a
> good idea?  It might break some setups, so it clearly needs to be done
> during a development cycle if ever.  While sort of cosmetic, I think
> this would be a good thing, and in the series of making vdr feel "more
> mature" like someone mentioned in a recent "don't run as root" thread.
> 

I support Udo's opinion too. We have a vdr running at work. It starts up
as a daemon on boot. So this would match your argumentation. But we
found out, that for cutting, it is easier to login to the vdr server and
start a special vdr with softdevice(xv) output and (optional)
streamdev-client input. The output is forwarded sufficiently fast via
ssh-XForwarding in a Gigabit network. With this application in mind, I
vote for not moving vdr to /usr/sbin.

Anyway, I also have my own installation method, and can handle the move
on my side, too.

So long,

Patrick Cernko
  

Patch

--- vdr-1.3.39/INSTALL~	2006-01-14 13:27:45.000000000 +0200
+++ vdr-1.3.39/INSTALL	2006-01-19 22:32:19.000000000 +0200
@@ -106,7 +106,7 @@ 
 When starting the program through an entry in /etc/inittab, use the --terminal
 option to set the controlling terminal, as in
 
-vdr:123:respawn:/usr/local/bin/vdr --terminal=/dev/tty8 -w 60
+vdr:123:respawn:/usr/local/sbin/vdr --terminal=/dev/tty8 -w 60
 
 See the man page vdr(1) for complete information about all command line options.
 
--- vdr-1.3.39/vdr.5~	2006-01-14 12:57:37.000000000 +0200
+++ vdr-1.3.39/vdr.5	2006-01-19 22:36:30.000000000 +0200
@@ -649,7 +649,7 @@ 
 This file will be read at program startup in order to restore the results of
 previous EPG scans.
 .SH SEE ALSO
-.BR vdr (1)
+.BR vdr (8)
 .SH AUTHOR
 Written by Klaus Schmidinger.
 .SH REPORTING BUGS
--- vdr-1.3.39/vdr.1~	2006-01-14 13:09:08.000000000 +0200
+++ vdr-1.3.39/vdr.1	2006-01-19 22:37:38.000000000 +0200
@@ -10,7 +10,7 @@ 
 .\"
 .\" $Id: vdr.1 1.22 2006/01/14 11:09:08 kls Exp $
 .\"
-.TH vdr 1 "08 Jan 2006" "1.3.38" "Video Disk Recorder"
+.TH vdr 8 "08 Jan 2006" "1.3.38" "Video Disk Recorder"
 .SH NAME
 vdr - the Video Disk Recorder
 .SH SYNOPSIS
--- vdr-1.3.39/Make.config.template~	2006-01-13 18:06:11.000000000 +0200
+++ vdr-1.3.39/Make.config.template	2006-01-19 22:31:55.000000000 +0200
@@ -20,7 +20,7 @@ 
 
 #DVBDIR   = ../DVB
 MANDIR   = /usr/local/man
-BINDIR   = /usr/local/bin
+BINDIR   = /usr/local/sbin
 
 PLUGINDIR= ./PLUGINS
 PLUGINLIBDIR= $(PLUGINDIR)/lib
--- vdr-1.3.39/Makefile~	2006-01-13 18:04:56.000000000 +0200
+++ vdr-1.3.39/Makefile	2006-01-19 22:40:15.000000000 +0200
@@ -26,7 +26,7 @@ 
 
 LSIDIR   = ./libsi
 MANDIR   = /usr/local/man
-BINDIR   = /usr/local/bin
+BINDIR   = /usr/local/sbin
 LIBS     = -ljpeg -lpthread -ldl -lcap
 INCLUDES =
 
@@ -202,9 +202,9 @@ 
 install:
 	@mkdir -p $(BINDIR)
 	@cp vdr runvdr $(BINDIR)
-	@mkdir -p $(MANDIR)/man1
+	@mkdir -p $(MANDIR)/man8
 	@mkdir -p $(MANDIR)/man5
-	@gzip -c vdr.1 > $(MANDIR)/man1/vdr.1.gz
+	@gzip -c vdr.8 > $(MANDIR)/man8/vdr.8.gz
 	@gzip -c vdr.5 > $(MANDIR)/man5/vdr.5.gz
 	@if [ ! -d $(VIDEODIR) ]; then\
             mkdir -p $(VIDEODIR);\