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

Cargo.toml: add more package metadata #4

Merged
merged 2 commits into from
Jan 30, 2025

Conversation

a3f
Copy link
Contributor

@a3f a3f commented Jan 30, 2025

Running cargo bitbake to generate a bitbake recipe currently results
in a few error messages due to missing metadata:

  No package.description set in your Cargo.toml, using package.name
  No package.homepage set in your Cargo.toml, trying package.repository
  error: No package.repository set in your Cargo.toml

Fill out these gaps and add some more related keys found while reading
through the documentation at https://doc.rust-lang.org/cargo/reference/manifest.html

Cargo.toml Show resolved Hide resolved
@michaelolbrich
Copy link
Owner

Can you rebase? I'd like to run the new checks with the older rust/cargo versions as well.

a3f added 2 commits January 30, 2025 09:06
cargo 1.84.0 (66221abde 2024-11-19) moves around the [[package]] section
with no functional change.

To avoid this noise making it into commits with actual functional
difference, let's move it now.

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Running cargo bitbake to generate a bitbake recipe currently results
in a few error messages due to missing metadata:

  No package.description set in your Cargo.toml, using package.name
  No package.homepage set in your Cargo.toml, trying package.repository
  error: No package.repository set in your Cargo.toml

Fill out these gaps and add some more related keys found while reading
through the documentation at [1].

[1]: https://doc.rust-lang.org/cargo/reference/manifest.html

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
@michaelolbrich michaelolbrich merged commit 467bbbd into michaelolbrich:main Jan 30, 2025
8 checks passed
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