{"action":"closed","issue":{"url":"https://api.github.com/repos/OpenVPN/easy-rsa/issues/152","repository_url":"https://api.github.com/repos/OpenVPN/easy-rsa","labels_url":"https://api.github.com/repos/OpenVPN/easy-rsa/issues/152/labels{/name}","comments_url":"https://api.github.com/repos/OpenVPN/easy-rsa/issues/152/comments","events_url":"https://api.github.com/repos/OpenVPN/easy-rsa/issues/152/events","html_url":"https://github.com/OpenVPN/easy-rsa/issues/152","id":257099966,"number":152,"title":"breaking change to default values in bugfix releases","user":{"login":"majewsky","id":24696,"avatar_url":"https://avatars0.githubusercontent.com/u/24696?v=4","gravatar_id":"","url":"https://api.github.com/users/majewsky","html_url":"https://github.com/majewsky","followers_url":"https://api.github.com/users/majewsky/followers","following_url":"https://api.github.com/users/majewsky/following{/other_user}","gists_url":"https://api.github.com/users/majewsky/gists{/gist_id}","starred_url":"https://api.github.com/users/majewsky/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/majewsky/subscriptions","organizations_url":"https://api.github.com/users/majewsky/orgs","repos_url":"https://api.github.com/users/majewsky/repos","events_url":"https://api.github.com/users/majewsky/events{/privacy}","received_events_url":"https://api.github.com/users/majewsky/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-09-12T16:07:47Z","updated_at":"2017-12-13T01:38:09Z","closed_at":"2017-12-13T01:38:09Z","author_association":"NONE","body":"In 534f67345997603e3b1fbf6b673bbaa65937019e, the default value for the `EASYRSA` variable was changed from `$PWD` to the directory containing the `easyrsa` binary, usually `/usr/bin`. This breaks a lot of other assumptions in my workflow, for example `$EASYRSA/openssl-1.0.cnf` used to work when `EASYRSA` defaulted to `$PWD` (which is `/etc/easy-rsa` for me). Now it's looking for `/usr/bin/openssl-1.0.cnf` which is nonsensical.\r\n\r\nOf course I can easily work around this by `export EASYRSA=/etc/easy-rsa`, but there's a lot that's unprofessional about what happened:\r\n\r\n1. Changes like this should be announced as backwards-incompatible in the changelog and the release notes.\r\n2. Changes like this should never happen in bugfix releases. Even if you don't follow semver.org, having a breaking change in a bugfix release is extremely surprising and unexpected.\r\n\r\nAs a result of this issue, I'd like:\r\n\r\n1. clear documentation in the changelog about what was changed and why, and\r\n2. optionally, a discussion whether this change should stay in or be reverted."}}
{"actor":{"display_login":"ecrist"}}