[FFmpeg-cvslog] r21659 - trunk/configure

Ramiro Polla ramiro.polla
Wed Feb 10 14:40:45 CET 2010


On Mon, Feb 8, 2010 at 11:14 PM, Ramiro Polla <ramiro.polla at gmail.com> wrote:
> On Mon, Feb 8, 2010 at 8:51 PM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
>> Ramiro Polla <ramiro.polla <at> gmail.com> writes:
>>> > libgsm installs headers in a subdirectory, use gsm.h from that subdirectory.
>>> > Approved by Diego at FOSDEM.
>>>
>>> Was this even tested?
>>>
>>> It should need attached patch.
>>>
>>> Attachment (gsm_gsm_h.diff): application/octet-stream, 427 bytes
>>
>> Please apply, my suspicion is that /usr/include/gsm.h was added because some
>> application (FFmpeg or another one) used it instead of /usr/include/gsm/gsm.h.
>
> I'd rather wait for Diego's or Reinhard's comment first.

Diego ping... You approved this change, so could you please comment?

Carl prefers gsm/gsm.h, I prefer just gsm.h. The later can be fixed in
configure with --extra-(c|ld)flags, the former can't.



More information about the ffmpeg-cvslog mailing list