{"action":"created","issue":{"url":"https://api.github.com/repos/openwrt/packages/issues/2729","repository_url":"https://api.github.com/repos/openwrt/packages","labels_url":"https://api.github.com/repos/openwrt/packages/issues/2729/labels{/name}","comments_url":"https://api.github.com/repos/openwrt/packages/issues/2729/comments","events_url":"https://api.github.com/repos/openwrt/packages/issues/2729/events","html_url":"https://github.com/openwrt/packages/pull/2729","id":156200001,"number":2729,"title":"[URFC] net/wget: Remove use broken use of postrm/postinst symlinks","user":{"login":"dfc-d","id":5790165,"avatar_url":"https://avatars.githubusercontent.com/u/5790165?v=3","gravatar_id":"","url":"https://api.github.com/users/dfc-d","html_url":"https://github.com/dfc-d","followers_url":"https://api.github.com/users/dfc-d/followers","following_url":"https://api.github.com/users/dfc-d/following{/other_user}","gists_url":"https://api.github.com/users/dfc-d/gists{/gist_id}","starred_url":"https://api.github.com/users/dfc-d/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dfc-d/subscriptions","organizations_url":"https://api.github.com/users/dfc-d/orgs","repos_url":"https://api.github.com/users/dfc-d/repos","events_url":"https://api.github.com/users/dfc-d/events{/privacy}","received_events_url":"https://api.github.com/users/dfc-d/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":29,"created_at":"2016-05-23T04:58:53Z","updated_at":"2016-06-27T05:38:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openwrt/packages/pulls/2729","html_url":"https://github.com/openwrt/packages/pull/2729","diff_url":"https://github.com/openwrt/packages/pull/2729.diff","patch_url":"https://github.com/openwrt/packages/pull/2729.patch"},"body":"Symlinks in order to use full version of command vs\r\ndefault version are frowned upon by openwrt/lede,\r\nand further results in wget-ssl failing to conflict\r\nwith wget-nossl.  As mentioned in the github issue\r\nregarding this (https://github.com/openwrt/packages/issues/2728)\r\nit is also unnessary in current openwrt/lede.\r\n\r\nThis patch therefore fixes the wget packages to\r\nboth install to /usr/bin/wget where they will\r\ncorrectly conflict with each other and be\r\nearlier in the PATH (and therefore preferred)\r\ncompared to the uclient-fetch symlink in /bin.\r\n\r\nSigned-off-by: Daniel Dickinson <lede@daniel.thecshore.com>\r\n\r\nThis is labelled as an URFC (Untested Request For Comments) because I have not yet tested, but submit the PR in case someone feels the changes are straightforward enough to do their own testing and merge the PR before I get to the testing."},"comment":{"url":"https://api.github.com/repos/openwrt/packages/issues/comments/228658142","html_url":"https://github.com/openwrt/packages/pull/2729#issuecomment-228658142","issue_url":"https://api.github.com/repos/openwrt/packages/issues/2729","id":228658142,"user":{"login":"dfc-d","id":5790165,"avatar_url":"https://avatars.githubusercontent.com/u/5790165?v=3","gravatar_id":"","url":"https://api.github.com/users/dfc-d","html_url":"https://github.com/dfc-d","followers_url":"https://api.github.com/users/dfc-d/followers","following_url":"https://api.github.com/users/dfc-d/following{/other_user}","gists_url":"https://api.github.com/users/dfc-d/gists{/gist_id}","starred_url":"https://api.github.com/users/dfc-d/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dfc-d/subscriptions","organizations_url":"https://api.github.com/users/dfc-d/orgs","repos_url":"https://api.github.com/users/dfc-d/repos","events_url":"https://api.github.com/users/dfc-d/events{/privacy}","received_events_url":"https://api.github.com/users/dfc-d/received_events","type":"User","site_admin":false},"created_at":"2016-06-27T05:38:56Z","updated_at":"2016-06-27T05:38:56Z","body":"On Sun, 2016-06-26 at 01:30 -0700, Christian Schoenebeck wrote:\n> for ddns-scripts I prefer/continue to look for the specific binary.\n> That's the save way.\n> \nSo you're okay with this version then (it allows to use specific\nbinary)?\n\nRegards,\n\nDaniel\n> \n> —\n> You are receiving this because you authored the thread.\n> Reply to this email directly, view it on GitHub, or mute the thread.\n> \n> \n\n\n\n"}}
{"actor":{"display_login":"dfc-d"}}