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

Support install to other dir and package to one or multi packages #1788

Merged
merged 1 commit into from
Mar 18, 2020
Merged

Support install to other dir and package to one or multi packages #1788

merged 1 commit into from
Mar 18, 2020

Conversation

laura-ding
Copy link
Contributor

@laura-ding laura-ding commented Feb 18, 2020

  • If you want to install rpm package to other dir
rpm -ivh --prefix=${your_dir} nebula-graph-${version}.rpm
  • If you want to package nebula to one package
cd nebula/package
./package.sh -v <version>
  • If you want to package nebula to multi packages
cd nebula/package
./package.sh -v <version> -n OFF

fix #1783

@amber-moe amber-moe self-assigned this Feb 19, 2020
@jude-zhu jude-zhu requested review from dutor, yixinglu and critical27 and removed request for dutor and yixinglu February 20, 2020 10:11
@jude-zhu jude-zhu added the ready-for-testing PR: ready for the CI test label Feb 20, 2020
yixinglu
yixinglu previously approved these changes Mar 2, 2020
Copy link
Contributor

@yixinglu yixinglu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jude-zhu jude-zhu requested a review from Shylock-Hg March 3, 2020 09:19
@amber-moe amber-moe mentioned this pull request Mar 12, 2020
Support package one or multi packages
Copy link
Contributor

@dutor dutor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dutor dutor merged commit 1e9423a into vesoft-inc:master Mar 18, 2020
@codecov-io
Copy link

Codecov Report

Merging #1788 into master will decrease coverage by 0.03%.
The diff coverage is 88.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1788      +/-   ##
==========================================
- Coverage   86.90%   86.86%   -0.04%     
==========================================
  Files         636      636              
  Lines       59819    59837      +18     
==========================================
- Hits        51984    51979       -5     
- Misses       7835     7858      +23     
Impacted Files Coverage Δ
src/graph/GoExecutor.cpp 75.81% <0.00%> (+0.36%) ⬆️
src/kvstore/raftex/RaftPart.cpp 76.88% <ø> (-1.06%) ⬇️
src/meta/processors/admin/BalancePlan.h 52.63% <ø> (ø)
src/meta/processors/admin/BalanceProcessor.cpp 0.00% <0.00%> (ø)
src/meta/processors/admin/BalanceTask.h 83.33% <ø> (ø)
src/meta/processors/admin/Balancer.h 75.00% <ø> (ø)
src/storage/admin/AdminProcessor.h 68.61% <38.46%> (-1.68%) ⬇️
src/common/filter/Expressions.cpp 87.44% <52.63%> (-0.25%) ⬇️
src/meta/test/BalancerTest.cpp 97.66% <97.92%> (ø)
src/graph/test/GoTest.cpp 96.80% <100.00%> (ø)
... and 23 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7da052f...5929922. Read the comment docs.

tong-hao pushed a commit to tong-hao/nebula that referenced this pull request Jun 1, 2021
Support package one or multi packages
yixinglu pushed a commit to yixinglu/nebula that referenced this pull request Jan 31, 2023
* Fix test case.

* Reorganize category of audit log sentence.

* Fix compile error.

* Fix show grants display.

Co-authored-by: Sophie <84560950+Sophie-Xie@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support specifying installation directory by prefix option when installing through nebula package
6 participants