Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
On 18.02.2013 21:51, Dominic Evans wrote: On 16 February 2013 10:48, Klaus Schmidinger klaus.schmidin...@tvdr.de wrote: On 16.02.2013 11:43, Joachim Wilke wrote: 2013/2/15 Klaus Schmidinger klaus.schmidin...@tvdr.de: I'm not planning to make any more changes to the Makefiles for version 2.0.0. So it will remain broken. Thats sad, I provided the patch already before release of 1.7.37. Well, according to Christopher Reimer it isn't broken: http://www.linuxtv.org/pipermail/vdr/2013-January/027173.html Well. If you wanted to install different VDR versions to /opt/vdr-$APIVERSION, you might have compiled with PREFIX=/opt/vdr-$APIVERSION, and the header files would end up at /opt/vdr-$APIVERSION/include which obviously isn't on the default search path. Unless I'm misreading Christopher's e-mail just seemed to say that this patch didn't make a difference to the problem he described - that doesn't mean its not potentially useful. If you do this, you will obviously have built VDR with LCLBLD=1 (and a copy of Make.config.template), in which case /opt/vdr-$APIVERSION/include will be in the vdr.pc file as part of the CXXFLAGS. Since you will need to build your plugins with make VDRDIR=/opt/vdr-$APIVERSION this will read /opt/vdr-$APIVERSION/vdr.pc and everything should work just fine. Even if you don't add INCDIR directly to the CFLAGS/CXXFLAGS you could add it to the pkg-config variables dumped into vdr.pc so that plugin authors can use it if they need it. Currently its not available in either place. This has also been suggested here: http://www.vdr-portal.de/board1-news/board2-vdr-news/p1127697-announce-vdr-developer-version-1-7-38/#post1127697 so if there are two more people who ACK this, I'll add it for the next version. Klaus ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
On 19.02.2013 09:07, Klaus Schmidinger wrote: On 18.02.2013 21:51, Dominic Evans wrote: ... Even if you don't add INCDIR directly to the CFLAGS/CXXFLAGS you could add it to the pkg-config variables dumped into vdr.pc so that plugin authors can use it if they need it. Currently its not available in either place. This has also been suggested here: http://www.vdr-portal.de/board1-news/board2-vdr-news/p1127697-announce-vdr-developer-version-1-7-38/#post1127697 so if there are two more people who ACK this, I'll add it for the next version. I'm afraid I have to cancel this. There was a valid point at http://www.vdr-portal.de/board1-news/board2-vdr-news/p1127905-announce-vdr-developer-version-1-7-38/#post1127905 which stated that if plugins provide header files for other plugins, we might end up having to compile/install the plugins in a very specific order, which is unacceptable. Klaus ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
On 16 February 2013 10:48, Klaus Schmidinger klaus.schmidin...@tvdr.de wrote: On 16.02.2013 11:43, Joachim Wilke wrote: 2013/2/15 Klaus Schmidinger klaus.schmidin...@tvdr.de: I'm not planning to make any more changes to the Makefiles for version 2.0.0. So it will remain broken. Thats sad, I provided the patch already before release of 1.7.37. Well, according to Christopher Reimer it isn't broken: http://www.linuxtv.org/pipermail/vdr/2013-January/027173.html Well. If you wanted to install different VDR versions to /opt/vdr-$APIVERSION, you might have compiled with PREFIX=/opt/vdr-$APIVERSION, and the header files would end up at /opt/vdr-$APIVERSION/include which obviously isn't on the default search path. Unless I'm misreading Christopher's e-mail just seemed to say that this patch didn't make a difference to the problem he described - that doesn't mean its not potentially useful. Even if you don't add INCDIR directly to the CFLAGS/CXXFLAGS you could add it to the pkg-config variables dumped into vdr.pc so that plugin authors can use it if they need it. Currently its not available in either place. ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
On 16.02.2013 11:43, Joachim Wilke wrote: 2013/2/15 Klaus Schmidinger klaus.schmidin...@tvdr.de: I'm not planning to make any more changes to the Makefiles for version 2.0.0. So it will remain broken. Thats sad, I provided the patch already before release of 1.7.37. Well, according to Christopher Reimer it isn't broken: http://www.linuxtv.org/pipermail/vdr/2013-January/027173.html Klaus ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
2013/1/26 Christopher Reimer c.reimer1...@gmail.com: No, I tried it and it works with and without your patch. CINCLUDES has a lower priority, the order of the parameters matters. And the default INCLUDE stuff comes after CXXFLAGS (CINCLUDES is a part of CXXFLAGS in plugin makefiles) As the fix is not yet included in 1.7.37, is it scheduled for the next release? -- Best Regards, Joachim. ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
On 15.02.2013 11:14, Joachim Wilke wrote: 2013/1/26 Christopher Reimer c.reimer1...@gmail.com: No, I tried it and it works with and without your patch. CINCLUDES has a lower priority, the order of the parameters matters. And the default INCLUDE stuff comes after CXXFLAGS (CINCLUDES is a part of CXXFLAGS in plugin makefiles) As the fix is not yet included in 1.7.37, is it scheduled for the next release? I'm not planning to make any more changes to the Makefiles for version 2.0.0. Klaus ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
No, I tried it and it works with and without your patch. CINCLUDES has a lower priority, the order of the parameters matters. And the default INCLUDE stuff comes after CXXFLAGS (CINCLUDES is a part of CXXFLAGS in plugin makefiles) 2013/1/26 Joachim Wilke joachim.wi...@gmail.com: 2013/1/25 Christopher Reimer c.reimer1...@gmail.com: Hmm, it is problematic. We can't make sure that make plugins uses the header files of the current source tree. plugins of 1.7.36 should be compiled against the 1.7.36 headers, even if there are older header files in /usr/include/vdr. This is an issue even without my suggested patch, right? -- Best Regards, Joachim. ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
2013/1/23 Joachim Wilke joachim.wi...@gmail.com: 2013/1/22 Klaus Schmidinger klaus.schmidin...@tvdr.de On 20.01.2013 20:24, Andreas Brachold wrote: I think into vdr.pc is parameter includedir= missed, if $(INCDIR) not a standard directory. If you have an installed version of VDR on your system (i.e. there is a vdr.pc file in /usr/share/pkgconfig) and you 'make' a plugin from within the plugin's source directory, the information stored in that vdr.pc file will be used. I think what Andreas means ist, that the INCDIR of the *installed* VDR headers is currently not included in the CXXFLAGS in vdr.pc. This causes plugins to not compile for themselves even if VDR has installed (using make install) before. But it works without any problems here. You don't need to add /usr/include or /usr/local/include to the CXXFLAGS because they are always included. I can't imagine a reason why the includedir should not be a standard directory. ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
Hi, Am Freitag, den 25.01.2013, 15:31 +0100 schrieb Christopher Reimer: I can't imagine a reason why the includedir should not be a standard directory. I use this, to install multiple versions side by side. Please remember you, the program vdr-1.7.x is a developer version ... Andreas ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
Ever tried LCLBLD=1 or ONEDIR=1 ? Christopher 2013/1/25 Andreas Brachold m...@deltab.de: Hi, Am Freitag, den 25.01.2013, 15:31 +0100 schrieb Christopher Reimer: I can't imagine a reason why the includedir should not be a standard directory. I use this, to install multiple versions side by side. Please remember you, the program vdr-1.7.x is a developer version ... Andreas ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
2013/1/25 Christopher Reimer c.reimer1...@gmail.com I think what Andreas means ist, that the INCDIR of the *installed* VDR headers is currently not included in the CXXFLAGS in vdr.pc. This causes plugins to not compile for themselves even if VDR has installed (using make install) before. But it works without any problems here. You don't need to add /usr/include or /usr/local/include to the CXXFLAGS because they are always included. I can't imagine a reason why the includedir should not be a standard directory. So your imagination is limited. Nvertheless, it is no good practise to expect something to be included by default. As long as INCDIR or PREFIX can be changed in Make.config, this *has* to be considered when creating vdr.pc. I'm not requesting a feature, I'm just providing a bugfix. ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
OK, OK, OK... If it fixes your problem I don't see any problem with adding INCDIR to CINCLUDES. I was a bit worried about the DESTDIR stuff, but this change doesn't affect anything there. Christopher 2013/1/25 Joachim Wilke joachim.wi...@gmail.com: 2013/1/25 Christopher Reimer c.reimer1...@gmail.com I think what Andreas means ist, that the INCDIR of the *installed* VDR headers is currently not included in the CXXFLAGS in vdr.pc. This causes plugins to not compile for themselves even if VDR has installed (using make install) before. But it works without any problems here. You don't need to add /usr/include or /usr/local/include to the CXXFLAGS because they are always included. I can't imagine a reason why the includedir should not be a standard directory. So your imagination is limited. Nvertheless, it is no good practise to expect something to be included by default. As long as INCDIR or PREFIX can be changed in Make.config, this *has* to be considered when creating vdr.pc. I'm not requesting a feature, I'm just providing a bugfix. ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
Hmm, it is problematic. We can't make sure that make plugins uses the header files of the current source tree. plugins of 1.7.36 should be compiled against the 1.7.36 headers, even if there are older header files in /usr/include/vdr. 2013/1/25 Christopher Reimer c.reimer1...@gmail.com: OK, OK, OK... If it fixes your problem I don't see any problem with adding INCDIR to CINCLUDES. I was a bit worried about the DESTDIR stuff, but this change doesn't affect anything there. Christopher 2013/1/25 Joachim Wilke joachim.wi...@gmail.com: 2013/1/25 Christopher Reimer c.reimer1...@gmail.com I think what Andreas means ist, that the INCDIR of the *installed* VDR headers is currently not included in the CXXFLAGS in vdr.pc. This causes plugins to not compile for themselves even if VDR has installed (using make install) before. But it works without any problems here. You don't need to add /usr/include or /usr/local/include to the CXXFLAGS because they are always included. I can't imagine a reason why the includedir should not be a standard directory. So your imagination is limited. Nvertheless, it is no good practise to expect something to be included by default. As long as INCDIR or PREFIX can be changed in Make.config, this *has* to be considered when creating vdr.pc. I'm not requesting a feature, I'm just providing a bugfix. ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
s pozdravom Marek Hajduk France-tech s.r.o. konatel spolocnosti -Original Message- From: Christopher Reimer c.reimer1...@gmail.com Sender: vdr-boun...@linuxtv.org Date: Fri, 25 Jan 2013 18:00:09 To: VDR Mailing Listvdr@linuxtv.org Reply-To: VDR Mailing List vdr@linuxtv.org Subject: Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36 Ever tried LCLBLD=1 or ONEDIR=1 ? Christopher 2013/1/25 Andreas Brachold m...@deltab.de: Hi, Am Freitag, den 25.01.2013, 15:31 +0100 schrieb Christopher Reimer: I can't imagine a reason why the includedir should not be a standard directory. I use this, to install multiple versions side by side. Please remember you, the program vdr-1.7.x is a developer version ... Andreas ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
2013/1/22 Klaus Schmidinger klaus.schmidin...@tvdr.de On 20.01.2013 20:24, Andreas Brachold wrote: I think into vdr.pc is parameter includedir= missed, if $(INCDIR) not a standard directory. If you have an installed version of VDR on your system (i.e. there is a vdr.pc file in /usr/share/pkgconfig) and you 'make' a plugin from within the plugin's source directory, the information stored in that vdr.pc file will be used. I think what Andreas means ist, that the INCDIR of the *installed* VDR headers is currently not included in the CXXFLAGS in vdr.pc. This causes plugins to not compile for themselves even if VDR has installed (using make install) before. -- Best Regards, Joachim. ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
On 20.01.2013 20:24, Andreas Brachold wrote: Hi, I think into vdr.pc is parameter includedir= missed, if $(INCDIR) not a standard directory. After a recreating of a new PLUGIN Makefile, building from vdr root directory work, but i can't build plugin from own directory. # cd PLUGINS/src/dvdswitch/ # make all g++ -g -O0 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -c -DPLUGIN_NAME_I18N='dvdswitch' -o dvdswitch.o dvdswitch.c dvdswitch.c:10:24: fatal error: vdr/plugin.h: No such file or directory compilation terminated. make: *** [dvdswitch.o] Error 1 If you have an installed version of VDR on your system (i.e. there is a vdr.pc file in /usr/share/pkgconfig) and you 'make' a plugin from within the plugin's source directory, the information stored in that vdr.pc file will be used. If, in such a scenario, you want to build the plugin using VDR header files from the VDR source directory, you need to do make VDRDIR=/path/to/your/vdr/source If you want to be able to build a plugin from within its source directory, and use the VDR source from ../../.. without setting the VDRDIR macro, you need to make sure that there is no vdr.pc in /usr/share/pkgconfig. Klaus ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
[vdr] [ANNOUNCE] VDR developer version 1.7.36
VDR developer version 1.7.36 is now available at ftp://ftp.tvdr.de/vdr/Developer/vdr-1.7.36.tar.bz2 A 'diff' against the previous version is available at ftp://ftp.tvdr.de/vdr/Developer/vdr-1.7.35-1.7.36.diff MD5 checksums: e514f72a2a8c44f39e47b540d6ad325f vdr-1.7.36.tar.bz2 9312a0d10bcda87d3c3c7e6dfbebcd05 vdr-1.7.35-1.7.36.diff WARNING: This is a *developer* version. Even though *I* use it in my productive environment. I strongly recommend that you only use it under controlled conditions and for testing and debugging. The changes since version 1.7.35: - Added maximum SNR value for PCTV Systems nanoStick T2 290e (thanks to Antti Hartikainen). - Added a remark indicating that the coordinates of Rect in a call to cDevice::CanScaleVideo() are in the range of the width and height returned by GetOsdSize() (suggested by Reinhard Nissl). - Modified the Makefiles (thanks to Christopher Reimer). By default VDR is now built according to the FHS (File system Hierarchy Standard), and a plain make in the VDR source directory just builds everything, but doesn't copy it to ./PLUGINS/lib and ./locale any more. You can use a Make.config file (copied from Make.config.template) and set the parameter LCLBLD=1 to have everything built and installed under the VDR source tree (as was the default in previous versions). If you already have your own Make.config file, you may want to copy the new Make.config.template and adapt it to your needs. If you don't want VDR's data files to be spread around your system according to the FHS, you can set the parameter ONEDIR=1 (using Make.config) to have all files in one /video directory as before. - Fixed the example for cReceiver in PLUGINS.html. - Fixed sorting recordings in case two folders have the same name, but one of them ends in an additional digit, as in abc and abc2 (reported by Andreas Mair). - Added repeat function when using the keyboard to control VDR (thanks to Reinhard Nissl). - The SVDRP command LSTR now knows the additional parameter path, which can be given to get the actual file name of a recording's directory (suggested by Stefan Stolz). - Fixed multiple occurrences of the same directory in the recordings list in case there are directories that only differ in non-alphanumeric characters (reported by Andreas Mair). - Absolute jumps when replaying a recording (via the Red key) are now only performed if an actual value has been entered (suggested by Ulf Kiener). - The last replayed recording is now stored in setup.conf, which allows the blue Resume key in the main menu to work even after a restart of VDR. - The SVDRP command NEWT no longer checks whether a timer with the given data already exists (suggested by Malte Forkel). - Implemented scaling of SPU bitmaps (thanks to Johann Friedrichs). - Improved cutting MPEG-2 video (thanks to Sören Moch). - Reduced the number of retries in cTransfer::Receive() to avoid blocking recordings in case the primary device can't handle the current live signal. Have fun! Klaus ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
Le dimanche 20 janvier 2013 14:52:09, Klaus Schmidinger a écrit : - The last replayed recording is now stored in setup.conf, which allows the blue Resume key in the main menu to work even after a restart of VDR. Thanks for this function ;o) Best regards ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
I noticed that DVBDIR is missing in the new Make.config.template. I use media_build drivers and pre-VDR-1.7.36 would set DVBDIR to point to my media_build dir. How would I do that now? ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
On 20.01.2013 19:06, VDR User wrote: I noticed that DVBDIR is missing in the new Make.config.template. I use media_build drivers and pre-VDR-1.7.36 would set DVBDIR to point to my media_build dir. How would I do that now? Just write it into your Make.config as usual, it will be used. Apparently, with all that patching, the DVBDIR example in Make.config.template got lost. I'll add it in the next version. Klaus ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [ANNOUNCE] VDR developer version 1.7.36
Hi,
I think into vdr.pc is parameter includedir= missed, if $(INCDIR) not a
standard directory.
After a recreating of a new PLUGIN Makefile, building from vdr root
directory work, but i can't build plugin from own directory.
# cd PLUGINS/src/dvdswitch/
# make all
g++ -g -O0 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC
-D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE
-D_LARGEFILE64_SOURCE -c -DPLUGIN_NAME_I18N='dvdswitch' -o
dvdswitch.o dvdswitch.c
dvdswitch.c:10:24: fatal error: vdr/plugin.h: No such file or directory
compilation terminated.
make: *** [dvdswitch.o] Error 1
Andreas
--- Makefile.org 2013-01-12 14:45:01.0 +0100
+++ Makefile 2013-01-20 19:51:31.0 +0100
@@ -149,8 +149,9 @@
@echo locdir=$(LOCDIR) $@
@echo plgcfg=$(PLGCFG) $@
@echo apiversion=$(APIVERSION) $@
- @echo cflags=$(CFLAGS) $(CDEFINES) $(CINCLUDES) $(HDRDIR) $@
- @echo cxxflags=$(CXXFLAGS) $(CDEFINES) $(CINCLUDES) $(HDRDIR) $@
+ @echo includedir=$(DESTDIR)$(INCDIR) $@
+ @echo cflags=$(CFLAGS) $(CDEFINES) $(CINCLUDES) $(HDRDIR) -I\$${includedir} $@
+ @echo cxxflags=$(CXXFLAGS) $(CDEFINES) $(CINCLUDES) $(HDRDIR) -I\$${includedir} $@
@echo $@
@echo Name: VDR $@
@echo Description: Video Disk Recorder $@
___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
