{"action":"created","issue":{"url":"https://api.github.com/repos/pantsbuild/pants/issues/6250","repository_url":"https://api.github.com/repos/pantsbuild/pants","labels_url":"https://api.github.com/repos/pantsbuild/pants/issues/6250/labels{/name}","comments_url":"https://api.github.com/repos/pantsbuild/pants/issues/6250/comments","events_url":"https://api.github.com/repos/pantsbuild/pants/issues/6250/events","html_url":"https://github.com/pantsbuild/pants/pull/6250","id":345329597,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA0NTExMjk0","number":6250,"title":"[WIP] Override interpreter constraints if global option is passed down","user":{"login":"alanbato","id":11902612,"node_id":"MDQ6VXNlcjExOTAyNjEy","avatar_url":"https://avatars3.githubusercontent.com/u/11902612?v=4","gravatar_id":"","url":"https://api.github.com/users/alanbato","html_url":"https://github.com/alanbato","followers_url":"https://api.github.com/users/alanbato/followers","following_url":"https://api.github.com/users/alanbato/following{/other_user}","gists_url":"https://api.github.com/users/alanbato/gists{/gist_id}","starred_url":"https://api.github.com/users/alanbato/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alanbato/subscriptions","organizations_url":"https://api.github.com/users/alanbato/orgs","repos_url":"https://api.github.com/users/alanbato/repos","events_url":"https://api.github.com/users/alanbato/events{/privacy}","received_events_url":"https://api.github.com/users/alanbato/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-07-27T18:48:25Z","updated_at":"2018-07-31T20:18:03Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/pantsbuild/pants/pulls/6250","html_url":"https://github.com/pantsbuild/pants/pull/6250","diff_url":"https://github.com/pantsbuild/pants/pull/6250.diff","patch_url":"https://github.com/pantsbuild/pants/pull/6250.patch"},"body":"### Problem\r\nThe original issue is documented at #6081, an attempt to solve this was #6234, but we've decided to take another path instead.\r\n\r\n### Solution\r\nInstead of relying on interpreter constraints when building a pex in the repl task, we're checking if the `--python-setup-interpreter-constraints` is flagged. If so, it overrides the interpreter filters at interpreter selection time.\r\n\r\n### Result\r\nUsers should now be able to further filter the python interpreter to use even if the task does not declare an option to do so (like in the repl task)"},"comment":{"url":"https://api.github.com/repos/pantsbuild/pants/issues/comments/409353851","html_url":"https://github.com/pantsbuild/pants/pull/6250#issuecomment-409353851","issue_url":"https://api.github.com/repos/pantsbuild/pants/issues/6250","id":409353851,"node_id":"MDEyOklzc3VlQ29tbWVudDQwOTM1Mzg1MQ==","user":{"login":"stuhood","id":46740,"node_id":"MDQ6VXNlcjQ2NzQw","avatar_url":"https://avatars1.githubusercontent.com/u/46740?v=4","gravatar_id":"","url":"https://api.github.com/users/stuhood","html_url":"https://github.com/stuhood","followers_url":"https://api.github.com/users/stuhood/followers","following_url":"https://api.github.com/users/stuhood/following{/other_user}","gists_url":"https://api.github.com/users/stuhood/gists{/gist_id}","starred_url":"https://api.github.com/users/stuhood/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stuhood/subscriptions","organizations_url":"https://api.github.com/users/stuhood/orgs","repos_url":"https://api.github.com/users/stuhood/repos","events_url":"https://api.github.com/users/stuhood/events{/privacy}","received_events_url":"https://api.github.com/users/stuhood/received_events","type":"User","site_admin":false},"created_at":"2018-07-31T20:18:03Z","updated_at":"2018-07-31T20:18:03Z","author_association":"MEMBER","body":"I like the idea behind this patch. There is a small amount of prior art that could be lifted up to make this more natural: see https://github.com/pantsbuild/pants/blob/master/src/python/pants/build_graph/bundle_mixin.py#L39-L56\r\n\r\nThat logic slots a target's field value in \"between\" the value from the command line and the value from `pants.ini`.\r\n\r\n----\r\n\r\nBut I think you'll need to update a few more locations, as I discovered in #6284 while trying to do something similar. Sorry, I didn't realize until afterward that this patch is relevant."}}
{"actor":{"display_login":"stuhood"}}