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.

Sign up for GitHub

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

Jump to bottom

fix(evdev): Add missing include for strerror #6047

Merged
merged 1 commit into from
Apr 15, 2024
Merged

fix(evdev): Add missing include for strerror #6047

merged 1 commit into from
Apr 15, 2024

Conversation

niklasf
Copy link
Contributor

@niklasf niklasf commented Apr 9, 2024

No description provided.

@kisvegabor
Copy link
Member

@FASTSHIFT mentioned that strerror might not be supported in NuttX. I wonder if evdev is supported in NuttX because if so, we might want to avoid using strerror here.

@niklasf
Copy link
Contributor Author

niklasf commented Apr 13, 2024

Looks like the virtio-input is close, but some more changes would be required. So just keep using it until NuttX support is actually added?

@FASTSHIFT
Copy link
Collaborator

@FASTSHIFT mentioned that strerror might not be supported in NuttX. I wonder if evdev is supported in NuttX because if so, we might want to avoid using strerror here.

NuttX does not support evdev. You can ignore NuttX.

@kisvegabor kisvegabor merged commit 4f68cd6 into lvgl:master Apr 15, 2024
16 checks passed
@niklasf niklasf deleted the fix/evdev-include-strerror branch April 15, 2024 16:41
kisvegabor pushed a commit to zjanosy/lvgl that referenced this pull request Apr 17, 2024
@niklasf @kisvegabor
fix(evdev): add missing include for strerror (lvgl#6047)
57af779
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kisvegabor kisvegabor kisvegabor approved these changes

@FASTSHIFT FASTSHIFT FASTSHIFT approved these changes

@liamHowatt liamHowatt liamHowatt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@niklasf @kisvegabor @FASTSHIFT @liamHowatt