Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GitHub: Silence clang-12 cross builds #668

Closed
wants to merge 2 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/cross-bootstrap-tools.yml
Expand Up @@ -2,7 +2,7 @@ name: Cross-build Kernel

on:
push:
branches: [ main, 'stable/13' ]
branches: [ main, 'stable/13', silence-github-clang-12 ]
pull_request:
branches: [ main ]

Expand Down
10 changes: 1 addition & 9 deletions tools/build/cross-build/include/linux/unistd.h
Expand Up @@ -41,17 +41,9 @@
/* Ensure that unistd.h pulls in getopt */
#define __USE_POSIX2
#endif
/*
* Before version 2.25, glibc's unistd.h would define the POSIX subset of
* getopt.h by defining __need_getopt, including getopt.h (which would
* disable the header guard) and then undefining it so later including
* getopt.h explicitly would define the extensions. However, we wrap getopt,
* and so the wrapper's #pragma once breaks that. Thus getopt.h must be
* included before the real unistd.h to ensure we get all the extensions.
*/
#include <getopt.h>
#include_next <unistd.h>
#include <fcntl.h>
#include <getopt.h>
#include <stdlib.h>
#include <string.h>
#include <sys/syscall.h>
Expand Down