-
Notifications
You must be signed in to change notification settings - Fork 115
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
Run GH-workflow on Linux, Windows and Mac #1081
Conversation
1eccb97
to
7964418
Compare
@HannesWell the |
7964418
to
ed30934
Compare
Thanks. Fixed it. |
ed30934
to
33a3915
Compare
The Test result upload for the Windows build still fails with: I don't understand that. Anybody has a hint? |
Sadly there is a bug in the upload workflow see for example: |
ba29b63
to
97a6414
Compare
97a6414
to
32caab5
Compare
@HannesWell do you like to finish this? |
32caab5
to
3411832
Compare
6e90837
to
5560b90
Compare
@kwin you mentioned in actions/upload-artifact#240 (comment) that with a recent node.js the windows issues should be fixed. Can you explain what has to be changed in this repo to get a fixed version? If we don't make it work on windows and #1154 at least we can alread merge this for Linux and Mac only and add Windows later, when it is working. But the current failure on Windows seem to be a baseline issue. |
Nothing, it should just work if GitHub Runners are connected to this repo and not custom ones.
Indeed not related to the NodeJS issues related to long paths on Windows. |
5560b90
to
7d82263
Compare
We use the GH-runners so that should work already.
How did you identify the long paths as an issue? From my local builds I only know that there are baseline issues because on Windows text files are usually checked out with windows-style lineendings why they are committed in git and in the p2 base-line repo with linuy-style lineendings. |
I didn’t but @laeubi did in #1081 (comment) |
Maybe we can disable Windows until the baseline issues are resolved (or just disable baselining for the time being on Windows with a specific Maven profile) |
Ah, sorry I misunderstood your comment. Just disregard mine. |
Yes, I thought about that as well. Disabling it temporarily in a profile on windows (or at least downgrade it to warnings) seems reasonable in that case. I'll try that tomorrow. |
7d82263
to
2905921
Compare
Now we have again the initial error... |
2905921
to
00b12e8
Compare
On Linux and Mac this passes now. So lets enable that for now and add Windows later, when it is working. |
Lets see if the build passes on Windows as well.
Fixes #1080.