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 aperezdc force-pushed the aperezdc/bump-libwpe branch from 0874083 to bc2a51b Compare March 6, 2024 15:52
@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.

@aperezdc aperezdc requested a review from psaavedra March 8, 2024 11:56
@aperezdc aperezdc force-pushed the aperezdc/bump-libwpe branch from bc2a51b to 3f9a8b4 Compare March 8, 2024 11:56
@aperezdc aperezdc requested a review from csaavedra March 8, 2024 11:56
@aperezdc aperezdc force-pushed the aperezdc/bump-libwpe branch from 3f9a8b4 to 9af0922 Compare March 21, 2024 17:46
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 force-pushed the aperezdc/bump-libwpe branch from 9af0922 to c12bf19 Compare March 27, 2024 11:06
@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