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.

Sign up for GitHub

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

Jump to bottom

Include version information of C libraries in cargo --version --verbose #6275

Closed
alexcrichton opened this issue Nov 7, 2018 · 5 comments
Closed

Include version information of C libraries in cargo --version --verbose #6275

alexcrichton opened this issue Nov 7, 2018 · 5 comments
Labels
C-feature-request Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted` Command-version

Comments

@alexcrichton
Copy link
Member

Sort of like how rustc -vV will report the version of LLVM, we should do the same for Cargo! Cargo critically depends on a few C libraries, and it'd be great if we could include those in the -vV output of Cargo:

  • libgit2 - both the version number and git revision
  • curl - both the version number and git revision
  • OpenSSL - (on relevant platforms) both git and version number
  • SChannel - (on Windows) we can probably include a version here hopefully
  • SecureTransport - (on OSX) hopefully can include a version here
  • libssh2 - version + git rev
  • zlib - version + git rev

Most of this will probably require changes in upstrem crates to report this information, but it'd be great to have!
@alexcrichton alexcrichton added the C-feature-request Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted` label Nov 7, 2018
@alexcrichton
Copy link
Member Author

cc @dwijnand, you might be interested in this!

@dwijnand
Copy link
Member

dwijnand commented Nov 8, 2018

Sounds like something simple enough to do, but I've got too many things on my plate to get to this any time soon.

Still might get to it, but if anyone wants to do it (or start it seeing as there are multiple parts) feel free!

@lu-zero
Copy link
Contributor

lu-zero commented Nov 9, 2018

This would require some cooperation with pkg-config or what is used in build.rs I guess.

@lukaslueg
Copy link
Contributor

Let me shamelessly point to built, which may provide at least part of that.

@ehuss ehuss mentioned this issue Jul 14, 2020
Open
4 tasks
This was referenced Sep 27, 2021
Merged
Merged
@ehuss ehuss mentioned this issue Oct 8, 2021
Merged
bors added a commit that referenced this issue Oct 11, 2021
@bors
Auto merge of #9968 - ehuss:verbose-version, r=alexcrichton,Eh2406
c7957a7 
Add some more information to verbose version.

This adds a little bit of extra information to the verbose version command.

cc #6275 and #6161

Some examples of what it might look like:

```
cargo 1.57.0
release: 1.57.0
host: x86_64-unknown-linux-gnu
libgit2: 1.3.0 (sys:0.13.23 vendored)
libcurl: 7.79.1-DEV (sys:0.4.49+curl-7.79.1 vendored ssl:OpenSSL/1.1.1k)
ssl: OpenSSL 1.1.1k  25 Mar 2021
```

```
cargo 1.57.0
release: 1.57.0
host: x86_64-pc-windows-msvc
libgit2: 1.3.0 (sys:0.13.23 vendored)
libcurl: 7.79.1-DEV (sys:0.4.49+curl-7.79.1 vendored ssl:Schannel)
```

```
cargo 1.57.0
release: 1.57.0
host: x86_64-apple-darwin
libgit2: 1.3.0 (sys:0.13.23 vendored)
libcurl: 7.64.1 (sys:0.4.49+curl-7.79.1 system ssl:(SecureTransport) LibreSSL/2.8.3)
ssl: OpenSSL 1.1.1l  24 Aug 2021
```
@epage
Copy link
Contributor

epage commented Oct 24, 2023

Going to mark this as closed by #9968 which includes an analysis of why requested items from this issue weren't included.

@epage epage closed this as completed Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-feature-request Category: proposal for a feature. Before PR, ping rust-lang/cargo if this is not `Feature accepted` Command-version
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@ehuss @epage @alexcrichton @lu-zero @dwijnand @lukaslueg