This one time, at band camp, Bj|rn Englund said: > Sun Apr 30 2006, Stephen Gran wrote: > > This one time, at band camp, Björn Englund said: > > > I forgot to attach the patch in the previous email, here it is. > > > > > + dvd_stat_t statbuf; > > > + > > > + if ( DVDFileStat(_dvd, 0, DVD_READ_INFO_FILE, &statbuf) != -1 ) { > > > > The struct dvd_stat_t and the function DVDFileStat appear not to be > > defined in the libdvdread headers, so this patch doesn't work as I expected > > it to. Or is this patch based on a version of dvdbackup where these are > > defined in dvdbackup itself? Being dead upstream, it has probably > > forked significantly in different people's hands. > > > As I wrote in the previous email, this patch works with the cvs of libdvdread > which will be released as libdvdread-0.9.6 next week (probably tomorrow).
Oh, I see - I misunderstood that part. Sorry about that. I will retest after the new libdvdread is released. > And I am pretty sure dvdbackup is the only program that is using udffindfile, > I have not seen any comments about it for other programs. OK - as I said, if we are fairly sure that dvdbackup is the only one, then it's likely not a problem. I was bringing it up because it is common practice, so as to avoid problems with apps we don't know about. > And it was not included in the installed header files in 0.9.4. True. Well, I leave to to your discretion, then. Take care, -- ----------------------------------------------------------------- | ,''`. Stephen Gran | | : :' : [EMAIL PROTECTED] | | `. `' Debian user, admin, and developer | | `- http://www.debian.org | -----------------------------------------------------------------
signature.asc
Description: Digital signature