Summary: | tools_error2aub-error2aub fails to build: error: implicit declaration of function ‘va_start’ | ||
---|---|---|---|
Product: | Mesa | Reporter: | Paul Menzel <pmenzel+bugs.freedesktop.org> |
Component: | Other | Assignee: | mesa-dev |
Status: | RESOLVED MOVED | QA Contact: | mesa-dev |
Severity: | normal | ||
Priority: | medium | CC: | pmenzel+bugs.freedesktop.org |
Version: | 18.2 | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
Paul Menzel
2018-10-02 09:18:45 UTC
Here is the first error. ``` /dev/shm/bee-root/mesalib/mesalib-18.2.1-0/source/src/intel/tools/error2aub.c:46:4: error: implicit declaration of function ‘va_start’; did you mean ‘assert’? [-Werror=implicit-function-declaration] va_start(args, format); ^~~~~~~~ assert ``` (In reply to Lionel Landwerlin from comment #2) > Sent https://patchwork.freedesktop.org/patch/254342/ Thanks, I just wanted to upload the same change. Two nitpicks. 1. Should the include be inserted lexicographically? 2. I always prefer to have the error message in the commit message. PS: I am not very involved in Mesa, but isn’t there a build tester, which should have caught that error? (Also, why did nobody else get it?) (In reply to Paul Menzel from comment #3) > (In reply to Lionel Landwerlin from comment #2) > > Sent https://patchwork.freedesktop.org/patch/254342/ > > Thanks, I just wanted to upload the same change. Two nitpicks. > > 1. Should the include be inserted lexicographically? > 2. I always prefer to have the error message in the commit message. > > PS: I am not very involved in Mesa, but isn’t there a build tester, which > should have caught that error? (Also, why did nobody else get it?) Feel free to upload a better change, I'll review and push to master. There are build testers. Not quite sure why it wasn't caught... The only explanation I could think of is that we have different build option (we usually build a large set of drivers, not just intel ones) and that somehow pulls in a header file already including stdarg.h. -- GitLab Migration Automatic Message -- This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/mesa/mesa/issues/928. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.