Skip to content

scp: fix build with gcc >= 14.x - #474

Closed
bkuhls wants to merge 1 commit into
mkj:mainfrom
bkuhls:DROPBEAR_2026.93-gcc16
Closed

scp: fix build with gcc >= 14.x#474
bkuhls wants to merge 1 commit into
mkj:mainfrom
bkuhls:DROPBEAR_2026.93-gcc16

Conversation

@bkuhls

@bkuhls bkuhls commented Jul 21, 2026

Copy link
Copy Markdown

Fixes build errors seen with gcc 14.x:

src/scp.c: In function ‘sink’:
src/scp.c:1214:32: error: implicit declaration of function ‘ascii_isdigit’
 [-Wimplicit-function-declaration]

src/scp.c: In function ‘okname’:
src/scp.c:1501:22: error: implicit declaration of function ‘ascii_isalpha’
 [-Wimplicit-function-declaration]

Patch provided by SVoxel: #473

Fixes build errors seen with gcc 14.x:

src/scp.c: In function ‘sink’:
src/scp.c:1214:32: error: implicit declaration of function ‘ascii_isdigit’
 [-Wimplicit-function-declaration]

src/scp.c: In function ‘okname’:
src/scp.c:1501:22: error: implicit declaration of function ‘ascii_isalpha’
 [-Wimplicit-function-declaration]

Patch provided by SVoxel: mkj#473

Signed-off-by: Bernd Kuhls <bernd@kuhls.net>
@benoit-pierre

Copy link
Copy Markdown

You're not getting an error at link time? (ascii_isdigit is implemented in dbutil.c, which scp is not linked with)

@bkuhls

bkuhls commented Jul 21, 2026

Copy link
Copy Markdown
Author

You're not getting an error at link time? (ascii_isdigit is implemented in dbutil.c, which scp is not linked with)

Correct, for whatever reason.

I found the build error while preparing the version bump for the buildroot project and most of their Gitlab pipelines failed without this fix, only some toolchains using older gcc versions could build the new version:
https://gitlab.com/bkuhls/buildroot/-/pipelines/2694914612

With this patch all pipelines passed:
https://gitlab.com/bkuhls/buildroot/-/pipelines/2694690178

@benoit-pierre

Copy link
Copy Markdown

It's because you're building with MULTI=1: one binary with everything included.

@mkj mkj added the regression label Jul 22, 2026
@mkj

mkj commented Jul 22, 2026

Copy link
Copy Markdown
Owner

#475 has a fix merged now

@mkj mkj closed this Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants