[
https://issues.apache.org/jira/browse/IO-678?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17166444#comment-17166444
]
Gary D. Gregory commented on IO-678:
------------------------------------
[~J_aka_J]
Thank you for the update, but this code lives in a package that does not exist
for this component and clashes with the existing
{{org.apache.commons.io.FileUtilsCopyDirectoryToDirectoryTestCase}} when I try
a simple copy/past type of import (same method name as an existing method and
so on). For your issue to be quickly processed, your best choice is a PR on
GitHut at [https://github.com/apache/commons-io]
I do not want to take the time to massage this code this way and that way, this
is why PRs are so great.
Also not providing a PR forces the reviewer to comment on your code here
instead of in-line in GitHub's UI which lets everyone see exactly what a
reviewer is talking about instead of me saying here:
* Don't print to the console
* Don't catch exceptions and log them to the console.
Thank you!
> FileUtils.copyFile does not maintain file permissions
> -----------------------------------------------------
>
> Key: IO-678
> URL: https://issues.apache.org/jira/browse/IO-678
> Project: Commons IO
> Issue Type: Bug
> Components: Utilities
> Affects Versions: 2.7
> Reporter: Jorge
> Priority: Major
> Fix For: 2.8
>
> Attachments: FileUtilsCopyDirectoryToDirectoryTestCase.java
>
>
> I found that permissions (specifically, execute) are not maintained when
> using FileUtils.copyFile. The attached test demonstrates the behavior.
>
> With version 2.6 the following permissions are obtained:
> [OTHERS_READ, OWNER_READ, OWNER_WRITE, GROUP_READ, GROUP_WRITE]
>
> while with version 2.7:
> [OWNER_READ, OWNER_WRITE]
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
