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

Add new qemu platform for riscv64 - 2nd try #286

Closed
wants to merge 3 commits into from

Conversation

Hamlin-Li
Copy link

Hi,
Can you help to review this simple patch trying to #284 in github?
(This is the 2nd try of #285)
Thanks!

@codecov-commenter
Copy link

codecov-commenter commented Oct 31, 2023

Codecov Report

Merging #286 (00e8fc4) into master (89c7dcb) will decrease coverage by 0.15%.
The diff coverage is n/a.

❗ Current head 00e8fc4 differs from pull request most recent head 941302e. Consider uploading reports for the commit 941302e to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@             Coverage Diff              @@
##             master     #286      +/-   ##
============================================
- Coverage     59.74%   59.60%   -0.15%     
  Complexity      301      301              
============================================
  Files            26       26              
  Lines          1421     1421              
  Branches        164      164              
============================================
- Hits            849      847       -2     
- Misses          417      419       +2     
  Partials        155      155              

@luben
Copy link
Owner

luben commented Oct 31, 2023

that doesn't look to work and also breaks the s390x tests

@Hamlin-Li
Copy link
Author

Yes, both riscv and s390x encountered below issue:

/github_workspace/sbt-java-module-info /github_workspace
Downloading sbt launcher for 1.9.0:
  From  https://repo1.maven.org/maven2/org/scala-sbt/sbt-launch/1.9.0/sbt-launch-1.9.0.jar
    To  /root/.sbt/launchers/1.9.0/sbt-launch.jar
Downloading sbt launcher 1.9.0 md5 hash:
  From  https://repo1.maven.org/maven2/org/scala-sbt/sbt-launch/1.9.0/sbt-launch-1.9.0.jar.md5
    To  /root/.sbt/launchers/1.9.0/sbt-launch.jar.md5
cat: /root/.sbt/launchers/1.9.0/sbt-launch.jar.md5: No such file or directory
md5sum: 'standard input': no properly formatted MD5 checksum lines found
Checksum does not match
Could not download and verify the launcher. Obtain the jar manually and place it at /root/.sbt/launchers/1.9.0/sbt-launch.jar
Error: Process completed with exit code 1.

But I can access https://repo1.maven.org/maven2/org/scala-sbt/sbt-launch/1.9.0/sbt-launch-1.9.0.jar.md5.
Not sure what causes the issue.

@luben
Copy link
Owner

luben commented Nov 1, 2023

Can you try to rebase it - I have updated the sbt used by that plugin, so may be it will not fail with the new version

@Hamlin-Li
Copy link
Author

Seems we still face the same issue.

@luben
Copy link
Owner

luben commented Nov 1, 2023

Fails again. s390 workflow getting cancelled due to the other task failure. Can you add strategy: fail-fast: false to let it run to completion?

@luben luben closed this Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants