Skip to content

Conversation

@Guelakais
Copy link

the added flake.nix is there to be able to build this package in nix environments.
A Cargo.lock file is urgently required for certain packet managers. Therefore, the corresponding entry in the .gitignore has been removed to enable its generation.

@Guelakais Guelakais force-pushed the main branch 3 times, most recently from afaebdb to 16517e3 Compare April 16, 2025 16:04
@Guelakais Guelakais force-pushed the main branch 2 times, most recently from 4bdc809 to a074259 Compare April 29, 2025 06:31
@esteve
Copy link
Contributor

esteve commented May 11, 2025

@Guelakais please avoid adding any changes that are unrelated to the goal of this PR. You mentioned that you're reenabling Cargo.lock, but at the same time you've added changes to the source code and to the toolchain, please revert those.

I've also seen the PR at NixOS/nixpkgs#398799 in the nixpkgs repository, my suggestion in general is that you should listen more to the maintainers of a project and not take feedback personally, not everyone has as much patience as we have demonstrated multiple times in this project.

@Guelakais Guelakais changed the title added nix development environment and Cargo.lock added nix development environment, Cargo.lock, Edition and dependency update May 12, 2025
@Guelakais Guelakais changed the title added nix development environment, Cargo.lock, Edition and dependency update added nix development environment and Cargo.lock May 12, 2025
@Guelakais Guelakais changed the title added nix development environment and Cargo.lock added nix development environment, Cargo.lock, updated toolchain May 12, 2025
@Guelakais Guelakais force-pushed the main branch 3 times, most recently from 3dfa3b8 to b78ff04 Compare May 12, 2025 14:15
Guelakais added 2 commits May 12, 2025 16:18
added rust-toolchain.toml for the nix flake

The nix flake needs a rust-toolchain.toml to rely on
Guelakais added 3 commits May 12, 2025 16:27
```rust
warning: the borrowed expression implements the required traits
   --> src/lib.rs:149:23
    |
149 |         std::fs::copy(&src, &dest).with_context(|| {
    |                       ^^^^ help: change this to: `src`
    |
    = help: for further information visit
https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
    = note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by
default

warning: `cargo-ament-build` (lib) generated 1 warning (run `cargo
clippy --fix --lib -p cargo-ament-build` to apply 1 suggestion)
```
@Guelakais Guelakais changed the title added nix development environment, Cargo.lock, updated toolchain added nix development environment, Cargo.lock, edition update, dependency update May 12, 2025
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.

2 participants