[fix][cli] Fix start cli error

[coderzc]

Motivation

Run bin/hugegraph start error

Starting HugeGraphServer...
bin/hugegraph: line 68: /start-hugegraph.sh: No such file or directory
Failed to start service

Modifications

Add check for Variables

[codecov]

Codecov Report

Merging #2024 (3c88b2d) into master (4c232a6) will decrease coverage by 5.50%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2024      +/-   ##
============================================
- Coverage     64.38%   58.88%   -5.51%     
  Complexity      978      978              
============================================
  Files           482      482              
  Lines         41463    41463              
  Branches       5890     5890              
============================================
- Hits          26698    24414    -2284     
- Misses        12122    14616    +2494     
+ Partials       2643     2433     -210     

Impacted Files	Coverage Δ
...hugegraph/backend/store/raft/RaftStoreClosure.java	0.00% <0.00%> (-100.00%)	⬇️
...egraph/backend/serializer/BinaryEntryIterator.java	0.00% <0.00%> (-87.04%)	⬇️
...ugegraph/backend/store/rocksdb/RocksDBMetrics.java	0.00% <0.00%> (-85.87%)	⬇️
...gegraph/backend/store/rocksdb/RocksDBFeatures.java	4.16% <0.00%> (-83.34%)	⬇️
...baidu/hugegraph/backend/store/raft/RaftResult.java	0.00% <0.00%> (-81.25%)	⬇️
...aidu/hugegraph/backend/store/raft/RaftContext.java	0.00% <0.00%> (-80.96%)	⬇️
...hugegraph/backend/store/raft/RaftBackendStore.java	0.00% <0.00%> (-75.27%)	⬇️
...hugegraph/backend/store/raft/rpc/RpcForwarder.java	0.00% <0.00%> (-69.50%)	⬇️
...ugegraph/backend/store/raft/StoreStateMachine.java	0.00% <0.00%> (-65.77%)	⬇️
.../backend/store/raft/rpc/StoreCommandProcessor.java	0.00% <0.00%> (-65.22%)	⬇️
... and 60 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[imbajin]

change the comment, it requires user to set a fixed abs path manually (should't use a relative path)

[coderzc]

You are right. and I think should add check for Variables

[javeme]

we can try ${INSTALL_DIR:?"Please setting variables 'INSTALL_DIR'"}
for more details: https://stackoverflow.com/a/16753536

[imbajin]

https://stackoverflow.com/a/16753536

@javeme seems a little strange, could it use directly or test it in local evn? (or should in a if condition?)

and it alert:

failed with all cases in my ubuntu21

[imbajin]

😺, nice

typo `setting --> set``

[imbajin]

I'll enhance this script later