[FFmpeg-cvslog] r21659 - trunk/configure

[Måns Rullgård]

Ramiro Polla <ramiro.polla at gmail.com> writes:

> On Mon, Feb 8, 2010 at 12:09 PM, Ramiro Polla <ramiro.polla at gmail.com> wrote:
>> On Sat, Feb 6, 2010 at 2:52 PM, siretart <subversion at mplayerhq.hu> wrote:
>>> Author: siretart
>>> Date: Sat Feb ?6 17:52:38 2010
>>> New Revision: 21659
>>>
>>> Log:
>>> 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.
>
> Actually I see no such mention in the libgsm Makefile to install the
> header to include/gsm/gsm.h. And in my ubuntu I see "dpkg -L
> libgsm1-dev":
> [...]
> /usr/include/gsm/gsm.h
> [...]
> /usr/include/gsm.h
> [...]
>
> ls -l /usr/include/gsm.h
> lrwxrwxrwx 1 root root 9 2008-11-14 16:25 /usr/include/gsm.h -> gsm/gsm.h
>
> So I see no reason for this patch, even if it were related to how
> things are sorted on Debian.
>
> Diego, ok to revert?

So debian installs the header in two places.  We should be using
whichever is more official, and we should obviously use the same
name everywhere.

-- 
M?ns Rullg?rd
mans at mansr.com