[SCM] libva/master: use variable $(X11_LIBS) instead of hardcoded -lX11

Jonas Smedegaard 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
>>>++  $(X11_LIBS)
>>
>>
>> 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.

Ah, ok.

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

-- 
  * 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
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20110108/72cf01f4/attachment-0001.pgp>


More information about the pkg-multimedia-maintainers mailing list