Remove tools entries from spec dependencies #723
Comments
jw3
changed the title
jw3
added a commit
that referenced
this issue
Dec 30, 2022
Addresses findings from the first Fedora review. Also includes some other distantly related, but supportive, changes. Addresses several findings from first Fedora review (#715): - Adding desktop file - Adding man page - Proper installation of translated help - Cleaning up lints in the spec - Reduces rpm deps by removing tool-only deps Also - Upgrades Clap - Makes the top level Rust project a virtual project - Makes crates/tools a standalone crate - Upgrades some Actions - Adds a checksum to wf to aid in debugging Closes #616 Closes #722 Closes #723 # Conflicts: # Cargo.toml # crates/pyo3/src/check.rs # crates/tools/src/fapolicy_profiler.rs # fapolicy-analyzer.spec # help/__init__.py # scripts/srpm/fapolicy-analyzer.spec
jw3
added a commit
that referenced
this issue
Dec 30, 2022
Addresses findings from the first Fedora review. Also includes some other distantly related, but supportive, changes. Addresses several findings from first Fedora review (#715): - Adding desktop file - Adding man page - Proper installation of translated help - Cleaning up lints in the spec - Reduces rpm deps by removing tool-only deps Also - Upgrades Clap - Makes the top level Rust project a virtual project - Makes crates/tools a standalone crate - Upgrades some Actions - Adds a checksum to wf to aid in debugging Closes #616 Closes #722 Closes #723 # Conflicts: # Cargo.toml # crates/pyo3/src/check.rs # crates/tools/src/fapolicy_profiler.rs # fapolicy-analyzer.spec # help/__init__.py # scripts/srpm/fapolicy-analyzer.spec
jw3
added a commit
that referenced
this issue
Jan 2, 2023
Addresses findings from the first Fedora review. Also includes some other distantly related, but supportive, changes. Addresses several findings from first Fedora review (#715): - Adding desktop file - Adding man page - Proper installation of translated help - Cleaning up lints in the spec - Reduces rpm deps by removing tool-only deps Also - Upgrades Clap - Makes the top level Rust project a virtual project - Makes crates/tools a standalone crate - Upgrades some Actions - Adds a checksum to wf to aid in debugging Closes #616 Closes #722 Closes #723 # Conflicts: # Cargo.toml # crates/pyo3/src/check.rs # crates/tools/src/fapolicy_profiler.rs # fapolicy-analyzer.spec # help/__init__.py # scripts/srpm/fapolicy-analyzer.spec
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The fapolicy-tools crate is built by request-only, not during the RPM build. There are dependencies in the spec file that are taken from the global lock file which includes these tool-only deps.
Quick and ephemeral way to resolve this is to exclude the tools from the workspace, generate new lock file, run lock2spec.
The text was updated successfully, but these errors were encountered: