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, siret...@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

Attachment: signature.asc
Description: Digital signature

pkg-multimedia-maintainers mailing list

Reply via email to