{"action":"created","issue":{"url":"https://api.github.com/repos/rust-lang-nursery/multirust-rs/issues/211","repository_url":"https://api.github.com/repos/rust-lang-nursery/multirust-rs","labels_url":"https://api.github.com/repos/rust-lang-nursery/multirust-rs/issues/211/labels{/name}","comments_url":"https://api.github.com/repos/rust-lang-nursery/multirust-rs/issues/211/comments","events_url":"https://api.github.com/repos/rust-lang-nursery/multirust-rs/issues/211/events","html_url":"https://github.com/rust-lang-nursery/multirust-rs/issues/211","id":144707815,"number":211,"title":"Maybe `~/.profile` is where PATH modifications should go","user":{"login":"brson","id":147214,"avatar_url":"https://avatars.githubusercontent.com/u/147214?v=3","gravatar_id":"","url":"https://api.github.com/users/brson","html_url":"https://github.com/brson","followers_url":"https://api.github.com/users/brson/followers","following_url":"https://api.github.com/users/brson/following{/other_user}","gists_url":"https://api.github.com/users/brson/gists{/gist_id}","starred_url":"https://api.github.com/users/brson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brson/subscriptions","organizations_url":"https://api.github.com/users/brson/orgs","repos_url":"https://api.github.com/users/brson/repos","events_url":"https://api.github.com/users/brson/events{/privacy}","received_events_url":"https://api.github.com/users/brson/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2016-03-30T20:18:35Z","updated_at":"2016-03-31T18:00:45Z","closed_at":null,"body":"@Diggsey had multirust-rs installing path modifications to `~/.profile` and I changed it because `~/.profile` is only read for login shells, and that seemed wrong since most shells are not login shells. I'm happy to change this to do the right thing, but I'd like to understand better why it's ok not to set the `PATH` for non-login shells.\r\n\r\n[Source](https://internals.rust-lang.org/t/beta-testing-rustup-rs/3316/41?u=brson)."},"comment":{"url":"https://api.github.com/repos/rust-lang-nursery/multirust-rs/issues/comments/204056052","html_url":"https://github.com/rust-lang-nursery/multirust-rs/issues/211#issuecomment-204056052","issue_url":"https://api.github.com/repos/rust-lang-nursery/multirust-rs/issues/211","id":204056052,"user":{"login":"nodakai","id":90726,"avatar_url":"https://avatars.githubusercontent.com/u/90726?v=3","gravatar_id":"","url":"https://api.github.com/users/nodakai","html_url":"https://github.com/nodakai","followers_url":"https://api.github.com/users/nodakai/followers","following_url":"https://api.github.com/users/nodakai/following{/other_user}","gists_url":"https://api.github.com/users/nodakai/gists{/gist_id}","starred_url":"https://api.github.com/users/nodakai/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nodakai/subscriptions","organizations_url":"https://api.github.com/users/nodakai/orgs","repos_url":"https://api.github.com/users/nodakai/repos","events_url":"https://api.github.com/users/nodakai/events{/privacy}","received_events_url":"https://api.github.com/users/nodakai/received_events","type":"User","site_admin":false},"created_at":"2016-03-31T18:00:45Z","updated_at":"2016-03-31T18:00:45Z","body":"I think it is the best practice to have `. ~/.profile` inside `~/.xprofile` assuming either one of them gets executed upon starting a console/GUI session."}}