Explicit gthread linking required about resynthesizer HOT 4 CLOSED

sed -i 's/LDADD.*/\0 -lgthread-2.0/' src/Makefile.am

from resynthesizer.

Thanks. Please give your advice on the following question:

g_thread_init() is deprecated in glib starting at some (that I can't say without looking it up) version of glib. Should I eliminate g_thread_init from the code? That might break resynthesizer for users of older versions of glib.

I don't know whether lgthread-2.0 exists in older versions of GLIB. I would guess that it does exist in older versions of GLIB, and that this build error is about a change in the linker: it no longer links in dependent libraries. See the resynthesizer issue about not building because lmath was not explicitly specified in LDADD.

On the other hand, it seems strange that only g_thread_init() is undefined and not all the threading functions ofGLIB. Maybe lgthread-2.0 is a compatibility library only included in newer versions of GLIB?

Bottom line: I am still studying the issue. I am open to your fix. I am not yet sure it is backward compatible for older GLIB.

from resynthesizer.

libgthread-2.0 is present at least as far back as glib 2.16 from early 2008 (I checked the package in Ubuntu Hardy), so explicitly linking it should be safe.

I wouldn't recommend dropping the old threading API entirely at this point; glib 2.32 (which is where that API was deprecated) only came out this year. You could #ifdef the functions in question (the changes affect more than g_thread_init(); there's a list of deprecated functions here), but I'm not sure there'd be any particular benefit to doing that.

from resynthesizer.

Thanks. Fixed.

from resynthesizer.