[Github-comments] [geany/geany-plugins] Added common utils lib (2). (#622)

Matthew Brush notifications at xxxxx
Sun Oct 15 17:03:00 UTC 2017


I also mentioned some weirdness about `AM_LDFLAGS` in [the PR](https://github.com/LarsGit223/geany-plugins/pull/1) I made for [the other/first](https://github.com/geany/geany-plugins/pull/620) PR for this, but it's mysteriously missing now. I think it was that it cannot be used by anything but plugins since it puts libtool `-module -avoid-version` flags in there. That is why [the Makefile.am explicitly uses `GP_LDFLAGS`](https://github.com/geany/geany-plugins/pull/622/commits/1f6923e012aa26fd95fc3f3cabd66e2b7d32ee20#diff-00b6bde048b921fd76923b9d7416829cR13) instead of implicitly inheriting the `AM_LDFLAGS`.

Also, I left [a TODO](1f6923e012aa26fd95fc3f3cabd66e2b7d32ee20#diff-4bf485625e498fde51df9a8ce1925c08R4) in the .m4 template that might be nice to fix.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/622#issuecomment-336725504
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20171015/4d5fe6b0/attachment-0001.html>


More information about the Github-comments mailing list