* [Buildroot] [PATCH 1/2] package/nghttp2: add host variant @ 2022-04-22 18:14 Joseph Kogut 2022-04-22 18:14 ` [Buildroot] [PATCH 2/2] package/libcurl: add host-nghttp2 to host deps Joseph Kogut 0 siblings, 1 reply; 6+ messages in thread From: Joseph Kogut @ 2022-04-22 18:14 UTC (permalink / raw) To: buildroot; +Cc: Joseph Kogut, Anisse Astier, Thomas Petazzoni, Matt Weber Signed-off-by: Joseph Kogut <joseph.kogut@gmail.com> --- package/nghttp2/nghttp2.mk | 1 + 1 file changed, 1 insertion(+) diff --git a/package/nghttp2/nghttp2.mk b/package/nghttp2/nghttp2.mk index 9190fa30d6..5f46f24431 100644 --- a/package/nghttp2/nghttp2.mk +++ b/package/nghttp2/nghttp2.mk @@ -21,3 +21,4 @@ endef NGHTTP2_POST_INSTALL_TARGET_HOOKS += NGHTTP2_INSTALL_CLEAN_HOOK $(eval $(autotools-package)) +$(eval $(host-autotools-package)) -- 2.36.0 _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [Buildroot] [PATCH 2/2] package/libcurl: add host-nghttp2 to host deps 2022-04-22 18:14 [Buildroot] [PATCH 1/2] package/nghttp2: add host variant Joseph Kogut @ 2022-04-22 18:14 ` Joseph Kogut 2022-04-22 21:56 ` Arnout Vandecappelle 0 siblings, 1 reply; 6+ messages in thread From: Joseph Kogut @ 2022-04-22 18:14 UTC (permalink / raw) To: buildroot; +Cc: Joseph Kogut, Anisse Astier, Thomas Petazzoni, Matt Weber This fixes building host-libcurl, which otherwise would fail with undefined references to nghttp2 symbols. /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_pack_settings_payload' /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_http2_strerror' /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_request' /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_rst_stream' <snip> collect2: error: ld returned 1 exit status Signed-off-by: Joseph Kogut <joseph.kogut@gmail.com> --- package/libcurl/libcurl.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package/libcurl/libcurl.mk b/package/libcurl/libcurl.mk index 97857954db..d5805a1d32 100644 --- a/package/libcurl/libcurl.mk +++ b/package/libcurl/libcurl.mk @@ -173,7 +173,7 @@ endef LIBCURL_POST_INSTALL_TARGET_HOOKS += LIBCURL_TARGET_CLEANUP endif -HOST_LIBCURL_DEPENDENCIES = host-openssl +HOST_LIBCURL_DEPENDENCIES = host-openssl host-nghttp2 HOST_LIBCURL_CONF_OPTS = \ --disable-manual \ --disable-ntlm-wb \ -- 2.36.0 _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH 2/2] package/libcurl: add host-nghttp2 to host deps 2022-04-22 18:14 ` [Buildroot] [PATCH 2/2] package/libcurl: add host-nghttp2 to host deps Joseph Kogut @ 2022-04-22 21:56 ` Arnout Vandecappelle 2022-04-22 22:09 ` Joseph Kogut 0 siblings, 1 reply; 6+ messages in thread From: Arnout Vandecappelle @ 2022-04-22 21:56 UTC (permalink / raw) To: Joseph Kogut, buildroot; +Cc: Anisse Astier, Thomas Petazzoni, Matt Weber On 22/04/2022 20:14, Joseph Kogut wrote: > This fixes building host-libcurl, which otherwise would fail with > undefined references to nghttp2 symbols. > > /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_pack_settings_payload' > /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_http2_strerror' > /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_request' > /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_rst_stream' I guess this happens because it somehow finds libnghttp2 installed on the host but then fails to link with it? In any case, we don't actually need host-libcurl to be built with nghttp2 I think. So just pass --without-nghttp2. Regards, Arnout > <snip> > collect2: error: ld returned 1 exit status > > Signed-off-by: Joseph Kogut <joseph.kogut@gmail.com> > --- > package/libcurl/libcurl.mk | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/package/libcurl/libcurl.mk b/package/libcurl/libcurl.mk > index 97857954db..d5805a1d32 100644 > --- a/package/libcurl/libcurl.mk > +++ b/package/libcurl/libcurl.mk > @@ -173,7 +173,7 @@ endef > LIBCURL_POST_INSTALL_TARGET_HOOKS += LIBCURL_TARGET_CLEANUP > endif > > -HOST_LIBCURL_DEPENDENCIES = host-openssl > +HOST_LIBCURL_DEPENDENCIES = host-openssl host-nghttp2 > HOST_LIBCURL_CONF_OPTS = \ > --disable-manual \ > --disable-ntlm-wb \ _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH 2/2] package/libcurl: add host-nghttp2 to host deps 2022-04-22 21:56 ` Arnout Vandecappelle @ 2022-04-22 22:09 ` Joseph Kogut 2022-04-22 22:19 ` Arnout Vandecappelle 0 siblings, 1 reply; 6+ messages in thread From: Joseph Kogut @ 2022-04-22 22:09 UTC (permalink / raw) To: Arnout Vandecappelle Cc: Matt Weber, Anisse Astier, Thomas Petazzoni, buildroot On Fri, Apr 22, 2022 at 2:56 PM Arnout Vandecappelle <arnout@mind.be> wrote: > > > > On 22/04/2022 20:14, Joseph Kogut wrote: > > This fixes building host-libcurl, which otherwise would fail with > > undefined references to nghttp2 symbols. > > > > /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_pack_settings_payload' > > /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_http2_strerror' > > /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_request' > > /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_rst_stream' > > I guess this happens because it somehow finds libnghttp2 installed on the host > but then fails to link with it? In any case, we don't actually need host-libcurl > to be built with nghttp2 I think. So just pass --without-nghttp2. > It seems so. I think either providing the library or disabling it will work fine, I just didn't want to assume what other users need here. I'll respin with that config, thanks for the review. > > Regards, > Arnout > > > <snip> > > collect2: error: ld returned 1 exit status > > > > Signed-off-by: Joseph Kogut <joseph.kogut@gmail.com> > > --- > > package/libcurl/libcurl.mk | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/package/libcurl/libcurl.mk b/package/libcurl/libcurl.mk > > index 97857954db..d5805a1d32 100644 > > --- a/package/libcurl/libcurl.mk > > +++ b/package/libcurl/libcurl.mk > > @@ -173,7 +173,7 @@ endef > > LIBCURL_POST_INSTALL_TARGET_HOOKS += LIBCURL_TARGET_CLEANUP > > endif > > > > -HOST_LIBCURL_DEPENDENCIES = host-openssl > > +HOST_LIBCURL_DEPENDENCIES = host-openssl host-nghttp2 > > HOST_LIBCURL_CONF_OPTS = \ > > --disable-manual \ > > --disable-ntlm-wb \ _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH 2/2] package/libcurl: add host-nghttp2 to host deps 2022-04-22 22:09 ` Joseph Kogut @ 2022-04-22 22:19 ` Arnout Vandecappelle 2022-04-22 22:32 ` Joseph Kogut 0 siblings, 1 reply; 6+ messages in thread From: Arnout Vandecappelle @ 2022-04-22 22:19 UTC (permalink / raw) To: Joseph Kogut; +Cc: Matt Weber, Anisse Astier, Thomas Petazzoni, buildroot On 23/04/2022 00:09, Joseph Kogut wrote: > On Fri, Apr 22, 2022 at 2:56 PM Arnout Vandecappelle <arnout@mind.be> wrote: >> >> >> >> On 22/04/2022 20:14, Joseph Kogut wrote: >>> This fixes building host-libcurl, which otherwise would fail with >>> undefined references to nghttp2 symbols. >>> >>> /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_pack_settings_payload' >>> /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_http2_strerror' >>> /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_request' >>> /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_rst_stream' >> >> I guess this happens because it somehow finds libnghttp2 installed on the host >> but then fails to link with it? In any case, we don't actually need host-libcurl >> to be built with nghttp2 I think. So just pass --without-nghttp2. >> > > It seems so. I think either providing the library or disabling it will > work fine, I just didn't want to assume what other users need here. > I'll respin with that config, thanks for the review. Actually, host-libcurl was only added for host-cargo. When host-cargo was removed, we should have removed host-libcurl as well. So the even better solution would be to simply revert commit 736e0fc5d6103300fb5d6b0cde5e22ce978b8a80. How did you even run in to this error? There should be no reason to build host-libcurl... Regards, Arnout > >> >> Regards, >> Arnout >> >>> <snip> >>> collect2: error: ld returned 1 exit status >>> >>> Signed-off-by: Joseph Kogut <joseph.kogut@gmail.com> >>> --- >>> package/libcurl/libcurl.mk | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/package/libcurl/libcurl.mk b/package/libcurl/libcurl.mk >>> index 97857954db..d5805a1d32 100644 >>> --- a/package/libcurl/libcurl.mk >>> +++ b/package/libcurl/libcurl.mk >>> @@ -173,7 +173,7 @@ endef >>> LIBCURL_POST_INSTALL_TARGET_HOOKS += LIBCURL_TARGET_CLEANUP >>> endif >>> >>> -HOST_LIBCURL_DEPENDENCIES = host-openssl >>> +HOST_LIBCURL_DEPENDENCIES = host-openssl host-nghttp2 >>> HOST_LIBCURL_CONF_OPTS = \ >>> --disable-manual \ >>> --disable-ntlm-wb \ _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH 2/2] package/libcurl: add host-nghttp2 to host deps 2022-04-22 22:19 ` Arnout Vandecappelle @ 2022-04-22 22:32 ` Joseph Kogut 0 siblings, 0 replies; 6+ messages in thread From: Joseph Kogut @ 2022-04-22 22:32 UTC (permalink / raw) To: Arnout Vandecappelle Cc: Matt Weber, Anisse Astier, Thomas Petazzoni, buildroot On Fri, Apr 22, 2022 at 3:19 PM Arnout Vandecappelle <arnout@mind.be> wrote: > > > > On 23/04/2022 00:09, Joseph Kogut wrote: > > On Fri, Apr 22, 2022 at 2:56 PM Arnout Vandecappelle <arnout@mind.be> wrote: > >> > >> > >> > >> On 22/04/2022 20:14, Joseph Kogut wrote: > >>> This fixes building host-libcurl, which otherwise would fail with > >>> undefined references to nghttp2 symbols. > >>> > >>> /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_pack_settings_payload' > >>> /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_http2_strerror' > >>> /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_request' > >>> /usr/bin/ld: ../lib/.libs/libcurl.so: undefined reference to `nghttp2_submit_rst_stream' > >> > >> I guess this happens because it somehow finds libnghttp2 installed on the host > >> but then fails to link with it? In any case, we don't actually need host-libcurl > >> to be built with nghttp2 I think. So just pass --without-nghttp2. > >> > > > > It seems so. I think either providing the library or disabling it will > > work fine, I just didn't want to assume what other users need here. > > I'll respin with that config, thanks for the review. > > Actually, host-libcurl was only added for host-cargo. When host-cargo was > removed, we should have removed host-libcurl as well. > > So the even better solution would be to simply revert commit > 736e0fc5d6103300fb5d6b0cde5e22ce978b8a80. > > How did you even run in to this error? There should be no reason to build > host-libcurl... > I have an external package that makes an HTTP query to populate a config file. I'm not sure if there's a better way to do this, if you have any thoughts, I'm all ears. > Regards, > Arnout > > > > >> > >> Regards, > >> Arnout > >> > >>> <snip> > >>> collect2: error: ld returned 1 exit status > >>> > >>> Signed-off-by: Joseph Kogut <joseph.kogut@gmail.com> > >>> --- > >>> package/libcurl/libcurl.mk | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/package/libcurl/libcurl.mk b/package/libcurl/libcurl.mk > >>> index 97857954db..d5805a1d32 100644 > >>> --- a/package/libcurl/libcurl.mk > >>> +++ b/package/libcurl/libcurl.mk > >>> @@ -173,7 +173,7 @@ endef > >>> LIBCURL_POST_INSTALL_TARGET_HOOKS += LIBCURL_TARGET_CLEANUP > >>> endif > >>> > >>> -HOST_LIBCURL_DEPENDENCIES = host-openssl > >>> +HOST_LIBCURL_DEPENDENCIES = host-openssl host-nghttp2 > >>> HOST_LIBCURL_CONF_OPTS = \ > >>> --disable-manual \ > >>> --disable-ntlm-wb \ _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-04-22 22:33 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-04-22 18:14 [Buildroot] [PATCH 1/2] package/nghttp2: add host variant Joseph Kogut 2022-04-22 18:14 ` [Buildroot] [PATCH 2/2] package/libcurl: add host-nghttp2 to host deps Joseph Kogut 2022-04-22 21:56 ` Arnout Vandecappelle 2022-04-22 22:09 ` Joseph Kogut 2022-04-22 22:19 ` Arnout Vandecappelle 2022-04-22 22:32 ` Joseph Kogut
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.