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

Goal of vergen is unclear from the readme. #350

Closed
gl-yziquel opened this issue Jun 16, 2024 · 3 comments · Fixed by #357
Closed

Goal of vergen is unclear from the readme. #350

gl-yziquel opened this issue Jun 16, 2024 · 3 comments · Fixed by #357

Comments

@gl-yziquel
Copy link

gl-yziquel commented Jun 16, 2024

Hi.

I've been trying to build redox from source, and it failed when trying to compile cosmic-files, which uses vergen.

pop-os/cosmic-files#250

I ended up here trying to debug that build. Unfortunately, the readme file is not written with someone like me in mind, who tries to get into a build system from the outside. It is pretty unclear what vergen does, and the readme file does seem to assume a pretty big deal of implicit knowledge when it comes to the way vergen is to be used and the use case it is trying to solve.

Please add one or two paragraphs in the readme clarifying the rôle of this crate.

@CraZySacX
Copy link
Member

That's a good point. Just so it's documented here (I will also update the README), vergen and the associated other tools allow you to embed environment variables determined at build time into your code with the env! or option_env! macros. The vast majority of my use is to include git related data into the projects I build.

@CraZySacX
Copy link
Member

Additionally, you may be interested about the mechanics. This is found in the Build Scripts section of the cargo documentation.

@gl-yziquel
Copy link
Author

@CraZySacX thank you for taking that remark constructively.

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