Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Mono.Android] Fix omitted Gl* constants. #9009

Merged
merged 1 commit into from
Jun 7, 2024
Merged

[Mono.Android] Fix omitted Gl* constants. #9009

merged 1 commit into from
Jun 7, 2024

Conversation

jpobst
Copy link
Contributor

@jpobst jpobst commented Jun 6, 2024

Fixes: #8612

Some OpenGL constants get removed because they conflict with method names after they go through our capitalization correction. For example, GLES20.GL_CULL_FACE becomes GLES20.GlCullFace which then conflicts with the method named GLES20.GlCullFace (int):

warning BG8401: Skipping 'Android.Opengl.GLES20.GlCullFace' due to a duplicate method name. (Java type: 'android.opengl.GLES20')

References:

Fix these cases by using metadata to assign explicit managedName values that are the expected constant name + Const to avoid the name collision.

@jpobst jpobst marked this pull request as ready for review June 7, 2024 14:17
@jpobst jpobst requested a review from jonpryor as a code owner June 7, 2024 14:17
@jonpryor jonpryor merged commit d2869b8 into main Jun 7, 2024
48 checks passed
@jonpryor jonpryor deleted the fix-gl-consts branch June 7, 2024 18:01
grendello added a commit that referenced this pull request Jun 17, 2024
* main: (22 commits)
  Bump to dotnet/android-tools@1c09dcc (#9026)
  Bump to dotnet/java-interop@ccafbe6 (#9025)
  [Mono.Android-Tests] Fix repo URL in redirect tests (#9035)
  [ci] Update checkout path for nightly build (#9028)
  [ci] Fix android source path for MAUI test job (#9030)
  Link Code of Conduct (#9034)
  [ci] Update sdk-insertions trigger to manual only (#9029)
  Update java-interop and android-tools submodule mentions (#9023)
  LEGO: Merge pull request 9022
  [Xamarin.Android.Build.Tasks] fastdev works with aab files (#8990)
  Use new binutils URL (#9019)
  Localized file check-in by OneLocBuild Task: Build definition ID 17928: Build ID 9686669 (#9011)
  LEGO: Merge pull request 9015
  [api-merge] Update "constant" values to mirror latest API levels (#9004)
  [Mono.Android] Fix wrong value for `ApplicationExitInfoReason.Other` (#9003)
  [Mono.Android] Fix omitted Gl* constants. (#9009)
  [manifest-attribute-codegen] Generate custom attribute declarations (#8781)
  [tests] Reduce default build output verbosity (#9002)
  [templates] Update Wear OS en template string (#9005)
  [build] Do not provision JDK 8 (#8999)
  ...
@github-actions github-actions bot locked and limited conversation to collaborators Jul 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Android.Opengl.GLES20 is missing constant for GL_CULL_FACE.
2 participants