{"action":"created","issue":{"url":"https://api.github.com/repos/open-mpi/ompi/issues/636","labels_url":"https://api.github.com/repos/open-mpi/ompi/issues/636/labels{/name}","comments_url":"https://api.github.com/repos/open-mpi/ompi/issues/636/comments","events_url":"https://api.github.com/repos/open-mpi/ompi/issues/636/events","html_url":"https://github.com/open-mpi/ompi/pull/636","id":87122773,"number":636,"title":"Remove build time sym links","user":{"login":"jsquyres","id":617316,"avatar_url":"https://avatars.githubusercontent.com/u/617316?v=3","gravatar_id":"","url":"https://api.github.com/users/jsquyres","html_url":"https://github.com/jsquyres","followers_url":"https://api.github.com/users/jsquyres/followers","following_url":"https://api.github.com/users/jsquyres/following{/other_user}","gists_url":"https://api.github.com/users/jsquyres/gists{/gist_id}","starred_url":"https://api.github.com/users/jsquyres/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jsquyres/subscriptions","organizations_url":"https://api.github.com/users/jsquyres/orgs","repos_url":"https://api.github.com/users/jsquyres/repos","events_url":"https://api.github.com/users/jsquyres/events{/privacy}","received_events_url":"https://api.github.com/users/jsquyres/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-06-10T22:02:42Z","updated_at":"2015-06-10T22:45:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/open-mpi/ompi/pulls/636","html_url":"https://github.com/open-mpi/ompi/pull/636","diff_url":"https://github.com/open-mpi/ompi/pull/636.diff","patch_url":"https://github.com/open-mpi/ompi/pull/636.patch"},"body":"One reason that we make compile-time sym links is because waaaaay back in olden days, SVN couldn't track sym links.  So we made sym links at compile time and avoided the problem.\r\n\r\nPer @siegmargross' usage, however, perhaps we should not make compile-time sym links any more.  Instead, just:\r\n\r\n* git commit sym links to one directory back\r\n* remove the compile-time logic to make sym links\r\n* update .gitignore\r\n\r\nThe \"make dist\" tarballs will actually contain *copies* of files (not sym links), but that's ok -- the sym link is actually useful to *developers*, not *end users*."},"comment":{"url":"https://api.github.com/repos/open-mpi/ompi/issues/comments/110939464","html_url":"https://github.com/open-mpi/ompi/pull/636#issuecomment-110939464","issue_url":"https://api.github.com/repos/open-mpi/ompi/issues/636","id":110939464,"user":{"login":"mellanox-github","id":9459943,"avatar_url":"https://avatars.githubusercontent.com/u/9459943?v=3","gravatar_id":"","url":"https://api.github.com/users/mellanox-github","html_url":"https://github.com/mellanox-github","followers_url":"https://api.github.com/users/mellanox-github/followers","following_url":"https://api.github.com/users/mellanox-github/following{/other_user}","gists_url":"https://api.github.com/users/mellanox-github/gists{/gist_id}","starred_url":"https://api.github.com/users/mellanox-github/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mellanox-github/subscriptions","organizations_url":"https://api.github.com/users/mellanox-github/orgs","repos_url":"https://api.github.com/users/mellanox-github/repos","events_url":"https://api.github.com/users/mellanox-github/events{/privacy}","received_events_url":"https://api.github.com/users/mellanox-github/received_events","type":"User","site_admin":false},"created_at":"2015-06-10T22:45:57Z","updated_at":"2015-06-10T22:45:57Z","body":"\nRefer to this link for build results (access rights to CI server needed): \nhttp://bgate.mellanox.com/job/gh-ompi-master-pr/620/\n"}}