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

Fedora is not recognized by the install_deps.sh script #5512

Closed
corollari opened this issue Nov 27, 2018 · 2 comments
Closed

Fedora is not recognized by the install_deps.sh script #5512

corollari opened this issue Nov 27, 2018 · 2 comments

Comments

@corollari
Copy link
Contributor

Description

Fedora is not recognized by the install_deps.sh script.

Environment

  • Operating system: Fedora 29

Steps to Reproduce

On a Fedora 29 install, run the following code from the terminal:

./scripts/install_deps.sh

Output should be:

Installing solidity dependencies on Fedora.
...

Output is:

ERROR - Unsupported or unidentified Linux distro.
See http://solidity.readthedocs.io/en/latest/installing-solidity.html for manual instructions.
If you would like to get your distro working, that would be fantastic.
Drop us a message at https://gitter.im/ethereum/solidity-dev.
corollari added a commit to corollari/solidity that referenced this issue Nov 27, 2018
Make the double quotes around the distro name optional so it matches properly in Fedora (`NAME=Fedora`)
@chriseth
Copy link
Contributor

Thanks for letting us know! Would you be able to extend the script so that it works for Fedora?

@chriseth
Copy link
Contributor

Ah sorry, only saw the pull request just now :)

chriseth pushed a commit to corollari/solidity that referenced this issue Nov 28, 2018
Make the double quotes around the distro name optional so it matches properly in Fedora (`NAME=Fedora`)
ChrisChinchilla pushed a commit that referenced this issue Nov 30, 2018
Make the double quotes around the distro name optional so it matches properly in Fedora (`NAME=Fedora`)
ChrisChinchilla pushed a commit that referenced this issue Nov 30, 2018
Make the double quotes around the distro name optional so it matches properly in Fedora (`NAME=Fedora`)
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

No branches or pull requests

2 participants