-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
feat: make sure for both cygwin and msys cygpath is used #343
Conversation
Please provide the Grinder links for both cases. Thanks |
What issue does this address? |
for x86_64-linux https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/5417
when I was checking why run-aqa/aqa-tests not working on windows-2019, i found the code in temurin-build https://github.com/adoptium/temurin-build/blob/0aee022f86093836998a48b9795002fa6afeb817/sbin/prepareWorkspace.sh#L420 checks on both cygwin and mingw convert path. I do not know if this could be problem if we use a newer version or anyone wants to use AQA but install with mingw on their windows, just think it is good to align the logic in both code. |
@zdtsw what error did you encounter on windows-2019? Did this PR resolve the problem? |
the error can be found from https://github.com/adoptium/run-aqa/runs/7789840940?check_suite_focus=true it is on 2022/latest. the PR is not for fix the problem, but only to match what we have in the build script's logic to support both cygwin and minGW. |
Given cygwin is a stated prereq for AQAvit, this change is likely not warranted. |
right! this PR probably not needed for the current requirement. |
Yes, in which case we should create an issue and discuss and design an approach, not just needle in a PR here or there without thinking of all of the implications (and testing that will need to happen) if we want to support a new environment. |
No description provided.