Skip to content

Conversation

@radekdoulik
Copy link
Member

No description provided.

@radekdoulik radekdoulik requested a review from lewing January 30, 2023 16:23
demands: ImageOverride -equals Build.Ubuntu.1804.Amd64
container:
image: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-18.04-webassembly3130-20230129084957-f7eb839
image: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-18.04-webassembly3130-20230130145247-f7eb839
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we're supposed to be using the "latest" docker tags, so just

Suggested change
image: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-18.04-webassembly3130-20230130145247-f7eb839
image: mcr.microsoft.com/dotnet-buildtools/prereqs:ubuntu-18.04-webassembly3130

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is temporary, so that we could prepare dotnet/runtime PR. once we have everything ready, it will use the ubuntu-18.04-webassembly image

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We'll probably need to keep different images in the future anyway since we're not planning on bumping emscripten in e.g. release/7.0 and the goal is to move to these "latest" tags everywhere

Copy link
Member Author

@radekdoulik radekdoulik Jan 30, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might make our mind meanwhile then and discuss what would work best and change it when the rest of our changes are ready. For now I would like to use these. The ubuntu-18.04-webassembly3130 image has only datetime tags right now, we can add floating tags later if needed.

When I chose the webassembly3130 name, I thought about it as temporary. If we want to introduce separate images, then something like webassembly-net8 might work better? The emscripten 3.1.30 might not be the final version yet.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup webassembly-net8 sounds better, I agree. The floating tag should be created automatically AFAIK (maybe only once there was a "production" rollout once a week).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah interesting, thanks

@radekdoulik radekdoulik merged commit dcc1f6e into dotnet:dotnet/main Feb 8, 2023
radical added a commit to radical/icu that referenced this pull request Feb 8, 2023
radical added a commit that referenced this pull request Feb 8, 2023
Revert "[wasm] Bump emscripten to 3.1.30 (#310)"
radekdoulik added a commit to radekdoulik/icu that referenced this pull request Feb 28, 2023
* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway
radekdoulik added a commit that referenced this pull request Feb 28, 2023
* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway
lewing added a commit that referenced this pull request Mar 1, 2023
radekdoulik added a commit to radekdoulik/icu that referenced this pull request Mar 9, 2023
* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway
radekdoulik added a commit that referenced this pull request Mar 9, 2023
* [wasm] Bump emscripten to 3.1.30 (#310)

* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway

* Use newer net8 images
radekdoulik added a commit to radekdoulik/icu that referenced this pull request Mar 10, 2023
* [wasm] Bump emscripten to 3.1.30 (dotnet#310)

* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway

* Use newer net8 images
radekdoulik added a commit that referenced this pull request Mar 10, 2023
* [wasm] Bump emscripten to 3.1.30 (#310)

* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway

* Use newer net8 images
steveisok pushed a commit to steveisok/icu that referenced this pull request May 25, 2023
* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway
steveisok pushed a commit to steveisok/icu that referenced this pull request May 25, 2023
steveisok pushed a commit to steveisok/icu that referenced this pull request May 25, 2023
* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway
steveisok pushed a commit to steveisok/icu that referenced this pull request May 25, 2023
steveisok pushed a commit to steveisok/icu that referenced this pull request May 25, 2023
* [wasm] Bump emscripten to 3.1.30 (dotnet#310)

* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway

* Use newer net8 images
steveisok pushed a commit to steveisok/icu that referenced this pull request May 25, 2023
* [wasm] Bump emscripten to 3.1.30 (dotnet#310)

* [wasm] Bump emscripten to 3.1.30

* Set right image for threading build

* Use newer images for wasi as well

It is probably not needed, but updated them anyway

* Use newer net8 images
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