Isira-Seneviratne commented on pull request #124: URL: https://github.com/apache/commons-io/pull/124#issuecomment-669928664
> > > > > @Isira-Seneviratne @chtompki > > > Here is another thought which I think will give us more bang for the buck: > > > All of these `FileUtils` time comparison APIs should convert their input `File` object to a`Path` with `toPath()` and forward the calls to `PathUtils` which opens the feature to any backing NIO file system, not old school `File` objects. > > > > > > That sounds like a good idea. > > However, there's a potential issue in that those methods would become unusable on most versions of Android, since the `java.nio.file` package is only supported on Android Oreo and higher (admittedly, I'm not sure if any Android apps will need to make use of those methods, and it's possible that the package [will be made available](https://issuetracker.google.com/issues/154309654) to all versions of Android via [desugaring](https://developer.android.com/studio/write/java8-support-table)). > > Other than that, sounds good to me :) > > Hi @Isira-Seneviratne > > After playing around a bit, I think I will proceed with this PR as is and leave a refactoring for later. Great! > > TBH, I had not considered Android. Would you have time to research if there is a way to build and test for Android on GitHub Actions and/or Travis-CI? Sure :) On that note, I've made use of some of the methods in some apps I've contributed to, and they work great: https://github.com/BOINC/boinc/tree/master/android/BOINC https://invent.kde.org/network/kdeconnect-android ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
