Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 872080

Summary: www-client/uget: Build error: Requires libcurl version >= 7.19.1
Product: Gentoo Linux Reporter: Randall <ran.dall>
Component: Current packagesAssignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status: RESOLVED FIXED    
Severity: normal CC: sam
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 526268    
Attachments: www-client/uget-2.2.1-r1/work/uget-2.2.1/config.log
emerge -1v uget
www-client/uget-2.2.1-r1/work/uget-2.2.1/configure
emerge --info uget
emerge -1v uget

Description Randall 2022-09-20 20:00:43 UTC
I was trying to install uget today, but I seem to be getting this confusing error since my curl is at 7.85.0-r2, which causes the entire build to fail.

Upon inspection of the error, it seems not to be finding the `let` command in the configure script.

Reproducible: Always
Comment 1 Randall 2022-09-20 20:01:19 UTC
Created attachment 813154 [details]
www-client/uget-2.2.1-r1/work/uget-2.2.1/config.log
Comment 2 Randall 2022-09-20 20:01:43 UTC
Created attachment 813157 [details]
emerge -1v uget
Comment 3 Randall 2022-09-20 20:03:00 UTC
Created attachment 813160 [details]
www-client/uget-2.2.1-r1/work/uget-2.2.1/configure
Comment 4 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-09-20 20:06:42 UTC
Please include emerge --info and re upload the build.log as it seems to have HTML entity characters in it

CCing Arsen in case related to our recent changes but unlikely. Not able to check right this second
Comment 5 Randall 2022-09-20 20:08:47 UTC
Created attachment 813163 [details]
emerge --info uget
Comment 6 Randall 2022-09-20 20:09:39 UTC
Created attachment 813166 [details]
emerge -1v uget
Comment 7 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-09-20 20:10:19 UTC
Ah, I think it's a bashism.

Does it work if you change /bin/sh to bash temporarily?
Comment 8 Randall 2022-09-20 20:19:07 UTC
> Does it work if you change /bin/sh to bash temporarily?

How would I go about doing that?
Comment 9 Randall 2022-09-20 20:35:27 UTC
> How would I go about doing that?

So ajak helped me figure it out in IRC.

Turns out, it was a bashism. Once I switched back to bash from dash, it did emerge correctly.
Comment 10 Randall 2022-09-20 20:41:14 UTC
I'll leave this open if there's a need to correct the bashism; otherwise, this can be closed.
Comment 11 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2022-09-20 23:51:43 UTC
(In reply to Randall from comment #10)
> I'll leave this open if there's a need to correct the bashism; otherwise,
> this can be closed.

It's a legitimate bug and it's a good find :)
Comment 12 Larry the Git Cow gentoo-dev 2023-04-07 05:18:00 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=5638c9b9103b86ad8264feff2b7edecd6e7d6c7f

commit 5638c9b9103b86ad8264feff2b7edecd6e7d6c7f
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2023-04-07 05:12:33 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2023-04-07 05:12:33 +0000

    www-client/uget: drop broken curl version check
    
    This check can't handle curl >= 8 or non-bash /bin/sh.
    
    Closes: https://bugs.gentoo.org/872080
    Signed-off-by: Sam James <sam@gentoo.org>

 .../uget/files/uget-2.2.3-broken-curl-check.patch       | 17 +++++++++++++++++
 www-client/uget/uget-2.2.3.ebuild                       |  1 +
 www-client/uget/uget-9999.ebuild                        |  1 +
 3 files changed, 19 insertions(+)