-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
add nebula-console package action #3905
Conversation
efe7d3f
to
7c59a2e
Compare
03c8ee1
to
c66f55d
Compare
31e8c1d
to
5710145
Compare
CMakeLists.txt
Outdated
@@ -25,6 +25,7 @@ project("Nebula Graph" C CXX) | |||
|
|||
option(ENABLE_PACK_ONE "Whether to package into one" ON) | |||
option(ENABLE_VERBOSE_BISON "Enable Bison to report state" OFF) | |||
option(ENABLE_CONSOLE_COMPILATION "Enable nebula-console comppilation" OFF) |
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.
typo: comppilation
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.
Fixed.
|
||
add_custom_command( | ||
OUTPUT ${CMAKE_BINARY_DIR}/nebula-console | ||
COMMAND bash ${CMAKE_SOURCE_DIR}/src/console/compile-console.sh ${git_branch} ${CMAKE_BINARY_DIR} |
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.
what happens if there not exists related branch in nebula-console repo?
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.
Compilation failed.
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.
Controlled by the cmake option ENABLE_CONSOLE_COMPILATION
.
fix nebula-console cpack fix small change trigger console compile fix typo
What type of PR is this?
What problem(s) does this PR solve?
Issue(s) number:
#3173
Description:
Package nebula-console into an RPM /deb package through action.
Checklist:
Tests:
Affects:
Release notes: