Fix the declaration of BASE_IMAGE build argument by martin-g · Pull Request #8 · su2code/Docker-Builds

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Merged

pcarruscag merged 1 commit intosu2code: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 intosu2code:masterfrom

martin-g:update-ubuntu-for-cross-build-2

Conversation

@martin-g

Copy link

Member

@martin-g martin-g commented

Jul 1, 2022

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

Copy link

Member Author

martin-g commented

Jul 3, 2022

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

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

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@martin-g @pcarruscag