On 16.12.2013 18:11, Marko Mäkelä wrote:
Hi all,
I am not sure if this was reported already, but I encountered this bug when trying to edit a recording without setting up VDR:
./vdr -v /tmp/video --edit /tmp/video/Rec_name/2013-08-12.20.58.50.99.rec
Without the attached patch, VDR would try to create /srv/vdr/video, and would fail to cut the recording.
With the patch, VDR successfully cut the recording. The source files were in PES format, and also the new recording is in PES format. Based on a discussion from 2008, this is the expected behaviour. Is there some tool for converting old PES recordings to TS format?
Marko
--- vdr.c 2013-10-16 12:46:36.000000000 +0300
+++ vdr.c 2013-12-16 18:47:42.526981566 +0200
@@ -331,6 +331,7 @@ int main(int argc, char *argv[])
}
break;
case 'e' | 0x100:
+ SetVideoDirectory(VideoDirectory);
return CutRecording(optarg) ? 0 : 2;
case 'E': EpgDataFileName = (*optarg != '-' ? optarg : NULL);
break;
I think I'll do it this way:
-------------------------------------------------------------------------------------------
--- vdr.c 2013/12/25 11:01:28 3.6
+++ vdr.c 2013/12/25 11:24:26
@@ -223,6 +223,7 @@
VdrUser = VDR_USER;
#endif
+ cVideoDirectory::SetName(VideoDirectory);
cPluginManager PluginManager(DEFAULTPLUGINDIR);
static struct option long_options[] = {
@@ -443,6 +444,7 @@
case 'v': VideoDirectory = optarg;
while (optarg && *optarg && optarg[strlen(optarg) - 1] == '/')
optarg[strlen(optarg) - 1] = 0;
+ cVideoDirectory::SetName(VideoDirectory);
break;
case 'w': if (isnumber(optarg)) {
int t = atoi(optarg);
@@ -663,7 +665,6 @@
// Directories:
- cVideoDirectory::SetName(VideoDirectory);
if (!ConfigDirectory)
ConfigDirectory = DEFAULTCONFDIR;
cPlugin::SetConfigDirectory(ConfigDirectory);
-------------------------------------------------------------------------------------------
This also fixes a possible segfault with --genindex and takes care of any other
places that would need the video directory name to be properly set in the future.
Klaus
_______________________________________________
vdr mailing list
vdr@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
