cvs commit: ports/multimedia/ffmpeg-devel Makefile ports/multimedia/ffmpeg-devel/files patch-libavcodec_x264.c patch-libavformat_movenc.c

[Shaun Amott]

On Fri, Sep 29, 2006 at 01:28:34PM -0400, Kris Kennaway wrote:
> On Fri, Sep 29, 2006 at 07:23:46AM -0400, Michael Johnson wrote:
> > On 9/29/06, Michael Johnson <ahze at ahze.net> wrote:
> > >
> > >
> > >
> > >On 9/28/06, Kris Kennaway <kris at obsecurity.org> wrote:
> > >>
> > >> On Sun, Sep 24, 2006 at 08:52:52PM -0400, Kris Kennaway wrote:
> > >> > On Mon, Sep 25, 2006 at 12:47:40AM +0000, Mario Sergio Fujikawa
> > >> Ferreira wrote:
> > >> > > lioux       2006-09-25 00:47:40 UTC
> > >> > >
> > >> > >   FreeBSD ports repository
> > >> > >
> > >> > >   Modified files:
> > >> > >     multimedia/ffmpeg-devel Makefile
> > >> > >     multimedia/ffmpeg-devel/files patch-libavcodec_x264.c
> > >> > >   Added files:
> > >> > >     multimedia/ffmpeg-devel/files patch-libavformat_movenc.c
> > >> > >   Log:
> > >> > >   Fix build on 4.x: fix devel/sdl header include
> > >> > >
> > >> > >   PR:             102431
> > >> > >   Submitted by:   Shaun Amott <shaun at FreeBSD.org >
> > >> >
> > >> > Are you sure this is still correct after the recent SDL changes?
> > >> >
> > >> > Kris
> > >>
> > >> Ping.  It appears to be broken.
> > >
> > >
> > patch-libavcodec_x264.c needs to be readded. It's there but it's empty.
> 
> Maybe so, but the Makefile also explicitly patches it to #include
> <SDL11/...> too, which is the wrongness I was referring to (SDL
> headers are now in <SDL/...>.
> 

Yes, this is wrong. This change was not part of my patch, even though I
seem to have been credited for it.

-- 
Shaun Amott // PGP: 0x6B387A9A
"A foolish consistency is the hobgoblin
of little minds." - Ralph Waldo Emerson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-all/attachments/20060929/af783710/attachment.pgp