Skip to content
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

fix: path quirk on an M1 Mac using Azul JDK 11 #5006

Merged
merged 2 commits into from
May 3, 2022
Merged

fix: path quirk on an M1 Mac using Azul JDK 11 #5006

merged 2 commits into from
May 3, 2022

Conversation

Cervator
Copy link
Member

Without this fix Gradle fails to find the templates directory for some reason. Exact same setup (although different JDK 11 I think) on my non-M1 Mac worked fine. Not sure if it is a base issue with M1 Macs or the JDK, but in either case this fixed it.

More issues are piled up beyond this for where there are no arm64 lib variants for the new Apple Silicon chip setup, first one is protobuf, but that's a problem for another day.

@Cervator Cervator added Type: Bug Issues reporting and PRs fixing problems Category: Build/CI Requests, Issues and Changes targeting gradle, groovy, Jenkins, etc. labels Apr 24, 2022
@jdrueckert
Copy link
Member

With an implementation that is hopefully robust against differences in working directory.
@keturn
Copy link
Member

keturn commented Apr 26, 2022

That's weird, but not completely unheard-of. Gradle warns against using new File a lot, maybe something about this other JDK bumped in to one of the reasons why?

I've taken this opportunity to refactor this logic to a new Task class, CopyButNeverOverwrite, since we have at least one other task that was doing this with largely different methods.

Can you test the copyInMissingTemplates and jmxPassword tasks in your environment?

@Cervator
Copy link
Member Author

Both tasks work fine, nice and clean this way, cool 👍

@keturn keturn merged commit de37cfa into develop May 3, 2022
@keturn keturn deleted the pathfix branch May 3, 2022 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Build/CI Requests, Issues and Changes targeting gradle, groovy, Jenkins, etc. Type: Bug Issues reporting and PRs fixing problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants