Skip to content

Conversation

@jpnurmi
Copy link
Collaborator

@jpnurmi jpnurmi commented Apr 17, 2025

Uses dotnet-install.sh from https://github.com/dotnet/install-scripts.

#skip-changelog

Close #2114

@jpnurmi jpnurmi requested a review from bitsandfoxes as a code owner April 17, 2025 09:30
Copy link
Contributor

@bitsandfoxes bitsandfoxes Apr 17, 2025

Choose a reason for hiding this comment

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

I'm sorry, I did not think this through in my comment.
Now that the build & copy logic moved into a bash-script, how much effort would it be to add dotnet to the docker and leave the rest as-is? 😅

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not much effort at all because I already experimented with that idea yesterday 😉

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I pushed the ubuntu:20.04 + dotnet-install.sh variant that feels nicest out of the discussed options. What do you think?

Copy link
Contributor

Choose a reason for hiding this comment

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

It's so beautiful, it's frightening! Thanks a lot!

Copy link
Contributor

@bitsandfoxes bitsandfoxes left a comment

Choose a reason for hiding this comment

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

Awesome! Thanks a lot!

@bitsandfoxes bitsandfoxes merged commit ec7f381 into main Apr 17, 2025
71 of 79 checks passed
@bitsandfoxes bitsandfoxes deleted the fix/ubuntu-20.04 branch April 17, 2025 10:59
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.

Dockerize building sentry-native for Linux

3 participants