Skip to content

Remove non-core APIs, normalize FloatABI values#2605

Merged
samsinsane merged 2 commits intopremake:masterfrom
nickclark2016:remove-vsandroid-extension-apis
Jan 15, 2026
Merged

Remove non-core APIs, normalize FloatABI values#2605
samsinsane merged 2 commits intopremake:masterfrom
nickclark2016:remove-vsandroid-extension-apis

Conversation

@nickclark2016
Copy link
Member

What does this PR do?

Removes (no deprecation) of non-first party APIs for VS Android projects. This gets rid of the VS extension support of Android from pre vs2015.

How does this PR change Premake's behavior?

Removes FPU and Endian APIs without deprecation.

Anything else we should know?

Preparation for 5.0.

Did you check all the boxes?

  • Focus on a single fix or feature; remove any unrelated formatting or code changes
  • Add unit tests showing fix or feature works; all tests pass
  • Mention any related issues (put closes #XXXX in comment to auto-close issue when PR is merged)
  • Follow our coding conventions
  • Minimize the number of commits
  • Align documentation to your changes

You can now support Premake on our OpenCollective. Your contributions help us spend more time responding to requests like these!

@nickclark2016 nickclark2016 requested a review from a team January 13, 2026 14:53
m.element("PlatformToolset", nil, toolset .. version)
m.element("AndroidArch", nil, archMap[arch])
elseif cfg.toolchainversion ~= nil then
local map = iif(cfg.toolset == "gcc", gcc_map, clang_map)
Copy link
Contributor

Choose a reason for hiding this comment

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

startswith to handle version?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm inclined to leave this as-is. This will be completely removed after beta8, since toolchainversion is deprecated.

@nickclark2016 nickclark2016 force-pushed the remove-vsandroid-extension-apis branch from 0fa69ce to f87055a Compare January 13, 2026 16:04
@samsinsane samsinsane merged commit f7fc559 into premake:master Jan 15, 2026
55 checks passed
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