Hi Sylvain, On Tue, Nov 29, 2005 at 04:02:26PM -0500, [EMAIL PROTECTED] wrote:
> thanks for taking the time to go through these autotools scripts; since > it doesn't seem to break anything on any of my systems, your patch is now > included upstream[1] -- expect a new packaged adesklets release before > December the 14th including your changes. Regards, I'm afraid that applying this patch upstream *does* have a portability impact; the patch will do the right thing on systems that use the GNU linker or a linker with similar features, but there are other Unices that don't support dynamic linking or whose library format doesn't store dependency information, so on these systems a hardcoded -lImlib2 won't work. Also, not all GNU/Linux systems put libImlib2 in the linker search path, so imlib2-config --libs may output -L values needed for linking. So this patch really is specific to Debian (and systems like it), not something that I would consider appropriate to include upstream. To get this fixed upstream, a better answer would be to get imlib2-config to add a separate --static option to be used for static linking, and to teach it how to handle dynamic linking on different platforms. -- Steve Langasek Give me a lever long enough and a Free OS Debian Developer to set it on, and I can move the world. [EMAIL PROTECTED] http://www.debian.org/
signature.asc
Description: Digital signature
