-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
nghttp3: add new package #22443
nghttp3: add new package #22443
Conversation
07aecb3
to
679c6cf
Compare
* these changes along with 2 PRs below and using non-standard openssl library allow for building curl with HTTP/3 support * openwrt#22443 * openwrt#22444 Signed-off-by: Stan Grishin <stangri@melmac.ca>
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.
It looks simple, understandable, thanks!
@neheb maybe there can be added PKG_BUILD_PARALLEL
as well?
Sure, why not.
Dne ne 22. 10. 2023 16:24 uživatel Stan Grishin ***@***.***>
napsal:
… @BKPepe <https://github.com/BKPepe> @neheb <https://github.com/neheb> --
your thoughts on cherry-picking it for 23.05?
—
Reply to this email directly, view it on GitHub
<#22443 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA7IDVFFIBUIS5DAMM67PDLYAUUBJAVCNFSM6AAAAAA6FXX6CWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZUGEYDQOJWGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Upstream supports CMake as well, would be nice to use it. (the same for ngtcp2) |
I've had limited experience/exposure with the Makefiles. Is it just a matter of adding If not, do you want to take over maintainership of these two libs and make necessary changes (either before or after this is merged)? |
Basically yes. diff --git a/libs/nghttp3/Makefile b/libs/nghttp3/Makefile
index 8ee97b3e84..29ecf73d27 100644
--- a/libs/nghttp3/Makefile
+++ b/libs/nghttp3/Makefile
@@ -12,10 +12,10 @@ PKG_LICENSE:=MIT
PKG_LICENSE_FILES:=COPYING
PKG_MAINTAINER:=Stan Grishin <stangri@melmac.ca>
-PKG_FIXUP:=autoreconf
-PKG_INSTALL:=1
+CMAKE_INSTALL:=1
include $(INCLUDE_DIR)/package.mk
+include $(INCLUDE_DIR)/cmake.mk
define Package/libnghttp3
SECTION:=libs
@@ -28,15 +28,7 @@ define Package/libnghttp3/description
nghttp3 is a thin HTTP/3 layer over an underlying QUIC stack.
endef
-CONFIGURE_ARGS += --enable-lib-only
-
-define Build/InstallDev
- $(INSTALL_DIR) $(1)/usr/include/nghttp3
- $(INSTALL_DATA) $(PKG_INSTALL_DIR)/usr/include/nghttp3/*.h $(1)/usr/include/nghttp3/
- $(INSTALL_DIR) $(1)/usr/lib/pkgconfig
- $(INSTALL_DATA) $(PKG_INSTALL_DIR)/usr/lib/pkgconfig/*.pc $(1)/usr/lib/pkgconfig/
- $(INSTALL_DATA) $(PKG_INSTALL_DIR)/usr/lib/libnghttp3.so* $(1)/usr/lib/
-endef
+CMAKE_OPTIONS += -DENABLE_LIB_ONLY=ON
define Package/libnghttp3/install
$(INSTALL_DIR) $(1)/usr/lib
|
* add new package to allow building of curl with HTTP/3 support * switch to using cmake Signed-off-by: Stan Grishin <stangri@melmac.ca>
679c6cf
to
b1d4241
Compare
* these changes along with 2 PRs below and using non-standard openssl library allow for building curl with HTTP/3 support * openwrt#22443 * openwrt#22444 Signed-off-by: Stan Grishin <stangri@melmac.ca> (cherry picked from commit 6bd2b89)
* these changes along with 2 PRs below and using non-standard openssl library allow for building curl with HTTP/3 support * openwrt#22443 * openwrt#22444 Signed-off-by: Stan Grishin <stangri@melmac.ca> (cherry picked from commit 6bd2b89)
* these changes along with 2 PRs below and using non-standard openssl library allow for building curl with HTTP/3 support * openwrt#22443 * openwrt#22444 Signed-off-by: Stan Grishin <stangri@melmac.ca>
Maintainer: me
Compile tested: x86_64, Sophos XG-135r3, OpenWrt 23.05.0
Run tested: x86_64, Sophos XG-135r3, OpenWrt 23.05.0
Description:
PS. I'd like to solicit feedback on cherry-picking it for 23.05 as well.