[SCM] libva/master: use variable $(X11_LIBS) instead of hardcoded -lX11
dr at jones.dk
Sat Jan 8 14:24:28 UTC 2011
On Sat, Jan 08, 2011 at 02:33:34PM +0100, Reinhard Tartler wrote:
>On Sat, Jan 08, 2011 at 14:14:04 (CET), Jonas Smedegaard wrote:
>> On Sat, Jan 08, 2011 at 01:00:38PM +0000,
>> siretart at users.alioth.debian.org wrote:
>>> use variable $(X11_LIBS) instead of hardcoded -lX11
>>>-+ -lX11 -lpthread
>> Above seems like an accident: Was it deliberate to replace not only
>> -X11 but also -lpthread there?
>the -lpthread was added by me a few commits before. I've noticed that
>doku fixed this bug this way a few minutes before I uploaded it to
>ubuntu. Anyways, I've taken his approach and it turns out that it
>builds this way properly, so I've dropped the -lpthread part.
I must say it surprised me: you tend to not make errors like that :-)
In the future it would be better IMO to commit such unrelated changes
separately (or at least mention both in the commit message). Even if
just few commits away, that still helps later investigation.
* Jonas Smedegaard - idealist & Internet-arkitekt
* Tlf.: +45 40843136 Website: http://dr.jones.dk/
[x] quote me freely [ ] ask before reusing [ ] keep private
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 836 bytes
Desc: Digital signature
More information about the pkg-multimedia-maintainers