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

Installation of cargo-generate on Ubuntu fails owing to missing OS packages #302

Closed
rod-chapman opened this issue Oct 1, 2021 · 7 comments · Fixed by #304
Closed

Installation of cargo-generate on Ubuntu fails owing to missing OS packages #302

rod-chapman opened this issue Oct 1, 2021 · 7 comments · Fixed by #304

Comments

@rod-chapman
Copy link
Contributor

I am trying to follow the installation instructions in section 1.4 on a stock Ubuntu Beaver installation.

The "cargo install cargo-generate" command fails owing to missing OS packages "libssl-dev" and "pkg-config".

I had to do "sudo apt install libssl-dev pkg-config" before it would work, so this should probably be mentioned in section 1.4 somewhere.

@eldruin
Copy link
Member

eldruin commented Oct 6, 2021

Good catch. Could you open a PR adding this information?

@rod-chapman
Copy link
Contributor Author

I'm not sure... Section 1.4 is supposed to be OS independent, with Linux-specific stuff following in section 1.4.1, so perhaps these sections need to be re-ordered a bit? I'm not confident enough to make such a change. Who has editorial/review control over this section?

@eldruin
Copy link
Member

eldruin commented Oct 6, 2021

Yeah I understand. You can just put it as a small note right in section 1.4. Something similar to:

We'll use this later to generate a project from a template.

cargo install cargo-generate

Note: on Linux (e.g. on Ubuntu) you may need to install the packages `libssl-dev` and `pkg-config` prior to installing cargo-generate

I think having the instructions for Ubuntu is enough since it represents a majority of users AFAIK and anybody knowledgeable-enough to choose Arch or Fedora instead will probably have no problem in finding the equivalent packages and installing them.

@rod-chapman
Copy link
Contributor Author

Do I need to fork this repo first, then propose a PR from there?

@eldruin
Copy link
Member

eldruin commented Oct 6, 2021

Yes

@rod-chapman
Copy link
Contributor Author

OK.. I will give it a go...

@rod-chapman
Copy link
Contributor Author

OK... PR#304 open. Can your review and merge?

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 a pull request may close this issue.

2 participants