Mailing List archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[linux-dvb] Re: linuxtv-dvb-apps-1.1.0 install patches



Ludwig Nussel wrote:
Johannes Stezenbach wrote:

Guido Draheim wrote:

Furthermore, I did add install-etc extra targets that do not only
handle bindir files - there are some files like *.rc5 and the
series of channels.conf-* that should be present in the non-source
installation to be usable by users. At the moment I did just push
them to $(sysconfdir)/dvb/* which might not be the best place but
at the same time it makes it easy to tell users where to look for
them in their rpm installation.
IMHO that's the wrong place. These files should either go
to /usr/share/dvb/ or /usr/lib/dvb/, or maybe even
/usr/share/docs/dvb/exaples/.

I will apply your patch anyway (because I don't care that
much about LSB compliance), but a follow-up patch that
corrects the locations will gladly be accepted.

JFYI, the SUSE rpm puts the remote config files in
/usr/share/av7110_loadkeys, include files in /usr/include/dvb and
the test programs in /usr/lib/dvb/test. The channels.conf* and scan
config files are just in /usr/share/doc/packages/dvb.

Hmm, that is about reasonable - here is a patch to test/Makefile
that makes the test/ binaries to install in $(libdir)/dvb/test/

cheers,
-- guido                                  http://google.de/search?q=guidod
GCS/E/S/P C++/++++$ ULHS L++w- N++@ s+:a d(+-) r+@>+++ y++ 5++X- (geekcode)
Index: test/Makefile
===================================================================
RCS file: /cvs/linuxtv/dvb-apps/test/Makefile,v
retrieving revision 1.3
diff -u -r1.3 Makefile
--- test/Makefile	14 Feb 2004 22:04:51 -0000	1.3
+++ test/Makefile	14 Feb 2004 23:36:24 -0000
@@ -37,4 +37,7 @@
 	rm -f $(TARGETS) *.o
 
 install:
+	test "$(libdir)/" != "/"
+	mkdir -p $(DESTDIR)$(libdir)/dvb/test
+	cp $(TARGETS) $(DESTDIR)$(libdir)/dvb/test/
 

Home | Main Index | Thread Index