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

Cleanup cargo manifest #16

Closed
wants to merge 3 commits into from
Closed

Cleanup cargo manifest #16

wants to merge 3 commits into from

Conversation

koonpeng
Copy link
Member

@koonpeng koonpeng commented Nov 7, 2023

Some minor updates to the cargo manifest, after this is merged, I intend to make a 0.1.0 release, then update osrf/nexus to use the release binaries instead of requiring users to build from source.

Something to note is that the LICENSE file is removed. Cargo recommends to use just the license field if the license is SPDX.

warning: only one of `license` or `license-file` is necessary
`license` should be used if the package license can be expressed with a standard SPDX expression.
`license-file` should be used if the package uses a non-standard license.
See https://doc.rust-lang.org/cargo/reference/manifest.html#the-license-and-license-file-fields for more information.

Teo Koon Peng added 3 commits November 6, 2023 14:36
it is not necessary when using SPDX license
```
warning: only one of `license` or `license-file` is necessary
`license` should be used if the package license can be expressed with a standard SPDX expression.
`license-file` should be used if the package uses a non-standard license.
See https://doc.rust-lang.org/cargo/reference/manifest.html#the-license-and-license-file-fields for more information.
```

Signed-off-by: Teo Koon Peng <[email protected]>
Signed-off-by: Teo Koon Peng <[email protected]>
Signed-off-by: Teo Koon Peng <[email protected]>
@koonpeng koonpeng closed this Jul 31, 2024
@koonpeng koonpeng deleted the kp/cleanup-manifest branch July 31, 2024 01:30
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.

1 participant