Skip to content

merge to base repo#14

Open
coolsnake wants to merge 54 commits intoVistara-Labs:mainfrom
zhangwenqiangnb:main
Open

merge to base repo#14
coolsnake wants to merge 54 commits intoVistara-Labs:mainfrom
zhangwenqiangnb:main

Conversation

@coolsnake
Copy link
Copy Markdown
Contributor

No description provided.

@coolsnake coolsnake marked this pull request as draft January 12, 2024 02:32
@coolsnake coolsnake marked this pull request as ready for review January 12, 2024 05:51
Comment thread config.toml Outdated
@mmchougule
Copy link
Copy Markdown
Contributor

lgtm

Comment thread config.toml
[components.celestia.bridge]
binary = "/tmp/vimana/celestia/celestia"
binary = "/usr/local/bin/celestia/celestia"
download = "/tmp/vimana/celestia/init.sh"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't think this will work because the /usr/local/bin/celestia directory isn't created when sudo mv command executes in init.sh. We need to either create the directory or remove it from here.

Copy link
Copy Markdown
Contributor Author

@coolsnake coolsnake Jan 15, 2024

Choose a reason for hiding this comment

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

in init.sh, I have changed INTERNAL_DIR="/usr/local/bin/celestia" which will create /usr/local/bin/celestia directory

@coolsnake coolsnake marked this pull request as draft January 15, 2024 14:00
@coolsnake coolsnake marked this pull request as ready for review January 15, 2024 14:01
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.

3 participants