search

graalvm / setup-graalvm

GitHub Action for setting up GraalVM distributions.
https://www.graalvm.org
Universal Permissive License v1.0
192 stars 27 forks source link

Cleanup fails for windows, and marks the entire workflow as failed #24

Closed divs1210 closed 1 year ago

divs1210 commented 1 year ago
Screenshot 2022-11-25 at 3 37 43 PM
fniephaus commented 1 year ago

Thanks for the bug report! Could you please provide a link to the failing build? It seems that the build output JSON is malformed, which we need to fix. Also, the build report infrastructure should not fail the build. We'll look into fixing both next week. In the meantime, please disable build reports on Windows.

divs1210 commented 1 year ago

Thanks!

Link to failing build.

fniephaus commented 1 year ago

Thanks for the link. It seems that you're setting "-H:Name" to a path, which I think may cause the problem. I'll take a closer look next week.

fniephaus commented 1 year ago

The builds should no longer be marked as failed if report generation fails. Also, we're looking into fixing the malformed JSON issue in the next GraalVM release.

fniephaus commented 1 year ago

The fix for Native Image landed with: https://github.com/oracle/graal/pull/5634.