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 |
 -----------------------------------------------------------------

Attachment: signature.asc
Description: Digital signature

Reply via email to