{"action":"created","issue":{"url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/2423","repository_url":"https://api.github.com/repos/ansible/ansible-modules-core","labels_url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/2423/labels{/name}","comments_url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/2423/comments","events_url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/2423/events","html_url":"https://github.com/ansible/ansible-modules-core/issues/2423","id":115483426,"number":2423,"title":"Unarchive module throws an error on OS X","user":{"login":"anthonygreen","id":1336742,"avatar_url":"https://avatars.githubusercontent.com/u/1336742?v=3","gravatar_id":"","url":"https://api.github.com/users/anthonygreen","html_url":"https://github.com/anthonygreen","followers_url":"https://api.github.com/users/anthonygreen/followers","following_url":"https://api.github.com/users/anthonygreen/following{/other_user}","gists_url":"https://api.github.com/users/anthonygreen/gists{/gist_id}","starred_url":"https://api.github.com/users/anthonygreen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anthonygreen/subscriptions","organizations_url":"https://api.github.com/users/anthonygreen/orgs","repos_url":"https://api.github.com/users/anthonygreen/repos","events_url":"https://api.github.com/users/anthonygreen/events{/privacy}","received_events_url":"https://api.github.com/users/anthonygreen/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ansible/ansible-modules-core/labels/bug_report","name":"bug_report","color":"fbca04"},{"url":"https://api.github.com/repos/ansible/ansible-modules-core/labels/needs_info","name":"needs_info","color":"eb6420"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":18,"created_at":"2015-11-06T10:58:35Z","updated_at":"2016-09-10T13:32:37Z","closed_at":null,"body":"#### Issue Type:\r\nBug Report\r\n\r\n#### Ansible Version:\r\n2.0.0-0.4.beta2\r\n\r\n### Ansible Configuration\r\n\r\n#### Environment\r\nRunning from: OS X 10.9.5\r\nManaging: OSX VM 10.10\r\n\r\n#### Summary:\r\n\r\nunarchive_module throws an error on OS X\r\n\r\n#### Steps to reproduce\r\n\r\nThis task fails\r\n\r\n    - name: Download a file\r\n      unarchive: \r\n        src: https://www.dropbox.com/path.to.some/file?raw=1\r\n        dest: /tmp\r\n        copy: no\r\n        creates: /tmp/file\r\n\r\nresults in the error\r\n\r\n    No such file or directory: '/tmp/__MACOSX/._\r\n\r\nThe file has been downloaded to /tmp and appears to unzipped but has thrown an error. \r\n"},"comment":{"url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/comments/246111985","html_url":"https://github.com/ansible/ansible-modules-core/issues/2423#issuecomment-246111985","issue_url":"https://api.github.com/repos/ansible/ansible-modules-core/issues/2423","id":246111985,"user":{"login":"eirnym","id":485399,"avatar_url":"https://avatars.githubusercontent.com/u/485399?v=3","gravatar_id":"","url":"https://api.github.com/users/eirnym","html_url":"https://github.com/eirnym","followers_url":"https://api.github.com/users/eirnym/followers","following_url":"https://api.github.com/users/eirnym/following{/other_user}","gists_url":"https://api.github.com/users/eirnym/gists{/gist_id}","starred_url":"https://api.github.com/users/eirnym/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eirnym/subscriptions","organizations_url":"https://api.github.com/users/eirnym/orgs","repos_url":"https://api.github.com/users/eirnym/repos","events_url":"https://api.github.com/users/eirnym/events{/privacy}","received_events_url":"https://api.github.com/users/eirnym/received_events","type":"User","site_admin":false},"created_at":"2016-09-10T13:32:37Z","updated_at":"2016-09-10T13:32:37Z","body":"I see your point. But I want to tell some disadvantages of this way:\r\n\r\n* You can enforce dependency on provisioning machine, but you can't enforce dependency on target.\r\n* DevOps will miss filesystem-specific flags like what exists in FreeBSD and OS X.\r\n* Practically you use only two GNU-specific options: `--diff` and `--show-transformed-names` which are implementable\r\n* I don't know why do you really need exact same output here. Almost all produced by Ansible Python scripts will throw this output away, other will know the output difference we had\r\n\r\nWhat can be done to make all implementations possible (Pure-Python, BSD and GNU ways):\r\n* With better interface and class organisation we can implement both with same functionality and output. We can wrap tar command itself, separate options and switches then call implementation-specific call.\r\n* We definitely need to refactor this module (too long and too complex) including method separation. I don't think you cover 60% with tests.\r\n* We need to write tests to be sure we are on same page.\r\n\r\nI understand, that way I suggest will take some more time than say \"we don't support\". But implementation will be more strait forward than rejection."}}
{"actor":{"display_login":"eirnym"}}