{"action":"created","issue":{"url":"https://api.github.com/repos/libjpeg-turbo/libjpeg-turbo/issues/124","repository_url":"https://api.github.com/repos/libjpeg-turbo/libjpeg-turbo","labels_url":"https://api.github.com/repos/libjpeg-turbo/libjpeg-turbo/issues/124/labels{/name}","comments_url":"https://api.github.com/repos/libjpeg-turbo/libjpeg-turbo/issues/124/comments","events_url":"https://api.github.com/repos/libjpeg-turbo/libjpeg-turbo/issues/124/events","html_url":"https://github.com/libjpeg-turbo/libjpeg-turbo/pull/124","id":193355328,"number":124,"title":"Build: Use GNUInstallDirs for standard path defaults","user":{"login":"rleigh-codelibre","id":9640520,"avatar_url":"https://avatars.githubusercontent.com/u/9640520?v=3","gravatar_id":"","url":"https://api.github.com/users/rleigh-codelibre","html_url":"https://github.com/rleigh-codelibre","followers_url":"https://api.github.com/users/rleigh-codelibre/followers","following_url":"https://api.github.com/users/rleigh-codelibre/following{/other_user}","gists_url":"https://api.github.com/users/rleigh-codelibre/gists{/gist_id}","starred_url":"https://api.github.com/users/rleigh-codelibre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rleigh-codelibre/subscriptions","organizations_url":"https://api.github.com/users/rleigh-codelibre/orgs","repos_url":"https://api.github.com/users/rleigh-codelibre/repos","events_url":"https://api.github.com/users/rleigh-codelibre/events{/privacy}","received_events_url":"https://api.github.com/users/rleigh-codelibre/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-12-04T15:00:08Z","updated_at":"2016-12-04T18:14:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/libjpeg-turbo/libjpeg-turbo/pulls/124","html_url":"https://github.com/libjpeg-turbo/libjpeg-turbo/pull/124","diff_url":"https://github.com/libjpeg-turbo/libjpeg-turbo/pull/124.diff","patch_url":"https://github.com/libjpeg-turbo/libjpeg-turbo/pull/124.patch"},"body":"The build currently uses its own variables (`BINDIR`, `DATADIR`, `DOCDIR`, `INCLUDEDIR`, `LIBDIR`, `MANDIR`) and defaults for these paths, as well as its own defaults for `CMAKE_INSTALL_PREFIX`.\r\n\r\nThis commit makes the build use the CMake [GNUInstallDirs](https://cmake.org/cmake/help/latest/module/GNUInstallDirs.html) module for these variables.  This module matches the behaviour of the GNU Coding Standards as implemented by Autoconf and Automake by default, as well as accounting for several Linux Filesystem Hierarchy Standard (FHS) behaviours and Debian/Ubuntu multiarch library directory naming.\r\n\r\nThis change will:\r\n\r\n- Make the behaviour closer to the expectations of users of the old Autotools build\r\n- Replace custom variables with standard variables which are expected by CMake users\r\n- Provide defaults similar to the old Autotools build\r\n- Provide defaults expected by CMake users for both the install prefix and the various installation directories\r\n- Integrate seamlessly into projects building multiple CMake-based projects which are already using the standard variable naming conventions and defaults\r\n\r\nI have provided a compatibility mapping from the old variables to the GNUInstallDirs variables to provide a clean transition.  I also quoted the variables to cope with spaces in paths explicitly.\r\n\r\n--------\r\n\r\nTesting:\r\n\r\n- Both the old and new option names will continue to work; using the old option names will elicit a warning\r\n- All uses of the old variable names should have been converted\r\n- The logic in the rpm and dpkg scripts might need checking in case there's some subtlety I overlooked"},"comment":{"url":"https://api.github.com/repos/libjpeg-turbo/libjpeg-turbo/issues/comments/264720219","html_url":"https://github.com/libjpeg-turbo/libjpeg-turbo/pull/124#issuecomment-264720219","issue_url":"https://api.github.com/repos/libjpeg-turbo/libjpeg-turbo/issues/124","id":264720219,"user":{"login":"dcommander","id":651941,"avatar_url":"https://avatars.githubusercontent.com/u/651941?v=3","gravatar_id":"","url":"https://api.github.com/users/dcommander","html_url":"https://github.com/dcommander","followers_url":"https://api.github.com/users/dcommander/followers","following_url":"https://api.github.com/users/dcommander/following{/other_user}","gists_url":"https://api.github.com/users/dcommander/gists{/gist_id}","starred_url":"https://api.github.com/users/dcommander/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dcommander/subscriptions","organizations_url":"https://api.github.com/users/dcommander/orgs","repos_url":"https://api.github.com/users/dcommander/repos","events_url":"https://api.github.com/users/dcommander/events{/privacy}","received_events_url":"https://api.github.com/users/dcommander/received_events","type":"User","site_admin":false},"created_at":"2016-12-04T18:14:53Z","updated_at":"2016-12-04T18:14:53Z","body":"I'm going to need to re-implement this, because you ripped out some intentional code and some of the build documentation, and several other things are broken.  Stand by.\r\n"}}
{"actor":{"display_login":"dcommander"}}