kugel- commented on this pull request.

I'm not sure this change is required at all.

Yes, there may be a circular dependency. But this is not a problem since 
tagmanager is build statically only.

Are their other projects that have a tagmanager or fork or has it become more 
or less geany-specific?

>  geany_includedir = $(includedir)/geany
 geany_include_HEADERS = \
+       $(TM_HDRS) \

You want to create a separate variable like this:

tagmanager_includedir = $(includedir)/geany/tagmanager
tagmanager_include_HEADERS = $(TM_HDRS)

Then headers will be installed in the subdirectory.

-- 
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/pull/1414#pullrequestreview-29780713

Reply via email to