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

feat: quick setup script #12

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

sanketshevkar
Copy link
Member

Referring to #10

Two new shell scripts have been added:

  1. setup.sh: It checks for all dependencies for the fabric network and hlf-cicero-contract. Check if fabric-samples exists or not. If not downloads it, sets-up env variables and starts the test-network, installs the chaincode on the network.
  2. path.sh: sets up global environment variables.

Quick Setup:

mkdir hlf-cicero-setup
cd hlf-cicero-setup
git clone https://github.com/accordproject/hlf-cicero-contract.git
cd hlf-cicero-contract
./setup.sh
source ./path.sh
./initialize.sh
./trigger.sh

Copy link
Contributor

@dselman dselman left a comment

Choose a reason for hiding this comment

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

This looks awesome. Thank you!

What systems did you test this on? Could you please also update the README.md to update the instructions to use this (noting the systems that it works on)?

@sanketshevkar
Copy link
Member Author

This looks awesome. Thank you!

What systems did you test this on? Could you please also update the README.md to update the instructions to use this (noting the systems that it works on)?

I tested it on my local machine running on ubuntu 18.04. Then I tested on an ec2 instance running on ubuntu 20.04. So, I guess it should work on any debian distribution. I'll check on fedora, maybe tomorrow. If you have macOS can you check on it? I don't have much hope from windows, but I'll try running the script on it once. Till then I'll update the readme for ubuntu.

@sanketshevkar
Copy link
Member Author

@dselman it's working on fedora, so I am assuming that it'll work on other Linux distributions too. The script wasn't working on windows.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants