{"action":"created","issue":{"url":"https://api.github.com/repos/JuliaLang/julia/issues/10888","labels_url":"https://api.github.com/repos/JuliaLang/julia/issues/10888/labels{/name}","comments_url":"https://api.github.com/repos/JuliaLang/julia/issues/10888/comments","events_url":"https://api.github.com/repos/JuliaLang/julia/issues/10888/events","html_url":"https://github.com/JuliaLang/julia/pull/10888","id":69362684,"number":10888,"title":"new__cp_function","user":{"login":"peter1000","id":6168860,"avatar_url":"https://avatars.githubusercontent.com/u/6168860?v=3","gravatar_id":"","url":"https://api.github.com/users/peter1000","html_url":"https://github.com/peter1000","followers_url":"https://api.github.com/users/peter1000/followers","following_url":"https://api.github.com/users/peter1000/following{/other_user}","gists_url":"https://api.github.com/users/peter1000/gists{/gist_id}","starred_url":"https://api.github.com/users/peter1000/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peter1000/subscriptions","organizations_url":"https://api.github.com/users/peter1000/orgs","repos_url":"https://api.github.com/users/peter1000/repos","events_url":"https://api.github.com/users/peter1000/events{/privacy}","received_events_url":"https://api.github.com/users/peter1000/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/JuliaLang/julia/labels/decision","name":"decision","color":"F08000"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":19,"created_at":"2015-04-19T00:11:52Z","updated_at":"2015-04-23T20:17:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/JuliaLang/julia/pulls/10888","html_url":"https://github.com/JuliaLang/julia/pull/10888","diff_url":"https://github.com/JuliaLang/julia/pull/10888.diff","patch_url":"https://github.com/JuliaLang/julia/pull/10888.patch"},"body":"hi @tkelman \r\n\r\nThis is what I came up with to address issue: #10506\r\n\r\nI took also a look at the older conversations at: \r\n\r\n* RFC: Recursive cp #10448\r\n* cp` does not work for directories #10434\r\n\r\n  * @jiahao noted that: For comparison, Python has `shutil.copy()` and `shutil.copytree()`.\r\n  * @simonster voted for: I'd vote for a `recursive` kwarg to match `rm`.\r\n\r\n.\r\n\r\nMy own opinion differs from @simonster and I would prefer to remove the `recursive` kwarg altogehter and just copy directories always recursively or export `cptree`.\r\n\r\nAnyway: I left it as it was: `cp` requires the `recursive=true` to be set to copy directories  and helper function `cptree` is not exported.\r\n\r\nTo have an option to keep symlinks as symlinks I added a kwarg `keepsym=false`.\r\n\r\n* Python uses: If `follow_symlinks` is false and src is a symbolic link, a new symbolic \r\nlink will be created instead of copying the file src points to. [shutil.copy](https://docs.python.org/3.4/library/shutil.html#shutil.copy)\r\n\r\nOn Linux (x86_64) all tests Succeeded\r\n\r\nThanks for your help.\r\n\r\n\r\nPS: \r\n- currently files can be copied over existing files\r\n- but: directories can not be copied over existing directories\r\n\r\nI kept it like that.\r\n\r\n**Question:** should we add an option: `force` to overwrite also directories.\r\nPython seems not to allow such: [shutil.copytree](https://docs.python.org/3.4/library/shutil.html#shutil.copytree)\r\nbut that does not mean `julia can't have it`.\r\nIf so - should that also apply to files or leave file to be overwritten anyway"},"comment":{"url":"https://api.github.com/repos/JuliaLang/julia/issues/comments/95707338","html_url":"https://github.com/JuliaLang/julia/pull/10888#issuecomment-95707338","issue_url":"https://api.github.com/repos/JuliaLang/julia/issues/10888","id":95707338,"user":{"login":"peter1000","id":6168860,"avatar_url":"https://avatars.githubusercontent.com/u/6168860?v=3","gravatar_id":"","url":"https://api.github.com/users/peter1000","html_url":"https://github.com/peter1000","followers_url":"https://api.github.com/users/peter1000/followers","following_url":"https://api.github.com/users/peter1000/following{/other_user}","gists_url":"https://api.github.com/users/peter1000/gists{/gist_id}","starred_url":"https://api.github.com/users/peter1000/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peter1000/subscriptions","organizations_url":"https://api.github.com/users/peter1000/orgs","repos_url":"https://api.github.com/users/peter1000/repos","events_url":"https://api.github.com/users/peter1000/events{/privacy}","received_events_url":"https://api.github.com/users/peter1000/received_events","type":"User","site_admin":false},"created_at":"2015-04-23T20:17:31Z","updated_at":"2015-04-23T20:17:31Z","body":"@tkelman should I go ahead and rename them to `rmdst => remove_destination` and `sym => follow_symlinks` and update the docs?"}}