{"action":"created","issue":{"url":"https://api.github.com/repos/Aseman-Land/libqtelegram-aseman-edition/issues/16","labels_url":"https://api.github.com/repos/Aseman-Land/libqtelegram-aseman-edition/issues/16/labels{/name}","comments_url":"https://api.github.com/repos/Aseman-Land/libqtelegram-aseman-edition/issues/16/comments","events_url":"https://api.github.com/repos/Aseman-Land/libqtelegram-aseman-edition/issues/16/events","html_url":"https://github.com/Aseman-Land/libqtelegram-aseman-edition/issues/16","id":109808529,"number":16,"title":"Install directories messed up","user":{"login":"nylocx","id":3329124,"avatar_url":"https://avatars.githubusercontent.com/u/3329124?v=3","gravatar_id":"","url":"https://api.github.com/users/nylocx","html_url":"https://github.com/nylocx","followers_url":"https://api.github.com/users/nylocx/followers","following_url":"https://api.github.com/users/nylocx/following{/other_user}","gists_url":"https://api.github.com/users/nylocx/gists{/gist_id}","starred_url":"https://api.github.com/users/nylocx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nylocx/subscriptions","organizations_url":"https://api.github.com/users/nylocx/orgs","repos_url":"https://api.github.com/users/nylocx/repos","events_url":"https://api.github.com/users/nylocx/events{/privacy}","received_events_url":"https://api.github.com/users/nylocx/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-05T14:30:13Z","updated_at":"2015-10-05T15:58:02Z","closed_at":null,"body":"Hi, I co-Maintain the libqtelegram-ae packages on the Arch User Repository and I noticed that the packages are currently broken due to a bug in the header install routine.\r\n\r\nWith commit 31bb7bcdfd757904b0cfc129458e23920bbca8d9 the PREFIX variable got removed so I no longer see a way to customize the INSTALL_PREFIX variable (Yes I know I could use sed in my build script, but I'm talking about official solutions here) which will make the packages non compliant with the Arch packaging standard as it is not good practice to install into the qt directory on the target system. \r\nThe second change that messed it up even more for me is: a40c2b755ed0dd99b618255d48373f2ecc3ea334\r\n\r\nBy replacing the directory references with $$PWD the paths now extend to my build jail path which would be ok if not they where used by the install routine to install header files in [qmake/headerinstall.pri](../blob/master/qmake/headerinstall.pri).\r\nThis leads to install paths like \"/usr/include/qt/libqtelegram-ae/tmp/makepkg/libqtelegram-ae-git/src/libqtelegram-ae-git/telegram/types/types.h\" for my system.\r\n\r\nI hope the problem got clear if you have any more questions pleas feel free to ask."},"comment":{"url":"https://api.github.com/repos/Aseman-Land/libqtelegram-aseman-edition/issues/comments/145580099","html_url":"https://github.com/Aseman-Land/libqtelegram-aseman-edition/issues/16#issuecomment-145580099","issue_url":"https://api.github.com/repos/Aseman-Land/libqtelegram-aseman-edition/issues/16","id":145580099,"user":{"login":"nylocx","id":3329124,"avatar_url":"https://avatars.githubusercontent.com/u/3329124?v=3","gravatar_id":"","url":"https://api.github.com/users/nylocx","html_url":"https://github.com/nylocx","followers_url":"https://api.github.com/users/nylocx/followers","following_url":"https://api.github.com/users/nylocx/following{/other_user}","gists_url":"https://api.github.com/users/nylocx/gists{/gist_id}","starred_url":"https://api.github.com/users/nylocx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nylocx/subscriptions","organizations_url":"https://api.github.com/users/nylocx/orgs","repos_url":"https://api.github.com/users/nylocx/repos","events_url":"https://api.github.com/users/nylocx/events{/privacy}","received_events_url":"https://api.github.com/users/nylocx/received_events","type":"User","site_admin":false},"created_at":"2015-10-05T15:58:02Z","updated_at":"2015-10-05T15:58:02Z","body":"First of all thanks for the quick response.\r\n\r\n> Did it fix the problem?\r\n\r\nAlmost ;)\r\nIMHO using a architecture specific directory for the libraries is debian or even ubuntu specific\r\n1c4e0d7005809a541000d4244238f5c57ff2f175#diff-99345e3cc43efe6e02a4db58c9e84b5dL18\r\n\r\nArch Linux follows the Filesystem Hierarchy Standard (FHS) more closely and uses /usr/lib for the libraries. Put that is something I can handle by patching the pro file via sed if it is out of scope for you.\r\n\r\n"}}