[FFmpeg-devel] [PATCH] Use pkg-config for libschroedinger libs

JonY 10walls
Tue May 13 02:03:13 CEST 2008


Ramiro Polla wrote:
> JonY wrote:
>> Ramiro Polla wrote:
>>> Reimar D?ffinger wrote:
>>>> On Mon, May 12, 2008 at 05:13:04PM +0100, Ramiro Polla wrote:
>>>>>>>>> also pkg-config is used in the line above for cflags,
>>>>>>>>> the patch just fixes one missing use of pkg-config
>>>>>>>> Two wrongs don't make a right.
>>>>>>> just pointing it out, leaving it like it is now sure is the worst
>>>>>>> solution.
>>>>>>>
>>>>>>> a) use pkg-config and use it in all cases
>>>>>>> b) dont use pkg-config at all
>>>>>>>
>>>>>>> the patch fixes configure to conform to a)
>>>>>>> you propose b) instead?
>>>>>>>
>>>>>>> what are those systems without pkg-config if i may ask?
>>>>>> That's irrelevant. As long as pkg-config isn't mandated by
>>>>>> POSIX, we should assume it might not be there.
>>>>> We should get rid of memalign() too. After all, it's not POSIX...
>>>> Huh? FFmpeg already works without memalign. And there is even a hack
>>>> that allows even the assembler optimizations to work without it.
>>> Well, then it should be a configure option:
>>> ./configure --enable-memalign
>>>
>>>> So to me it seems your example actually confirms Mans' opinion?
>>>> Also I think there is no problem to _also_ try pkg-config, but depending
>>>> on it really is not a great thing - also as it encourages projects using
>>>> pkg-config to do sloppy dependency-managment.
>>> Good idea. It doesn't take an extraordinary amount of time and effort to
>>> come up with an idea like attached patch.
>>>
>>> [...]
>>>
>>> Ramiro Polla
>>>
>>>
>> Hi,
>> Problem is add_cflags still depends on pkg-config for proper cflags. The
>> libschroedinger headers will not work properly without the correct cflags.
>>
>> Maybe configure should check for pkg-config earlier and determine how to
>> proceed if its not installed.
>
> Right, and if pkg-config is not installed we can maybe read a few bytes
> from /dev/urandom and feed them as CFLAGS.
>
> By the way, could you give us an example of what the libschroedinger
> CFLAGS look like (I'm too lazy to look it up atm =).
>

Hi,
On MSYS/MinGW, its

$ pkg-config --cflags schroedinger-1.0
-Ic:/mingw/include/schroedinger-1.0 -Ic:/mingw/include/liboil-0.3




More information about the ffmpeg-devel mailing list