-
Notifications
You must be signed in to change notification settings - Fork 62
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
Conversation
0874083
to
bc2a51b
Compare
Ping reviewers 🏓 |
platform/wayland/cog-utils-wl.h
Outdated
# 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 |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for this one.
bc2a51b
to
3f9a8b4
Compare
3f9a8b4
to
9af0922
Compare
There was a problem hiding this 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.
9af0922
to
c12bf19
Compare
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.