{"action":"created","issue":{"url":"https://api.github.com/repos/ElektraInitiative/libelektra/issues/177","labels_url":"https://api.github.com/repos/ElektraInitiative/libelektra/issues/177/labels{/name}","comments_url":"https://api.github.com/repos/ElektraInitiative/libelektra/issues/177/comments","events_url":"https://api.github.com/repos/ElektraInitiative/libelektra/issues/177/events","html_url":"https://github.com/ElektraInitiative/libelektra/issues/177","id":55159767,"number":177,"title":"Header files of libtool installed incorrectly","user":{"login":"markus2330","id":7949075,"avatar_url":"https://avatars.githubusercontent.com/u/7949075?v=3","gravatar_id":"","url":"https://api.github.com/users/markus2330","html_url":"https://github.com/markus2330","followers_url":"https://api.github.com/users/markus2330/followers","following_url":"https://api.github.com/users/markus2330/following{/other_user}","gists_url":"https://api.github.com/users/markus2330/gists{/gist_id}","starred_url":"https://api.github.com/users/markus2330/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/markus2330/subscriptions","organizations_url":"https://api.github.com/users/markus2330/orgs","repos_url":"https://api.github.com/users/markus2330/repos","events_url":"https://api.github.com/users/markus2330/events{/privacy}","received_events_url":"https://api.github.com/users/markus2330/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"fberlakovich","id":7955528,"avatar_url":"https://avatars.githubusercontent.com/u/7955528?v=3","gravatar_id":"","url":"https://api.github.com/users/fberlakovich","html_url":"https://github.com/fberlakovich","followers_url":"https://api.github.com/users/fberlakovich/followers","following_url":"https://api.github.com/users/fberlakovich/following{/other_user}","gists_url":"https://api.github.com/users/fberlakovich/gists{/gist_id}","starred_url":"https://api.github.com/users/fberlakovich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fberlakovich/subscriptions","organizations_url":"https://api.github.com/users/fberlakovich/orgs","repos_url":"https://api.github.com/users/fberlakovich/repos","events_url":"https://api.github.com/users/fberlakovich/events{/privacy}","received_events_url":"https://api.github.com/users/fberlakovich/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-01-22T14:34:54Z","updated_at":"2015-01-22T21:30:40Z","closed_at":null,"body":"The header files related to merging are located in a subfolder \"merging\". However, they are not installed in such a subdirectory. So trying to include them fails when Elektra is installed.\r\n\r\nI would propose to not add them in the public API at all, but instead only provide the simple new high-level API we discussed.\r\n\r\nAdditionally, files should ideally be prefixed with kdb, because elektra enforces the user to put /usr/include/elektra into the include path (otherwise recursive inclusion fails). So I would suggest kdbmerge.h as new name for the high level merge API."},"comment":{"url":"https://api.github.com/repos/ElektraInitiative/libelektra/issues/comments/71102827","html_url":"https://github.com/ElektraInitiative/libelektra/issues/177#issuecomment-71102827","issue_url":"https://api.github.com/repos/ElektraInitiative/libelektra/issues/177","id":71102827,"user":{"login":"fberlakovich","id":7955528,"avatar_url":"https://avatars.githubusercontent.com/u/7955528?v=3","gravatar_id":"","url":"https://api.github.com/users/fberlakovich","html_url":"https://github.com/fberlakovich","followers_url":"https://api.github.com/users/fberlakovich/followers","following_url":"https://api.github.com/users/fberlakovich/following{/other_user}","gists_url":"https://api.github.com/users/fberlakovich/gists{/gist_id}","starred_url":"https://api.github.com/users/fberlakovich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fberlakovich/subscriptions","organizations_url":"https://api.github.com/users/fberlakovich/orgs","repos_url":"https://api.github.com/users/fberlakovich/repos","events_url":"https://api.github.com/users/fberlakovich/events{/privacy}","received_events_url":"https://api.github.com/users/fberlakovich/received_events","type":"User","site_admin":false},"created_at":"2015-01-22T21:30:40Z","updated_at":"2015-01-22T21:30:40Z","body":"Should the private headers (e.g. automergestrategy.hpp) also be renamed (e.g. kdbautomergestrategy.hpp)? Am I right that making those headers private simply means not installing them via CMake?\r\n\r\nIn addition I would be glad to make them public again somewhere in the future when the merge API is more mature. The whole framewwork was designed from ground up to be highly flexible and modular and removing this flexibility by hiding the building blocks hurts. However, I totally understand that the framework is currently hard to use and the headers won't be of any use at the moment."}}