Re: [configurations] 2.9.0 throws NoSuchMethodException on JDK 1.8

2023-06-29 Thread Tres Finocchiaro
Personally, I use an explicit cast, which works on both JDK8 and JDK11: https://github.com/qzind/tray/commit/263d7982b7f8d5967385d8c87dcf4cf49ef78540

Re: [configurations] 2.9.0 throws NoSuchMethodException on JDK 1.8

2023-06-29 Thread Tres Finocchiaro
The buffer flip is a JDK regression. Please see here: https://stackoverflow.com/a/61267496/3196753 On Thu, Jun 29, 2023 at 12:13 PM Gary Gregory wrote: > Hello, > > Commons Configuration is tested with Java 8, 11 and 17: > >

Re: [text] How do I update the CommonsText to the 1.10 version in Windows Server 2016

2022-11-16 Thread Tres Finocchiaro
Hi, CommonsText is likely part of another application. Instructions for replacing this file should be obtained by the application developer to ensure it's compatible. The instructions would depend on the application. In most cases, the exact file you will be replacing will be called

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-29 Thread Tres Finocchiaro
https://github.com/apache/commons-io/pull/377

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-29 Thread Tres Finocchiaro
one now, that's > fine too. > > Gary > > On Sun, Aug 28, 2022, 12:21 Tres Finocchiaro > wrote: > > > > > > > Please create a PR on GitHub so we can see build results in all the OSs > > and > > > Java versions we test there. > > > &

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-28 Thread Tres Finocchiaro
> > Please create a PR on GitHub so we can see build results in all the OSs and > Java versions we test there. > Sure thing. The instructions say to reference a JIRA ticket number in the subject line. Should I create a formal bug report first or is a standalone PR sufficient? >

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-25 Thread Tres Finocchiaro
feedback is welcome. If the granular approach of "setTimes()" hepler (setting creation + modified + access times) is too much of a change, the function can be reverted or simplified. - tres.finocchi...@gmail.com On Mon, Aug 22, 2022 at 11:34 AM Tres Finocchiaro < tres.finocchi

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-22 Thread Tres Finocchiaro
> > > what I'd expect as a new user of Commons IO I think it's safe to say, at least on Windows, the default OS behavior is what 99% of people will want default in a copy utility; copying a file to a destination nearly always warrants the destination permissions, especially when the ACLs for

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-21 Thread Tres Finocchiaro
Unix results have near identical regressions using the stackoverflow snippet, but for "/opt/MyApp" (Linux) and "/Applications/MyApp.app" (Mac) *Ubuntu 22.04:* 2.8 and lower: - Files copied from $HOME/Desktop to /opt/MyApp *inherit target

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-21 Thread Tres Finocchiaro
I've pinpointed when this regression occurred, it was between 2.8.0 and 2.9.0. - (GOOD) 2.4, 2.5, 2.6, 2.7 and 2.8 properly inherit the destination permissions. - (BAD) 2.8, 2.9, 2.10 and 2.11 do not inherit the destination permissions. Scope of this issue: - *2005-2021 (16

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-20 Thread Tres Finocchiaro
y as input to PathUtils.copyDirectory(Path, > Path, CopyOption...) ? > > Gary > > On Sat, Aug 20, 2022 at 3:17 PM Tres Finocchiaro < > tres.finocchi...@gmail.com> > wrote: > > > Gary, > > > > Thanks again for your reply however COPY_ATTRIBUTES is th

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-20 Thread Tres Finocchiaro
I've answered my own question on stackoverflow. I'm curious how the community feels about filing this as a bug. It's pretty big regression for those upgrading from 2.4 with Windows target deployments. - tres.finocchi...@gmail.com On Sat, Aug 20, 2022 at 3:16 PM Tres Finocchiaro wrote

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-20 Thread Tres Finocchiaro
Gary, Thanks again for your reply however COPY_ATTRIBUTES is the source of the problem, it's being added and wiping out the permissions. I've linked my suspicion as to why in my previous email. For now, I will toggle-off the preserveFileDate, which seems to be the only workaround for this

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-20 Thread Tres Finocchiaro
> > Under the covers it ends up calling *java.nio.file.Files.copy(Path, Path, > CopyOption...)*. Sure, but from what I'm reading, it's adding CopyOptions that no one asked for and is not in alignment with how NIO works, so this statement is a bit misleading. I think this is the problematic

Re: [io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-20 Thread Tres Finocchiaro
> > Use *org.apache.commons.io.FileUtils.copyFile(File, File, boolean, > CopyOption...)* to exercise full control over what you want to do. > Thanks for the reply. >From my understanding, CopyOption supports one or more of three options: REPLACE_EXISTING, COPY_ATTRIBUTES, NOFOLLOW_LINKS, none of

[io] Regression in FileUtils.copyDirectory() for Windows?

2022-08-20 Thread Tres Finocchiaro
Hi, I'm updating some dependent libraries and hit a snag with commons-io. I was hoping someone here could help. I've noticed a major change in behavior with FileUtils.copyDirectory() between commons-io@2.4 and commons-io@2.11 that affects the permission of files being copied. I've posted the