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

platform: Bump libwpe requirement to 1.14 #687

Merged
merged 1 commit into from
Mar 27, 2024
Merged

Conversation

aperezdc
Copy link
Member

@aperezdc aperezdc commented Jan 3, 2024

Bump required libwpe version to 1.14.0, which allows dropping conditional compilation for setting target refresh rates. Always building the code reduces the testing matrix, and version 1.14.0 is over a year old so the change should not pose any issue for packagers.

@aperezdc aperezdc added the enhancement New feature or request label Jan 3, 2024
@aperezdc aperezdc self-assigned this Jan 3, 2024
@aperezdc
Copy link
Member Author

aperezdc commented Mar 6, 2024

Ping reviewers 🏓

# define HAVE_SHM_EXPORTED_BUFFER WPE_FDO_CHECK_VERSION(1, 9, 0)
# define HAVE_FULLSCREEN_HANDLING WPE_FDO_CHECK_VERSION(1, 11, 1)
#else
# define HAVE_SHM_EXPORTED_BUFFER 0
Copy link
Member

Choose a reason for hiding this comment

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

and similar thing here:

aperezdc/bump-libwpe psaavedra@golpe ~/local/git/WPEBackend-fdo $ grep -r  HAVE_SHM_EXPORTED_BUFFER  * 
platform/drm/cog-platform-drm.c:# define HAVE_SHM_EXPORTED_BUFFER WPE_FDO_CHECK_VERSION(1, 9, 0)
platform/drm/cog-platform-drm.c:# define HAVE_SHM_EXPORTED_BUFFER 0

Copy link
Member Author

Choose a reason for hiding this comment

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

Same for this one.

Copy link
Member

@psaavedra psaavedra left a comment

Choose a reason for hiding this comment

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

LGTM. r+

Bump required libwpe version to 1.14.0, which allows dropping
conditional compilation for setting target refresh rates. Always
building the code reduces the testing matrix, and version 1.14.0 is
over a year old so the change should not pose any issue for packagers.
@aperezdc aperezdc merged commit ecf0d3a into master Mar 27, 2024
5 checks passed
@aperezdc aperezdc deleted the aperezdc/bump-libwpe branch March 27, 2024 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants