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 getting started to include MacOS #4952

Merged
merged 2 commits into from
Mar 11, 2023
Merged

Update getting started to include MacOS #4952

merged 2 commits into from
Mar 11, 2023

Conversation

Cmdv
Copy link
Contributor

@Cmdv Cmdv commented Mar 7, 2023

Adding more information in the "getting started" documentation to reflect what is present on https://developers.cardano.org/docs/get-started/installing-cardano-node/#macos

@Cmdv Cmdv requested a review from a team as a code owner March 7, 2023 20:49
@angerman
Copy link
Contributor

angerman commented Mar 8, 2023

This looks good to me; and reminds me we have a an even less involved solution for on systems where nix is installed; which I should document better. For those that have nix installed or are ok to install it, and don't want to deal with brew or similar.

$ nix develop github:input-output-hk/devx#ghc8107 --refresh --no-write-lock-file
...
 _____ _____ _____    _____         _       _ _    _____ _       _ _ 
|     |     |   __|  |  |  |___ ___| |_ ___| | |  |   __| |_ ___| | |
|-   -|  |  |  |  |  |     | .'|_ -| '_| -_| | |  |__   |   | -_| | |
|_____|_____|_____|  |__|__|__,|___|_,_|___|_|_|  |_____|_|_|___|_|_|
                                                                     
[cardano-node]$ # this one is only needed on macOS right now, due to a bug in loading libcrypto.
[cardano-node]$ export DYLD_LIBRARY_PATH=$(pkg-config --libs-only-L libcrypto|cut -c 3-) 
[cardano-node]$ # Ensure the hackage and CHaP indexes are up to date.
[cardano-node]$ cabal update
[cardano-node]$ # build cardano-cli and cardano-node
[cardano-node]$ cabal build cardano-cli cardano-node -j --ghc-options=-j
...

which works on intel, and apple silicon Macs, as well as linux, and WSL the same. (Takes about ~40min on a m2 air)

@Cmdv
Copy link
Contributor Author

Cmdv commented Mar 8, 2023

@angerman think I might have to give the nix version a go as I'm getting LLVM version recognition errors 😞

edit: I got passed that and then openSSL came to bit me.

@angerman
Copy link
Contributor

angerman commented Mar 8, 2023

@angerman think I might have to give the nix version a go as I'm getting LLVM version recognition errors 😞

edit: I got passed that and then openSSL came to bit me.

Yea, that's why input-output-hk/devx now exists. We are working on a faster bootstrap, and might at some point even try to get something that works without nix installed. For now it requires nix to be installed first :-/

It's also supposed to work with almost any IOG project; same shell works every time 🤞

@Cmdv
Copy link
Contributor Author

Cmdv commented Mar 8, 2023

Yea, that's why input-output-hk/devx now exists. We are working on a faster bootstrap, and might at some point even try to get something that works without nix installed. For now it requires nix to be installed first :-/

Interesting good to know 👍

Comment on lines +65 to +69
#### You will need to install llvm in case you are using M1

```
brew install llvm@13
```
Copy link
Contributor

Choose a reason for hiding this comment

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

This we won't need once we are on 9.2 🎉

@angerman
Copy link
Contributor

angerman commented Mar 9, 2023

@olgahryniuk @CarlosLopezDeLara, you ok with this? Can this be merged?

@angerman
Copy link
Contributor

angerman commented Mar 9, 2023

@Cmdv this will need to be rebased onto master to get the status checked green.

@Cmdv
Copy link
Contributor Author

Cmdv commented Mar 9, 2023

cool let me do that now 😄

@angerman
Copy link
Contributor

@Cmdv you'll also need to sign all the commits

The base branch requires all commits to be signed. Learn more about signing commits.

@Cmdv
Copy link
Contributor Author

Cmdv commented Mar 10, 2023

grrrrr my emacs GPG has stopped working!! going to sign the previous commits manually!!

edit: ok got it sorry about that!!!

@angerman angerman enabled auto-merge March 10, 2023 13:50
@newhoggy newhoggy self-requested a review March 10, 2023 13:53
@angerman
Copy link
Contributor

@newhoggy why doesn't this trigger any GHAs?

@newhoggy
Copy link
Contributor

I believe it's because it is from a fork which doesn't run an actions for security reasons.

@newhoggy
Copy link
Contributor

I triggered it manually.

@angerman angerman added this pull request to the merge queue Mar 11, 2023
Merged via the queue into IntersectMBO:master with commit 9e039ff Mar 11, 2023
@angerman
Copy link
Contributor

I triggered it manually.

Don't we usually get some "this is from a fork, do you want to run the actions" button?

angerman added a commit to input-output-hk/devx that referenced this pull request Mar 26, 2023
…ll (#29)

* Fix issue #26: [macOS] DYLD_LIBRARY_PATH env var needed for dev shell

c.f. IntersectMBO/cardano-node#4952 (comment)

---------

Co-authored-by: Moritz Angermann <[email protected]>
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