[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