Skip to content

Early support of the Saigo toolchain#2862

Merged
illwieckz merged 1 commit intomasterfrom
illwieckz/saigo/sync
Nov 23, 2024
Merged

Early support of the Saigo toolchain#2862
illwieckz merged 1 commit intomasterfrom
illwieckz/saigo/sync

Conversation

@illwieckz
Copy link
Member

@illwieckz illwieckz added the A-Build When the code doesn't compile, or worse, compiles wrong label Dec 3, 2023
@illwieckz illwieckz marked this pull request as draft December 3, 2023 02:05
@illwieckz illwieckz force-pushed the illwieckz/saigo/sync branch 2 times, most recently from b798ab2 to 312035a Compare December 3, 2023 02:49
@illwieckz illwieckz force-pushed the illwieckz/saigo/sync branch from 312035a to 02f8abd Compare November 15, 2024 22:26
@illwieckz
Copy link
Member Author

First commit cmake: glm and tinygettext are cgame/sgame things was remade and merged as 6cff0a6

So only one commit is left.

@illwieckz illwieckz marked this pull request as ready for review November 15, 2024 23:06
@illwieckz illwieckz changed the title WIP: Saigo toolchain Early support of the Saigo toolchain Nov 16, 2024
@illwieckz illwieckz force-pushed the illwieckz/saigo/sync branch 3 times, most recently from bfdf060 to da64ec2 Compare November 19, 2024 22:04
CMakeLists.txt Outdated
DEPENDERS build)
foreach(NACL_VMS_PROJECT ${NACL_VMS_PROJECTS})
ExternalProject_Add_Step(${NACL_VMS_PROJECT} cbse-trigger
COMMENT "Finished CBSE generation, ready to build NaCl"
Copy link
Contributor

Choose a reason for hiding this comment

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

How about adding the project name in the comment so the message isn't identical every time?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done.

@illwieckz illwieckz force-pushed the illwieckz/saigo/sync branch 2 times, most recently from de543df to 479c684 Compare November 20, 2024 01:13
@slipher
Copy link
Contributor

slipher commented Nov 22, 2024

LGTM

@illwieckz illwieckz force-pushed the illwieckz/saigo/sync branch 2 times, most recently from 0eb9068 to d29e417 Compare November 23, 2024 19:26
@illwieckz illwieckz merged commit db49e16 into master Nov 23, 2024
@illwieckz illwieckz deleted the illwieckz/saigo/sync branch November 23, 2024 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Build When the code doesn't compile, or worse, compiles wrong

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants