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.

Sign up for GitHub

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

Jump to bottom

Fix folder permission check #290

Merged
merged 1 commit into from
Jan 22, 2024
Merged

Fix folder permission check #290

merged 1 commit into from
Jan 22, 2024

Conversation

WadeBarnes
Copy link
Member

  • Affecting MAC and Linux

@WadeBarnes
Fix folder permission check
95db705 
- Affecting MAC and Linux

Signed-off-by: Wade Barnes <[email protected]>
@WadeBarnes WadeBarnes requested review from swcurran and esune January 22, 2024 16:10
@WadeBarnes
Copy link
Member Author

Fixes: #194

@WadeBarnes WadeBarnes linked an issue Jan 22, 2024 that may be closed by this pull request
Permission Denied on executing Indy-CLI scripts #194
Closed
@WadeBarnes WadeBarnes mentioned this pull request Jan 22, 2024
Closed
@WadeBarnes
Copy link
Member Author

Tested on Windows (git-bash), MAC, and WSL.

@WadeBarnes
Copy link
Member Author

WadeBarnes commented Jan 22, 2024 •
edited
Loading

The issues impact MAC and Linux, and appear when running indy-cli templated batch scripts such as:

./manage \
  indy-cli create-wallet \
  walletName=local_net_trustee_wallet

Or any of the others like, apply-taa, attach-wallet, did-list, start-session, etc., since a copy of the script is written out to the cli-scripts directory containing the runtime parameter substitutions. These show up as .run files.

@esune esune merged commit 7ed127c into bcgov:main Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@esune esune esune approved these changes

@swcurran swcurran Awaiting requested review from swcurran

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Permission Denied on executing Indy-CLI scripts
2 participants
@WadeBarnes @esune