Skip to content

Add TBB for gcc and fix unzip on vcpkg#17

Merged
duncanspumpkin merged 2 commits intomasterfrom
addTbb
Jan 9, 2026
Merged

Add TBB for gcc and fix unzip on vcpkg#17
duncanspumpkin merged 2 commits intomasterfrom
addTbb

Conversation

@duncanspumpkin
Copy link
Copy Markdown
Contributor

This adds tbb as required for libstdc++ and it also adds 64bit images and also fixes vcpkg cache issues by including unzip.

@@ -0,0 +1,4 @@
FROM fedora:41
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fedora 41 is EOL; let's push this to 43 while we're at it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just saw OpenLoco/OpenLoco#3608 (comment). That sucks.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fedora 43 doesn't have sdl2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hang on, wouldn't sdl2-compat work still?

Copy link
Copy Markdown
Member

@AaronVanGeffen AaronVanGeffen left a comment

Choose a reason for hiding this comment

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

Any particular reason for creating separate 32-bit and 64-bit images? The OS is 64-bit regardless, right.

@duncanspumpkin
Copy link
Copy Markdown
Contributor Author

Any particular reason for creating separate 32-bit and 64-bit images? The OS is 64-bit regardless, right.

It can get messy trying to build 32bit when you have both the 64bit and 32bit dependencies installed.

@duncanspumpkin duncanspumpkin merged commit 632c96d into master Jan 9, 2026
14 checks passed
@duncanspumpkin duncanspumpkin deleted the addTbb branch January 9, 2026 09:23
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.

2 participants