Skip to content

hidapi: update to 0.12.0#18738

Merged
neheb merged 1 commit intoopenwrt:masterfrom
neheb:7
Jun 22, 2022
Merged

hidapi: update to 0.12.0#18738
neheb merged 1 commit intoopenwrt:masterfrom
neheb:7

Conversation

@neheb
Copy link
Copy Markdown
Contributor

@neheb neheb commented Jun 10, 2022

Switch to cmake.

Signed-off-by: Rosen Penev rosenp@gmail.com

Maintainer: @paulfertser
Compile tested: mips24

@neheb
Copy link
Copy Markdown
Contributor Author

neheb commented Jun 10, 2022

and find_package(Iconv) is broken again. What a surprise...

@neheb neheb force-pushed the 7 branch 2 times, most recently from 7d3bf78 to 38bcf2f Compare June 11, 2022 05:19
@neheb
Copy link
Copy Markdown
Contributor Author

neheb commented Jun 11, 2022

Added a basic meson build. Took care to match the cmake build.

@neheb
Copy link
Copy Markdown
Contributor Author

neheb commented Jun 11, 2022

@dhewg does PKG_INSTALL do anything with meson packages?

Switch to cmake.

Add upstream patch fixing iconv detection.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
@neheb
Copy link
Copy Markdown
Contributor Author

neheb commented Jun 20, 2022

@paulfertser updated based on upstream feedback.

@dhewg
Copy link
Copy Markdown
Contributor

dhewg commented Jun 21, 2022

@dhewg does PKG_INSTALL do anything with meson packages?

I don't think so. Looking at it again:
if PKG_INSTALL is set, Build/Install will default to Build/Install/Default.
cmake provides Build/Install/Default and sets PKG_INSTALL so that it gets used.
meson provides Build/Install/Meson and sets Build/Install to default to it, so there's no need for PKG_INSTALL.

It's inconsistent, may be that could be cleaned up. But I don't recall if there was a specific reason to not go the cmake route.

@neheb
Copy link
Copy Markdown
Contributor Author

neheb commented Jun 21, 2022

Yeah I removed it: df5d724

@neheb neheb merged commit c1aa8e1 into openwrt:master Jun 22, 2022
@neheb neheb deleted the 7 branch June 22, 2022 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants