Skip to content

Fix the declaration of BASE_IMAGE build argument - #8

Merged
pcarruscag merged 1 commit into
su2code:masterfrom
martin-g:update-ubuntu-for-cross-build-2
Jul 2, 2022
Merged

Fix the declaration of BASE_IMAGE build argument#8
pcarruscag merged 1 commit into
su2code:masterfrom
martin-g:update-ubuntu-for-cross-build-2

Conversation

@martin-g

@martin-g martin-g commented Jul 1, 2022

Copy link
Copy Markdown
Member

Follow up of #7

Signed-off-by: Martin Tzvetanov Grigorov <mgrigorov@apache.org>
@pcarruscag
pcarruscag merged commit b712109 into su2code:master Jul 2, 2022
@martin-g

martin-g commented Jul 3, 2022

Copy link
Copy Markdown
Member Author

Hi @pcarruscag,

It seems the Windows build needs manual steps: https://github.com/su2code/Docker-Builds/tree/master/build_cross#preparing-and-creating-msmpi-package

I won't be able to work on it next week though. I hope the broken release management work flow does not cause you problems!

@martin-g
martin-g deleted the update-ubuntu-for-cross-build-2 branch July 11, 2022 11:01
@martin-g

Copy link
Copy Markdown
Member Author

I am going to get the Windows binary from the old Docker image and put it in the build_cross folder.
I hope this is OK legally. At the moment the binary is inside the Docker container. Now it will be also in Git.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants