Page MenuHomeFreeBSD

Fix gcc build
ClosedPublic

Authored by lwhsu on Jun 12 2019, 1:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 19, 12:21 PM
Unknown Object (File)
Wed, Apr 16, 3:38 AM
Unknown Object (File)
Sun, Apr 13, 1:04 PM
Unknown Object (File)
Mar 23 2025, 12:03 PM
Unknown Object (File)
Feb 17 2025, 12:51 AM
Unknown Object (File)
Feb 13 2025, 5:02 PM
Unknown Object (File)
Feb 13 2025, 4:00 PM
Unknown Object (File)
Feb 13 2025, 1:43 PM
Subscribers

Details

Summary

Fix gcc build

https://ci.freebsd.org/job/FreeBSD-head-amd64-gcc/10338/console :

In file included from /workspace/src/stand/efi/boot1/boot1.c:34:0:
/workspace/src/stand/efi/boot1/boot_module.h:112:12: error: redundant redeclaration of 'vsnprintf' [-Werror=redundant-decls]
 extern int vsnprintf(char *str, size_t sz, const char *fmt, va_list ap);
            ^~~~~~~~~
In file included from /workspace/src/stand/efi/boot1/boot1.c:28:0:
/workspace/src/stand/libsa/stand.h:276:12: note: previous declaration of 'vsnprintf' was here
 extern int vsnprintf(char *buf, size_t size, const char *cfmt, __va_list);
            ^~~~~~~~~

I see most files already include stand.h so I suspect we can remove redeclaration in boot_module.h (or have a #ifndef guard?)

Test Plan

make CROSS_TOOLCHAIN=amd64-gcc buildworld buildkernel

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

yea, this never should have been in boot_module.h to start with.

This revision is now accepted and ready to land.Jun 12 2019, 9:52 PM
This revision was automatically updated to reflect the committed changes.