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

gbm_gralloc: Add mesa libgbm to shared lib based on platform sdk version #25

Merged
merged 1 commit into from
Sep 12, 2022

Conversation

Dhina17
Copy link
Contributor

@Dhina17 Dhina17 commented Aug 22, 2021

  • From Android 11, the mesa libgbm module renamed to libgbm_mesa to avoid conflict with minigbm.
    So Handle it correctly based on Platform SDK version.

[0] - https://android.googlesource.com/platform/external/mesa3d/+/7036b3872f7853dc732fb557dcd9b4ca31887fd6%5E%21

Signed-off-by: Dhina17 dhinalogu@gmail.com
Change-Id: I1d86f508a7729c8279d79a18666fb080be34745b

- From Android 11, the mesa libgbm module renamed to libgbm_mesa to avoid conflict with minigbm.
  So Handle it correctly based on Platform SDK version.

[0] - https://android.googlesource.com/platform/external/mesa3d/+/7036b3872f7853dc732fb557dcd9b4ca31887fd6%5E%21

Signed-off-by: Dhina17 <dhinalogu@gmail.com>
Change-Id: I1d86f508a7729c8279d79a18666fb080be34745b
@rsglobal
Copy link
Contributor

Thanks, but IIRC Android-11 still uses libgbm, only 12+ use libgbm_mesa
And you have to use your real name in both SOB and author,commiter fields..

@Dhina17
Copy link
Contributor Author

Dhina17 commented Aug 22, 2021

@rsglobal
Copy link
Contributor

Its updated in android-11.0.0_r38 tag. https://android.googlesource.com/platform/external/mesa3d/+/refs/tags/android-11.0.0_r38/src/gbm/Android.mk#39

I see. Thanks for pointing out.

@rsglobal
Copy link
Contributor

I assume this may be interesting to you: #26

@rsglobal
Copy link
Contributor

Hello @robherring ,

Can this be merged?

@rsglobal
Copy link
Contributor

related mainline mesa3d commit: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18533

@robherring robherring merged commit 6b742ba into robherring:master Sep 12, 2022
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.

None yet

3 participants