[DVDnav-discuss] [PATCH] restore old-upstream-compatible API (was Re: current revision strangeness)

Dominik 'Rathann' Mierzejewski dominik at greysector.net
Sat Aug 30 13:55:01 CEST 2008


On Friday, 29 August 2008 at 23:25, Dominik 'Rathann' Mierzejewski wrote:
> On Wednesday, 20 August 2008 at 21:03, Dominik 'Rathann' Mierzejewski wrote:
> > On Wednesday, 20 August 2008 at 20:00, Nico Sabbi wrote:
> [...]
> > > sorry, I missed or forgotten your post. Do you want to rename
> > > libdvdread as dvdread?
> > 
> > Rename the include path from libdvdread/*.h back to dvdread/*.h, yes.
> > Note that libdvdnav similarly has dvdnav/*.h for public headers.
> > 
> > > why? dvdread is a LIBrary, I don't see why it shouldn't be called
> > > libdvdread.
> > 
> > I'm not suggesting renaming the library itself.
> > 
> > > It's also another way to distinguish it from the old code.
> > 
> > Why do you want to distinguish it from the old code? It's a fork,
> > granted, but it's the only actively developed fork, which makes
> > it upstream.
> > 
> > > Or did I misunderstand what you mean?
> > 
> > Apparently, at least partially. So, do you agree to put the include
> > path back to dvdread/*.h?
> 
> And here's the patch. If there are no objections, I'm going to apply
> this on Monday.

As Nico agreed on the complementary change in MPlayer, I went ahead
and applied this.

Regards,
R.

-- 
Fedora http://fedoraproject.org/wiki/User:Rathann
Livna http://rpm.livna.org | MPlayer http://mplayerhq.hu
"Faith manages."
        -- Delenn to Lennier in Babylon 5:"Confessions and Lamentations"



More information about the DVDnav-discuss mailing list