bisakhmondal commented on a change in pull request #6213: URL: https://github.com/apache/apisix/pull/6213#discussion_r801302373
########## File path: docs/en/latest/how-to-build.md ########## @@ -120,12 +120,12 @@ This step is required if you have installed only Apache APISIX via RPM, Docker o You can install ETCD via Docker or binary etc. The following command installs ETCD via binary. ```shell -$ ETCD_VERSION='3.4.13' -$ wget https://github.com/etcd-io/etcd/releases/download/v${ETCD_VERSION}/etcd-v${ETCD_VERSION}-linux-amd64.tar.gz -$ tar -xvf etcd-v${ETCD_VERSION}-linux-amd64.tar.gz && \ - cd etcd-v${ETCD_VERSION}-linux-amd64 && \ - sudo cp -a etcd etcdctl /usr/bin/ -$ nohup etcd & +ETCD_VERSION='3.4.13' +wget https://github.com/etcd-io/etcd/releases/download/v${ETCD_VERSION}/etcd-v${ETCD_VERSION}-linux-amd64.tar.gz +tar -xvf etcd-v${ETCD_VERSION}-linux-amd64.tar.gz && \ + cd etcd-v${ETCD_VERSION}-linux-amd64 && \ + sudo cp -a etcd etcdctl /usr/bin/ +nohup etcd >/tmp/etcd.log 2>&1 & Review comment: nice ########## File path: docs/en/latest/how-to-build.md ########## @@ -185,23 +185,29 @@ Run the `apisix help` command to see the returned results and get commands and d ```shell # more actions find by `help` -$ apisix help +apisix help ``` ## Step 4: Run Test Cases 1. Install `cpanminus`, the package manager for `perl`. +```shell +cd /bin +curl -L https://cpanmin.us/ -o cpanm +chmod +x cpanm Review comment: ```suggestion curl -L https://cpanmin.us/ -o cpanm chmod +x cpanm # optional mv cpanm /bin ``` This should be a suggestion, not a strict requirement? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
