-
Notifications
You must be signed in to change notification settings - Fork 5k
Freemine.cmake.3.0 #30663
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
Freemine.cmake.3.0 #30663
Conversation
…right after DEP_ext_libuv
1. ext_lzma2 is only used by Linux 2. fix bug in taosd-ci-build.yml
…er than .externals
…ch(freemine.cmake.3.0) has not include taos_fetch_fields_e from upstream
…1,2} from CMakeLists.txt(s)
| TARGET_LINK_LIBRARIES( | ||
| joinTests | ||
| PRIVATE os util common executor gtest_main qcom function planner scalar nodes vnode | ||
| PRIVATE PRIVATE os util common executor qcom function planner scalar nodes vnode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
多了一个 PRIVATE, 下边几个也一样
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is executable target, PRIVATE/PUBLIC does not make big difference, i guess.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, it's typo, double PRIVATE
source/client/wrapper/CMakeLists.txt
Outdated
| ) | ||
|
|
||
| add_library(${TAOS_LIB_STATIC} STATIC ${WRAPPER_SRC}) | ||
| add_dependencies(${TAOS_LIB_STATIC} ${TAOS_NATIVE_LIB_STATIC}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个依赖关系是否必要
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems need to be removed
| run-tests-on-linux: | ||
| uses: taosdata/.github/.github/workflows/run-tests-on-linux.yml@main | ||
| # NOTE: using tomchon-patch-3 branch for the moment | ||
| uses: taosdata/.github/.github/workflows/run-tests-on-linux.yml@tomchon-patch-3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
需要恢复
| int err = uv_dlopen(libPath, pLib); | ||
| if (err != 0) { | ||
| fnError("can not load library %s. error: %s", libPath, uv_strerror(err)); | ||
| fnError("can not load library %s. error: %s, %s", libPath, uv_strerror(err), pLib->errmsg); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pLib 可能为空,改为 pLib ? "" : pLib->errmsg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
392: TAOS_UDF_CHECK_PTR_RCODE(libPath, pLib, funcName, func);
has already check?
| -v ${REP_REAL_PATH}/community/contrib/zlib/:${REP_DIR}/community/contrib/zlib \ | ||
| -v ${REP_REAL_PATH}/community/contrib/zstd/:${REP_DIR}/community/contrib/zstd \ | ||
| --rm --ulimit core=-1 taos_test:v1.0 sh -c "cd $REP_DIR; rm -rf debug; mkdir -p debug; cd debug; cmake .. -DBUILD_HTTP=false -DBUILD_TOOLS=true -DBUILD_TEST=ON -DWEBSOCKET=true -DBUILD_TAOSX=false -DJEMALLOC_ENABLED=OFF -DCMAKE_EXPORT_COMPILE_COMMANDS=1 ;make -j|| exit 1" | ||
| --rm --ulimit core=-1 taos_test:v1.0 sh -c "cd $REP_DIR; rm -rf debug; mkdir -p debug; cd debug; cmake .. -DBUILD_HTTP=false -DBUILD_TOOLS=true -DBUILD_TEST=ON -DWEBSOCKET=true -DBUILD_TAOSX=false -DJEMALLOC_ENABLED=OFF -DCMAKE_EXPORT_COMPILE_COMMANDS=1 ;make VERBOSE=1 -j|| exit 1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
是否需要恢复
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
| -v ${REP_REAL_PATH}/community/contrib/zlib/:${REP_DIR}/community/contrib/zlib \ | ||
| -v ${REP_REAL_PATH}/community/contrib/zstd/:${REP_DIR}/community/contrib/zstd \ | ||
| --rm --ulimit core=-1 taos_test:v1.0 sh -c "cd $REP_DIR; rm -rf debug; mkdir -p debug; cd debug; cmake .. -DBUILD_HTTP=internal -DBUILD_TOOLS=true -DBUILD_TEST=ON -DWEBSOCKET=true -DBUILD_SANITIZER=1 -DTOOLS_SANITIZE=true -DCMAKE_BUILD_TYPE=Debug -DTOOLS_BUILD_TYPE=Debug -DBUILD_TAOSX=false -DJEMALLOC_ENABLED=OFF; make -j|| exit 1 " | ||
| --rm --ulimit core=-1 taos_test:v1.0 sh -c "cd $REP_DIR; rm -rf debug; mkdir -p debug; cd debug; cmake .. -DBUILD_HTTP=internal -DBUILD_TOOLS=true -DBUILD_TEST=ON -DWEBSOCKET=true -DBUILD_SANITIZER=1 -DTOOLS_SANITIZE=true -DCMAKE_BUILD_TYPE=Debug -DTOOLS_BUILD_TYPE=Debug -DBUILD_TAOSX=false -DJEMALLOC_ENABLED=OFF; make VERBOSE=1 -j|| exit 1 " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
是否需要恢复?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
tests/parallel_test/run.sh
Outdated
| if [ $ret -eq 0 ]; then | ||
| break | ||
| fi | ||
| echo ========================== |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
该文件中的所有 echo 需要恢复
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
tests/parallel_test/run_case.sh
Outdated
| fi | ||
|
|
||
| if [ $ent -eq 0 ]; then | ||
| echo "community edition init env" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
该文件中的所有 echo 及其他修改,需要恢复
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
tests/script/sh/compile_udf.sh
Outdated
|
|
||
| # NOTE: debug for the moment | ||
| date | ||
| pwd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
该文件中的所有 echo 及其他修改,是否需要恢复
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
_GNU_SOURCE with guard
Description
#29936
Checklist
Please check the items in the checklist if applicable.