* [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.