Skip to content

Conversation

@alexander-penev
Copy link
Contributor

No description provided.

@mcbarton
Copy link
Contributor

As you've got it to download the tar for the main the sha is now out of date, as a new commit(s) have been made.

@mcbarton
Copy link
Contributor

As you've got it to download the tar for the main the sha is now out of date, as a new commit(s) have been made.

Maybe worth reverting back to downloading the tar for version 0.4.0

@alexander-penev
Copy link
Contributor Author

alexander-penev commented May 11, 2024

@mcbarton Yes, the download from main was a temporary workaround for the patch issue. Now the patch is not needed and we are using version 0.4.0 again.

@DerThorsten DerThorsten merged commit 08b6827 into emscripten-forge:main May 12, 2024
@vgvassilev vgvassilev mentioned this pull request May 12, 2024
@alexander-penev alexander-penev deleted the AddXeusCpp branch May 12, 2024 07:12
@vgvassilev
Copy link
Contributor

@DerThorsten, thanks for merging this. It seems that the build after merging this PR has a problem: https://github.com/emscripten-forge/recipes/actions/runs/9049611938/job/24863926846 Do you have an idea what's going wrong here?

@DerThorsten
Copy link
Contributor

@DerThorsten, thanks for merging this. It seems that the build after merging this PR has a problem: https://github.com/emscripten-forge/recipes/actions/runs/9049611938/job/24863926846 Do you have an idea what's going wrong here?

there was a missing dependency in the environment of the ci, I rebuild xeus-cpp in #938 to trigger a reupload

@DerThorsten
Copy link
Contributor

luizirber pushed a commit to luizirber/recipes that referenced this pull request Aug 1, 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.

4 participants