react-native-community / jsc-android-buildscripts

Script for building JavaScriptCore for Android (for React Native but not only)

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

statvfs build error

guzhonL opened this issue · comments

Issue Description

Hi, When I built 245459.0.0 myself on branch master, I got a build error like this:
/Users/guizhong/dev/workspace/jsc-android-buildscripts/build/target/webkit/Source/WTF/wtf/posix/FileSystemPOSIX.cpp:422:9: error: no matching constructor for initialization of 'statvfs'
_if (statvfs(fileSystemRepresentation(path).data(), &fileSystemStat)) {
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~_

/Users/guizhong/dev/workspace/20190401/Android/WBox/wbxjscore/jsc-android-buildscripts/build/target/toolchains/arm-linux-androideabi/sysroot/usr/include/sys/statvfs.h:46:8: note: candidate constructor (the implicit copy constructor) not viable: requires 1 argument, but 2 were provided
struct statvfs { __STATVFS64_BODY };
^

/Users/guizhong/dev/workspace/jsc-android-buildscripts/build/target/toolchains/arm-linux-androideabi/sysroot/usr/include/sys/statvfs.h:46:8: note: candidate constructor (the implicit move constructor) not viable: requires 1 argument, but 2 were provided
/Users/guizhong/dev/workspace/jsc-android-buildscripts/build/target/toolchains/arm-linux-androideabi/sysroot/usr/include/sys/statvfs.h:46:8: note: candidate constructor (the implicit default constructor) not viable: requires 0 arguments, but 2 were provided
1 error generated.
And "jsc_fix_build_error_statvfs.patch" applied. Would you give me a clue how to fix it? It's the best way that you could publish a new tag on version 245459.0.0. Thanks!!

Version, config, any additional info

ndk version: ndk-r17c
os info:macOs Mojave