Page MenuHomeFreeBSD

Fix gcc build
ClosedPublic

Authored by lwhsu on Jun 12 2019, 1:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 19, 3:06 PM
Unknown Object (File)
Sun, Dec 15, 2:52 PM
Unknown Object (File)
Thu, Dec 12, 7:18 PM
Unknown Object (File)
Tue, Dec 10, 5:27 PM
Unknown Object (File)
Sun, Dec 8, 10:25 AM
Unknown Object (File)
Sun, Dec 8, 10:20 AM
Unknown Object (File)
Sun, Dec 8, 10:20 AM
Unknown Object (File)
Fri, Dec 6, 1:50 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.