Bug 2259675 - Review Request: rust-sled - Lightweight high-performance pure-rust transactional embedded database
Summary: Review Request: rust-sled - Lightweight high-performance pure-rust transactio...
Keywords:
Status: CLOSED DUPLICATE of bug 2284641
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Fabio Valentini
QA Contact: Fedora Extras Quality Assurance
URL: https://crates.io/crates/sled
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-01-22 17:29 UTC by Daniel Mellado
Modified: 2024-10-13 20:14 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-10-13 20:14:50 UTC
Type: ---
Embargoed:
decathorpe: fedora-review?


Attachments (Terms of Use)

Description Daniel Mellado 2024-01-22 17:29:16 UTC
Spec URL: https://dmellado.fedorapeople.org/rust-sled.spec
SRPM URL: https://dmellado.fedorapeople.org/rust-sled-0.34.7-1.fc39.src.rpm
Description: Lightweight high-performance pure-rust transactional embedded database.
Fedora Account System Username: dmellado

Comment 1 Fedora Review Service 2024-01-22 17:41:02 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6940667
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2259675-rust-sled/fedora-rawhide-x86_64/06940667-rust-sled/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Fabio Valentini 2024-01-25 21:50:37 UTC
There are a few unsatisfiable optional dependencies:

Failed to resolve the transaction:
Problem 1: conflicting requests
  - nothing provides (crate(rio/default) >= 0.9.4 with crate(rio/default) < 0.10.0~) needed by rust-sled+rio-devel-0.34.7-1.fc40.noarch
 Problem 2: conflicting requests
  - nothing provides (crate(zstd/default) >= 0.9.0 with crate(zstd/default) < 0.10.0~) needed by rust-sled+zstd-devel-0.34.7-1.fc40.noarch
 Problem 3: package rust-sled+compression-devel-0.34.7-1.fc40.noarch requires crate(sled/zstd) = 0.34.7, but none of the providers can be installed
  - conflicting requests
  - nothing provides (crate(zstd/default) >= 0.9.0 with crate(zstd/default) < 0.10.0~) needed by rust-sled+zstd-devel-0.34.7-1.fc40.noarch
 Problem 4: package rust-sled+io_uring-devel-0.34.7-1.fc40.noarch requires crate(sled/rio) = 0.34.7, but none of the providers can be installed
  - conflicting requests
  - nothing provides (crate(rio/default) >= 0.9.4 with crate(rio/default) < 0.10.0~) needed by rust-sled+rio-devel-0.34.7-1.fc40.noarch
 Problem 5: package rust-sled+testing-devel-0.34.7-1.fc40.noarch requires crate(sled/compression) = 0.34.7, but none of the providers can be installed
  - package rust-sled+compression-devel-0.34.7-1.fc40.noarch requires crate(sled/zstd) = 0.34.7, but none of the providers can be installed
  - conflicting requests
  - nothing provides (crate(zstd/default) >= 0.9.0 with crate(zstd/default) < 0.10.0~) needed by rust-sled+zstd-devel-0.34.7-1.fc40.noarch

Looks like these would be used for io_uring and / or zstd compression support. Do you need these non-default / optional features?

If you don't need them, they could bet disabled (patched out / hidden from the package). If you *do* need them, the "rio" crate would need to be packaged, and the dependency on the "zstd" crate would need to be updated to a version that is available in Fedora.

Comment 3 Daniel Mellado 2024-02-22 11:12:40 UTC
Hmm besides that there seems to be still some issues, let me check those

Problem 1: nothing provides requested (crate(rand_distr/default) >= 0.3.0 with crate(rand_distr/default) < 0.4.0~)
 Problem 2: nothing provides requested (crate(zerocopy/default) >= 0.3.0 with crate(zerocopy/default) < 0.4.0~)

Comment 4 Daniel Mellado 2024-02-22 11:35:30 UTC
I'll remove this as well, as they're just dev dependencies.

Comment 5 Fabio Valentini 2024-02-28 21:26:41 UTC
I tried to review this today, but the package no longer compiles. Looks like something that was previously just a warning is now a hard error:

error: unnecessary qualification
   --> src/config.rs:276:15
    |
276 |         <T as std::convert::TryFrom<usize>>::Error: Debug,
    |               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
note: the lint level is defined here
   --> src/lib.rs:95:5
    |
95  |     unused_qualifications
    |     ^^^^^^^^^^^^^^^^^^^^^
help: remove the unnecessary path segments
    |
276 -         <T as std::convert::TryFrom<usize>>::Error: Debug,
276 +         <T as TryFrom<usize>>::Error: Debug,
    |

Comment 6 Fabio Valentini 2024-03-13 13:12:40 UTC
The error from the previous comment was due to compiler flag changes that have been reverted.

Now it looks like the issues pointed out in Comment 2 and 3 are still present.

Comment 7 Davide Cavalca 2024-06-03 17:37:22 UTC
I need this too and put up an updated version in https://bugzilla.redhat.com/show_bug.cgi?id=2284641

Comment 8 Fabio Valentini 2024-10-13 20:14:50 UTC

*** This bug has been marked as a duplicate of bug 2284641 ***


Note You need to log in before you can comment on or make changes to this bug.