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

Update installing-cardano-node.md #361

Merged
merged 1 commit into from
Oct 28, 2021

Conversation

extramileit
Copy link
Contributor

Updating documentation

Description of the change

Update the bash script code from 'cabal build all' to 'cabal build cardano-node cardano-cli'. If you try to run cabal build all on latest tag of cardano-node code (i.e., 1.30.1), the build fails on tx-generator package with error message

cabal: Failed to build tx-generator-1.29 (which is required by
exe:tx-generator from tx-generator-1.29).

Since this documentation is specifically about building the cardano-node and cardano-cli, we can work around this by only building the specific executables needed.

… cardano-node and cardano-cli. Trying to run cabal build all leads to a failure preventing the node and cli build from working.
@extramileit extramileit changed the title Update installing-cardano-node.md to specifically run cabal build for… Update installing-cardano-node.md Oct 15, 2021
Copy link
Collaborator

@rdlrt rdlrt left a comment

Choose a reason for hiding this comment

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

If you try to run cabal build all on latest tag of cardano-node code (i.e., 1.30.1), the build fails on tx-generator package with error message

While the above is not true and you might meed checking your logs further to know the cause, I agree with the change to not have other components built for the scope of this guide

Copy link
Member

@katomm katomm left a comment

Choose a reason for hiding this comment

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

Thank you @extramileit

@katomm katomm merged commit c64d3a9 into cardano-foundation:staging Oct 28, 2021
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.

None yet

4 participants