{"action":"created","comment":{"url":"https://api.github.com/repos/fxia22/HINGE/pulls/comments/84184174","pull_request_review_id":4977332,"id":84184174,"diff_hunk":"@@ -1,3 +1,7 @@\n cmake_minimum_required(VERSION 3.2)\n project(AwAssembler)\n-add_subdirectory(src bin)\n\\ No newline at end of file\n+\n+set(libexec \"lib/hinge\")\n+add_subdirectory(src bin)\n+\n+install(DIRECTORY scripts/ DESTINATION ${libexec})","path":"CMakeLists.txt","position":9,"original_position":9,"commit_id":"327ccf43b6d2b2d84f01036eb58dc10a3c174af0","original_commit_id":"327ccf43b6d2b2d84f01036eb58dc10a3c174af0","user":{"login":"afif-elghraoui","id":3968485,"avatar_url":"https://avatars.githubusercontent.com/u/3968485?v=3","gravatar_id":"","url":"https://api.github.com/users/afif-elghraoui","html_url":"https://github.com/afif-elghraoui","followers_url":"https://api.github.com/users/afif-elghraoui/followers","following_url":"https://api.github.com/users/afif-elghraoui/following{/other_user}","gists_url":"https://api.github.com/users/afif-elghraoui/gists{/gist_id}","starred_url":"https://api.github.com/users/afif-elghraoui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afif-elghraoui/subscriptions","organizations_url":"https://api.github.com/users/afif-elghraoui/orgs","repos_url":"https://api.github.com/users/afif-elghraoui/repos","events_url":"https://api.github.com/users/afif-elghraoui/events{/privacy}","received_events_url":"https://api.github.com/users/afif-elghraoui/received_events","type":"User","site_admin":false},"body":"I should have added the argument `USE_SOURCE_PERMISSIONS` here so that the executables from `scripts/` can keep their executable bit when getting installed. See http://stackoverflow.com/a/13623977","created_at":"2016-10-19T22:49:22Z","updated_at":"2016-10-19T22:49:23Z","html_url":"https://github.com/fxia22/HINGE/pull/78#discussion_r84184174","pull_request_url":"https://api.github.com/repos/fxia22/HINGE/pulls/78","_links":{"self":{"href":"https://api.github.com/repos/fxia22/HINGE/pulls/comments/84184174"},"html":{"href":"https://github.com/fxia22/HINGE/pull/78#discussion_r84184174"},"pull_request":{"href":"https://api.github.com/repos/fxia22/HINGE/pulls/78"}}},"pull_request":{"url":"https://api.github.com/repos/fxia22/HINGE/pulls/78","id":88566352,"html_url":"https://github.com/fxia22/HINGE/pull/78","diff_url":"https://github.com/fxia22/HINGE/pull/78.diff","patch_url":"https://github.com/fxia22/HINGE/pull/78.patch","issue_url":"https://api.github.com/repos/fxia22/HINGE/issues/78","number":78,"state":"closed","locked":false,"title":"Reorganize executables to be subcommands of a single program","user":{"login":"afif-elghraoui","id":3968485,"avatar_url":"https://avatars.githubusercontent.com/u/3968485?v=3","gravatar_id":"","url":"https://api.github.com/users/afif-elghraoui","html_url":"https://github.com/afif-elghraoui","followers_url":"https://api.github.com/users/afif-elghraoui/followers","following_url":"https://api.github.com/users/afif-elghraoui/following{/other_user}","gists_url":"https://api.github.com/users/afif-elghraoui/gists{/gist_id}","starred_url":"https://api.github.com/users/afif-elghraoui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afif-elghraoui/subscriptions","organizations_url":"https://api.github.com/users/afif-elghraoui/orgs","repos_url":"https://api.github.com/users/afif-elghraoui/repos","events_url":"https://api.github.com/users/afif-elghraoui/events{/privacy}","received_events_url":"https://api.github.com/users/afif-elghraoui/received_events","type":"User","site_admin":false},"body":"I was working on Debian packaging for HINGE, but since there are many diverse commands with generic names, I thought it would be helpful to have a master executable `hinge`, where you can call things like `hinge filter` and so forth. It needs just a bit more work, which I can't do without better understanding of the internal components. I noticed the examples have slightly different steps and it's not clear to me what some of the commands do or whether all of them should be wrapped by the `hinge` master program.\r\n\r\nI'd appreciate your consideration of these changes and if you could help finalize it. I'm definitely open to suggestions as well.","created_at":"2016-10-09T02:50:04Z","updated_at":"2016-10-19T22:49:23Z","closed_at":"2016-10-10T17:02:54Z","merged_at":"2016-10-10T17:02:54Z","merge_commit_sha":"536e6564fafb694d2d7666375e5e89c409679a8e","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/fxia22/HINGE/pulls/78/commits","review_comments_url":"https://api.github.com/repos/fxia22/HINGE/pulls/78/comments","review_comment_url":"https://api.github.com/repos/fxia22/HINGE/pulls/comments{/number}","comments_url":"https://api.github.com/repos/fxia22/HINGE/issues/78/comments","statuses_url":"https://api.github.com/repos/fxia22/HINGE/statuses/327ccf43b6d2b2d84f01036eb58dc10a3c174af0","head":{"label":"Debian:topic/command-reorganization","ref":"topic/command-reorganization","sha":"327ccf43b6d2b2d84f01036eb58dc10a3c174af0","user":{"login":"Debian","id":1854028,"avatar_url":"https://avatars.githubusercontent.com/u/1854028?v=3","gravatar_id":"","url":"https://api.github.com/users/Debian","html_url":"https://github.com/Debian","followers_url":"https://api.github.com/users/Debian/followers","following_url":"https://api.github.com/users/Debian/following{/other_user}","gists_url":"https://api.github.com/users/Debian/gists{/gist_id}","starred_url":"https://api.github.com/users/Debian/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Debian/subscriptions","organizations_url":"https://api.github.com/users/Debian/orgs","repos_url":"https://api.github.com/users/Debian/repos","events_url":"https://api.github.com/users/Debian/events{/privacy}","received_events_url":"https://api.github.com/users/Debian/received_events","type":"Organization","site_admin":false},"repo":{"id":70371336,"name":"HINGE","full_name":"Debian/HINGE","owner":{"login":"Debian","id":1854028,"avatar_url":"https://avatars.githubusercontent.com/u/1854028?v=3","gravatar_id":"","url":"https://api.github.com/users/Debian","html_url":"https://github.com/Debian","followers_url":"https://api.github.com/users/Debian/followers","following_url":"https://api.github.com/users/Debian/following{/other_user}","gists_url":"https://api.github.com/users/Debian/gists{/gist_id}","starred_url":"https://api.github.com/users/Debian/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Debian/subscriptions","organizations_url":"https://api.github.com/users/Debian/orgs","repos_url":"https://api.github.com/users/Debian/repos","events_url":"https://api.github.com/users/Debian/events{/privacy}","received_events_url":"https://api.github.com/users/Debian/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Debian/HINGE","description":"Software accompanying \"HINGE: Long-Read Assembly Achieves Optimal Repeat Resolution\"","fork":true,"url":"https://api.github.com/repos/Debian/HINGE","forks_url":"https://api.github.com/repos/Debian/HINGE/forks","keys_url":"https://api.github.com/repos/Debian/HINGE/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Debian/HINGE/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Debian/HINGE/teams","hooks_url":"https://api.github.com/repos/Debian/HINGE/hooks","issue_events_url":"https://api.github.com/repos/Debian/HINGE/issues/events{/number}","events_url":"https://api.github.com/repos/Debian/HINGE/events","assignees_url":"https://api.github.com/repos/Debian/HINGE/assignees{/user}","branches_url":"https://api.github.com/repos/Debian/HINGE/branches{/branch}","tags_url":"https://api.github.com/repos/Debian/HINGE/tags","blobs_url":"https://api.github.com/repos/Debian/HINGE/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Debian/HINGE/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Debian/HINGE/git/refs{/sha}","trees_url":"https://api.github.com/repos/Debian/HINGE/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Debian/HINGE/statuses/{sha}","languages_url":"https://api.github.com/repos/Debian/HINGE/languages","stargazers_url":"https://api.github.com/repos/Debian/HINGE/stargazers","contributors_url":"https://api.github.com/repos/Debian/HINGE/contributors","subscribers_url":"https://api.github.com/repos/Debian/HINGE/subscribers","subscription_url":"https://api.github.com/repos/Debian/HINGE/subscription","commits_url":"https://api.github.com/repos/Debian/HINGE/commits{/sha}","git_commits_url":"https://api.github.com/repos/Debian/HINGE/git/commits{/sha}","comments_url":"https://api.github.com/repos/Debian/HINGE/comments{/number}","issue_comment_url":"https://api.github.com/repos/Debian/HINGE/issues/comments{/number}","contents_url":"https://api.github.com/repos/Debian/HINGE/contents/{+path}","compare_url":"https://api.github.com/repos/Debian/HINGE/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Debian/HINGE/merges","archive_url":"https://api.github.com/repos/Debian/HINGE/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Debian/HINGE/downloads","issues_url":"https://api.github.com/repos/Debian/HINGE/issues{/number}","pulls_url":"https://api.github.com/repos/Debian/HINGE/pulls{/number}","milestones_url":"https://api.github.com/repos/Debian/HINGE/milestones{/number}","notifications_url":"https://api.github.com/repos/Debian/HINGE/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Debian/HINGE/labels{/name}","releases_url":"https://api.github.com/repos/Debian/HINGE/releases{/id}","deployments_url":"https://api.github.com/repos/Debian/HINGE/deployments","created_at":"2016-10-09T02:40:13Z","updated_at":"2016-10-09T02:40:15Z","pushed_at":"2016-10-09T02:44:03Z","git_url":"git://github.com/Debian/HINGE.git","ssh_url":"git@github.com:Debian/HINGE.git","clone_url":"https://github.com/Debian/HINGE.git","svn_url":"https://github.com/Debian/HINGE","homepage":"http://biorxiv.org/content/early/2016/07/05/062117","size":6043,"stargazers_count":0,"watchers_count":0,"language":"C++","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"fxia22:master","ref":"master","sha":"1cf1ae288a464138e7216d24ae0a2794fcb1c5a5","user":{"login":"fxia22","id":5158896,"avatar_url":"https://avatars.githubusercontent.com/u/5158896?v=3","gravatar_id":"","url":"https://api.github.com/users/fxia22","html_url":"https://github.com/fxia22","followers_url":"https://api.github.com/users/fxia22/followers","following_url":"https://api.github.com/users/fxia22/following{/other_user}","gists_url":"https://api.github.com/users/fxia22/gists{/gist_id}","starred_url":"https://api.github.com/users/fxia22/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fxia22/subscriptions","organizations_url":"https://api.github.com/users/fxia22/orgs","repos_url":"https://api.github.com/users/fxia22/repos","events_url":"https://api.github.com/users/fxia22/events{/privacy}","received_events_url":"https://api.github.com/users/fxia22/received_events","type":"User","site_admin":false},"repo":{"id":42931250,"name":"HINGE","full_name":"fxia22/HINGE","owner":{"login":"fxia22","id":5158896,"avatar_url":"https://avatars.githubusercontent.com/u/5158896?v=3","gravatar_id":"","url":"https://api.github.com/users/fxia22","html_url":"https://github.com/fxia22","followers_url":"https://api.github.com/users/fxia22/followers","following_url":"https://api.github.com/users/fxia22/following{/other_user}","gists_url":"https://api.github.com/users/fxia22/gists{/gist_id}","starred_url":"https://api.github.com/users/fxia22/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fxia22/subscriptions","organizations_url":"https://api.github.com/users/fxia22/orgs","repos_url":"https://api.github.com/users/fxia22/repos","events_url":"https://api.github.com/users/fxia22/events{/privacy}","received_events_url":"https://api.github.com/users/fxia22/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/fxia22/HINGE","description":"Software accompanying \"HINGE: Long-Read Assembly Achieves Optimal Repeat Resolution\"","fork":false,"url":"https://api.github.com/repos/fxia22/HINGE","forks_url":"https://api.github.com/repos/fxia22/HINGE/forks","keys_url":"https://api.github.com/repos/fxia22/HINGE/keys{/key_id}","collaborators_url":"https://api.github.com/repos/fxia22/HINGE/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/fxia22/HINGE/teams","hooks_url":"https://api.github.com/repos/fxia22/HINGE/hooks","issue_events_url":"https://api.github.com/repos/fxia22/HINGE/issues/events{/number}","events_url":"https://api.github.com/repos/fxia22/HINGE/events","assignees_url":"https://api.github.com/repos/fxia22/HINGE/assignees{/user}","branches_url":"https://api.github.com/repos/fxia22/HINGE/branches{/branch}","tags_url":"https://api.github.com/repos/fxia22/HINGE/tags","blobs_url":"https://api.github.com/repos/fxia22/HINGE/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/fxia22/HINGE/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/fxia22/HINGE/git/refs{/sha}","trees_url":"https://api.github.com/repos/fxia22/HINGE/git/trees{/sha}","statuses_url":"https://api.github.com/repos/fxia22/HINGE/statuses/{sha}","languages_url":"https://api.github.com/repos/fxia22/HINGE/languages","stargazers_url":"https://api.github.com/repos/fxia22/HINGE/stargazers","contributors_url":"https://api.github.com/repos/fxia22/HINGE/contributors","subscribers_url":"https://api.github.com/repos/fxia22/HINGE/subscribers","subscription_url":"https://api.github.com/repos/fxia22/HINGE/subscription","commits_url":"https://api.github.com/repos/fxia22/HINGE/commits{/sha}","git_commits_url":"https://api.github.com/repos/fxia22/HINGE/git/commits{/sha}","comments_url":"https://api.github.com/repos/fxia22/HINGE/comments{/number}","issue_comment_url":"https://api.github.com/repos/fxia22/HINGE/issues/comments{/number}","contents_url":"https://api.github.com/repos/fxia22/HINGE/contents/{+path}","compare_url":"https://api.github.com/repos/fxia22/HINGE/compare/{base}...{head}","merges_url":"https://api.github.com/repos/fxia22/HINGE/merges","archive_url":"https://api.github.com/repos/fxia22/HINGE/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/fxia22/HINGE/downloads","issues_url":"https://api.github.com/repos/fxia22/HINGE/issues{/number}","pulls_url":"https://api.github.com/repos/fxia22/HINGE/pulls{/number}","milestones_url":"https://api.github.com/repos/fxia22/HINGE/milestones{/number}","notifications_url":"https://api.github.com/repos/fxia22/HINGE/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/fxia22/HINGE/labels{/name}","releases_url":"https://api.github.com/repos/fxia22/HINGE/releases{/id}","deployments_url":"https://api.github.com/repos/fxia22/HINGE/deployments","created_at":"2015-09-22T12:04:32Z","updated_at":"2016-10-06T13:48:12Z","pushed_at":"2016-10-14T23:15:13Z","git_url":"git://github.com/fxia22/HINGE.git","ssh_url":"git@github.com:fxia22/HINGE.git","clone_url":"https://github.com/fxia22/HINGE.git","svn_url":"https://github.com/fxia22/HINGE","homepage":"http://biorxiv.org/content/early/2016/07/05/062117","size":6049,"stargazers_count":17,"watchers_count":17,"language":"C++","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":2,"mirror_url":null,"open_issues_count":4,"forks":2,"open_issues":4,"watchers":17,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/fxia22/HINGE/pulls/78"},"html":{"href":"https://github.com/fxia22/HINGE/pull/78"},"issue":{"href":"https://api.github.com/repos/fxia22/HINGE/issues/78"},"comments":{"href":"https://api.github.com/repos/fxia22/HINGE/issues/78/comments"},"review_comments":{"href":"https://api.github.com/repos/fxia22/HINGE/pulls/78/comments"},"review_comment":{"href":"https://api.github.com/repos/fxia22/HINGE/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/fxia22/HINGE/pulls/78/commits"},"statuses":{"href":"https://api.github.com/repos/fxia22/HINGE/statuses/327ccf43b6d2b2d84f01036eb58dc10a3c174af0"}}}}
{"actor":{"display_login":"afif-elghraoui"}}