-
Notifications
You must be signed in to change notification settings - Fork 43
Force String.toLowerCase() to use Locale.ENGLISH #40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Fix Turkish language, possibly others
|
I adapted See also the https://github.com/scijava/native-lib-loader/tree/test-issue-39 branch for the failing test on top of As the issue seems specific to Windows, the test passes e.g. on Mac OSX no matter if the fix is included or not, but it definitely solves the problem on Windows. So this PR also practically closes #39 🙂 Looks good to be merged for me 👍 (/cc @ctrueden) |
Shaquu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great addition! Never imagined such a bug.
|
Friendly bump! What's needed to get this merged? |
|
Hi again. The JSSC project will be releasing a new version very soon, it would be very nice to have this dependency resolved so that we do not have to build a one-off version (java-native/jssc#80) or remove this dependency altogether. |
|
What do you mean by "bump it in Maven"? @hinerm It seems version |
Thanks. The project which originally exposed this bug uses the maven central mirror. |
|
Weird... according to the POM configuration, the 2.3.6 release should have gone to OSS Sonatype and therefore be available from Maven Central. But indeed, it did not, and was deployed to the Releases repository of maven.scijava.org instead. I'll see if I can figure out why this failed to deploy as desired. |
|
I figured out what went wrong, and fixed some problems:
Unfortunately, I still don't have GPG signing of releases working on GitHub Actions. The error I'm troubleshooting now is |
Hi, friendly bump. Maven Central is still showing 2.3.5: https://mvnrepository.com/artifact/org.scijava/native-lib-loader. 🍻 Edit: I've added the following to my <repositories>
<repository>
<id>maven.scijava.org</id>
<name>SciJava</name>
<url>https://maven.scijava.org/content/groups/public/</url>
</repository>
</repositories> |
|
@tresf I finally got native-lib-loader 2.3.6 deployed to Maven Central, where it should be. Sorry for the delay on that. We just completed a migration from Travis CI to GitHub Actions, and there were some kinks to iron out. |
This is untested, but I believe it will fix the issues described in #39.
Theoretically, closes #39