Skip to content

Conversation

marshall-lee
Copy link

ioquake3 Makefile is gonna be deprecated by November 6, 2025.

See:

ioquake3 `Makefile` is gonna be deprecated by November 6, 2025.

See:
 - ioquake/ioq3#750
 - ioquake/ioq3@2c893f2
@cmitu
Copy link
Contributor

cmitu commented Sep 14, 2025

@marshall-lee thank you for the PR. I tested it and seems to work fine, using cmake simplifies things a bit. The only issue at this moment is the support for our current Pi images, which uses the old RaspoOS 10 'buster' release and which doesn't have the required pi version.

@marshall-lee
Copy link
Author

@cmitu

Thanks for testing it. So how can I help with older images in the context of this PR?

@cmitu
Copy link
Contributor

cmitu commented Sep 15, 2025

I can see 2 methods - either relaxing the cmake requirement in the CMakeLists.txt file or downloding a temporary cmake kit in $md_inst to be used during building.

EDIT: it seems the 1st method won't work, cmake needs a newer version than the one shipped by RaspiOS 10 (3.16).

@joolswills
Copy link
Member

joolswills commented Oct 8, 2025

We could use an older version of ioquake3 for systems with a version of cmake that is too old, but would need some additional build logic. Perhaps setting I_ACKNOWLEDGE_THE_MAKEFILE_IS_DEPRECATED=1 and using the last version that works with that option. That option could be set now as a quick workaround also I guess to fix building (Until the Makefile is removed or it doesn't work)

@marshall-lee
Copy link
Author

We could use an older version of ioquake3 for systems with a version of cmake that is too old, but would need some additional build logic. Perhaps setting I_ACKNOWLEDGE_THE_MAKEFILE_IS_DEPRECATED=1 and using the last version that works with that option. That option could be set now as a quick workaround also I guess to fix building (Until the Makefile is removed or it doesn't work)

Ok I'll try to add such logic

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