eu-emi / canl-java

Common authentication library (caNl), Java version
Other
14 stars 5 forks source link

Adapt to Apache Commons-IO 2.9 and later #112

Closed ellert closed 2 years ago

ellert commented 2 years ago

Fix for #111.

golbi commented 2 years ago

@ellert I've looked into that issue. So the proposed patch is not needed, canl works fine over newer commons-io. You problem is that commons-io now requires at least Java 8. Canl still Java 7. I think at this point (Java 8 is already EOL) it is safe to bump the required JDK to 8 (and the dependency to commons-io). No changes in code should be needed.

ellert commented 2 years ago

Thanks. Your suggestion works. I have replaced to patch proposed in this pull request with the following patch in the Debian package:

https://salsa.debian.org/ellert/canl-java/-/blob/master/debian/patches/canl-java-src-tgt-1.8.patch

Closing.