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

wpebackend-rdk: Bump up version to 1.20210402 #279

Closed
wants to merge 1 commit into from
Closed

Conversation

psaavedra
Copy link
Member

SSIA

@psaavedra psaavedra requested review from bertogg and aperezdc June 7, 2021 09:50
@psaavedra psaavedra self-assigned this Jun 7, 2021
Copy link
Member

@aperezdc aperezdc left a comment

Choose a reason for hiding this comment

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

I think that enabling libinput should be smarter, please check the comment below.

Comment on lines +3 to +5
SRCREV = "196f75360b8643b119d8d96208d914998e5d4fc9"

PV = "1.20210402"
Copy link
Member

Choose a reason for hiding this comment

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

Too bad upstream does not seem to have interest in making releases—I've filed WebPlatformForEmbedded/WPEBackend-rdk#35 ~2 years ago to no avail. Using date-coded versions is the best we can do here for now 🤷🏼

Comment on lines +25 to +26
PACKAGECONFIG[libinput] = "-DUSE_INPUT_LIBINPUT=ON,-DUSE_INPUT_LIBINPUT=OFF,libinput,"
PACKAGECONFIG[libinput-udev] = "-DUSE_INPUT_UDEV=ON,-DUSE_INPUT_UDEV=OFF,libinput eudev,"
Copy link
Member

Choose a reason for hiding this comment

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

Certain backends require usage of libinput to be able to receive input events. For example the src/viv-imx6/view-backend.cpp file uses the LibinputServer unconditionally, so passing -DUSE_INPUT_LIBINPUT=OFF will make the build fail. It looks to me that this kind of cases should be taken into account here.

As for -DUSE_INPUT_UDEV= option, what I would do is always enable it if libudev is being built. The amount of code added to the backend is very small, and libput will fallback to enumerate itself devices from /dev/input if udev is not available at runtime, so there is hardly a reason to have a build option for this. That being said, I have no idea how complicated to do the “enable option X if package Y is being built” with Bitbake, so if it's hard feel free to ignore the suggestion.

@github-actions
Copy link

This PR is stale because it has been open 90 days with no activity.

@clopez
Copy link
Contributor

clopez commented Dec 1, 2022

I have sent a new PR to update this backend to a newer version that obsoletes this PR (I think).
See: #433

@psaavedra
Copy link
Member Author

ok

@psaavedra psaavedra closed this Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants