Bug 2259225 - Review Request: rust-cached - Generic cache implementations and simplified function memoization
Summary: Review Request: rust-cached - Generic cache implementations and simplified fu...
Keywords:
Status: CLOSED DUPLICATE of bug 2290308
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/cached
Whiteboard:
Depends On: 2259210 2259222
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-01-19 16:17 UTC by Daniel Mellado
Modified: 2024-10-13 20:15 UTC (History)
3 users (show)

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


Attachments (Terms of Use)
The .spec file difference from Copr build 6922122 to 6946486 (348 bytes, patch)
2024-01-24 12:11 UTC, Fedora Review Service
no flags Details | Diff
The .spec file difference from Copr build 6983938 to 7034702 (2.82 KB, patch)
2024-02-19 13:11 UTC, Fedora Review Service
no flags Details | Diff

Description Daniel Mellado 2024-01-19 16:17:20 UTC
Spec URL: https://dmellado.fedorapeople.org/rust-cached.spec
SRPM URL: https://dmellado.fedorapeople.org/rust-cached-0.47.0-1.fc39.src.rpm
Description: Generic cache implementations and simplified function memoization.
Fedora Account System Username: dmellado

Comment 1 Daniel Mellado 2024-01-19 16:21:51 UTC
Note that Copr build will fail until the dependencies' issues are merged

Comment 2 Fedora Review Service 2024-01-19 16:23:21 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6922122
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2259225-rust-cached/fedora-rawhide-x86_64/06922122-rust-cached/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
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 3 Daniel Mellado 2024-01-24 12:07:11 UTC
Updating build to the new version containing the license fix for the subpackages listed as dependent. Will still be failing to build until those are merged and available

Comment 4 Daniel Mellado 2024-01-24 12:08:28 UTC
Spec URL: https://dmellado.fedorapeople.org/rust-cached.spec
SRPM URL: https://dmellado.fedorapeople.org/rust-cached-0.48.1-1.fc39.src.rpm
Description: Generic cache implementations and simplified function memoization.
Fedora Account System Username: dmellado

Comment 5 Fedora Review Service 2024-01-24 12:11:09 UTC
Created attachment 2010164 [details]
The .spec file difference from Copr build 6922122 to 6946486

Comment 6 Fedora Review Service 2024-01-24 12:11:12 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6946486
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2259225-rust-cached/fedora-rawhide-x86_64/06946486-rust-cached/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
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 7 Daniel Mellado 2024-01-30 10:52:57 UTC
[fedora-review-service-build]

Comment 8 Fedora Review Service 2024-01-30 10:55:52 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6973113
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2259225-rust-cached/fedora-rawhide-x86_64/06973113-rust-cached/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
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 9 Daniel Mellado 2024-01-30 11:17:43 UTC
Even if the packages have been built for rawhide already it may take some time for them to propagate to the servers, so this error is still showing up:

Problem 1: nothing provides requested (crate(cached_proc_macro/default) >= 0.19.1 with crate(cached_proc_macro/default) < 0.20.0~)
Problem 2: nothing provides requested (crate(cached_proc_macro_types/default) >= 0.1.1 with crate(cached_proc_macro_types/default) < 0.2.0~)

I'll wait for the packages to be propagated and then retrigger this, thanks!

Comment 10 Daniel Mellado 2024-01-31 15:05:10 UTC
[fedora-review-service-build]

Comment 11 Fedora Review Service 2024-01-31 15:07:50 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6976098
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2259225-rust-cached/fedora-rawhide-x86_64/06976098-rust-cached/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
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 12 Daniel Mellado 2024-02-03 06:48:41 UTC
[fedora-review-service-build]

Comment 13 Fedora Review Service 2024-02-03 20:02:03 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6983938
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2259225-rust-cached/fedora-rawhide-x86_64/06983938-rust-cached/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 14 Fabio Valentini 2024-02-04 22:05:48 UTC
Some of the built packages don't install:

Failed to resolve the transaction:
Problem 1: conflicting requests
  - nothing provides (crate(redis/ahash) >= 0.24.0 with crate(redis/ahash) < 0.25.0~) needed by rust-cached+redis_ahash-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/default) >= 0.24.0 with crate(redis/default) < 0.25.0~) needed by rust-cached+redis_ahash-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/r2d2) >= 0.24.0 with crate(redis/r2d2) < 0.25.0~) needed by rust-cached+redis_ahash-devel-0.48.1-1.fc40.noarch
 Problem 2: conflicting requests
  - nothing provides (crate(redis/default) >= 0.24.0 with crate(redis/default) < 0.25.0~) needed by rust-cached+redis_async_std-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/r2d2) >= 0.24.0 with crate(redis/r2d2) < 0.25.0~) needed by rust-cached+redis_async_std-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/aio) >= 0.24.0 with crate(redis/aio) < 0.25.0~) needed by rust-cached+redis_async_std-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/async-std-comp) >= 0.24.0 with crate(redis/async-std-comp) < 0.25.0~) needed by rust-cached+redis_async_std-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/async-std-tls-comp) >= 0.24.0 with crate(redis/async-std-tls-comp) < 0.25.0~) needed by rust-cached+redis_async_std-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/tls) >= 0.24.0 with crate(redis/tls) < 0.25.0~) needed by rust-cached+redis_async_std-devel-0.48.1-1.fc40.noarch
 Problem 3: conflicting requests
  - nothing provides (crate(redis/default) >= 0.24.0 with crate(redis/default) < 0.25.0~) needed by rust-cached+redis_connection_manager-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/r2d2) >= 0.24.0 with crate(redis/r2d2) < 0.25.0~) needed by rust-cached+redis_connection_manager-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/connection-manager) >= 0.24.0 with crate(redis/connection-manager) < 0.25.0~) needed by rust-cached+redis_connection_manager-devel-0.48.1-1.fc40.noarch
 Problem 4: conflicting requests
  - nothing provides (crate(redis/default) >= 0.24.0 with crate(redis/default) < 0.25.0~) needed by rust-cached+redis-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/r2d2) >= 0.24.0 with crate(redis/r2d2) < 0.25.0~) needed by rust-cached+redis-devel-0.48.1-1.fc40.noarch
 Problem 5: conflicting requests
  - nothing provides (crate(redis/default) >= 0.24.0 with crate(redis/default) < 0.25.0~) needed by rust-cached+redis_tokio-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/r2d2) >= 0.24.0 with crate(redis/r2d2) < 0.25.0~) needed by rust-cached+redis_tokio-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/aio) >= 0.24.0 with crate(redis/aio) < 0.25.0~) needed by rust-cached+redis_tokio-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/tls) >= 0.24.0 with crate(redis/tls) < 0.25.0~) needed by rust-cached+redis_tokio-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/tokio-comp) >= 0.24.0 with crate(redis/tokio-comp) < 0.25.0~) needed by rust-cached+redis_tokio-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/tokio-native-tls-comp) >= 0.24.0 with crate(redis/tokio-native-tls-comp) < 0.25.0~) needed by rust-cached+redis_tokio-devel-0.48.1-1.fc40.noarch
 Problem 6: conflicting requests
  - nothing provides (crate(instant/wasm-bindgen) >= 0.1.0 with crate(instant/wasm-bindgen) < 0.2.0~) needed by rust-cached+wasm-devel-0.48.1-1.fc40.noarch
 Problem 7: package rust-cached+redis_store-devel-0.48.1-1.fc40.noarch requires crate(cached/redis) = 0.48.1, but none of the providers can be installed
  - conflicting requests
  - nothing provides (crate(redis/default) >= 0.24.0 with crate(redis/default) < 0.25.0~) needed by rust-cached+redis-devel-0.48.1-1.fc40.noarch
  - nothing provides (crate(redis/r2d2) >= 0.24.0 with crate(redis/r2d2) < 0.25.0~) needed by rust-cached+redis-devel-0.48.1-1.fc40.noarch

It looks like you would either need to drop the features that depend on the "redis" crate, or package redis ^0.24.

Comment 15 Daniel Mellado 2024-02-19 13:06:01 UTC
Spec URL: https://dmellado.fedorapeople.org/rust-cached.spec
SRPM URL: https://dmellado.fedorapeople.org/rust-cached-0.48.1-1.fc39.src.rpm
Description: Generic cache implementations and simplified function memoization.
Fedora Account System Username: dmellado

Comment 16 Fedora Review Service 2024-02-19 13:11:02 UTC
Created attachment 2017658 [details]
The .spec file difference from Copr build 6983938 to 7034702

Comment 17 Fedora Review Service 2024-02-19 13:11:05 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7034702
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2259225-rust-cached/fedora-rawhide-x86_64/07034702-rust-cached/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 18 Fabio Valentini 2024-02-28 21:19:53 UTC
Sorry for the delay here. I'm just now working through my backlog.

===

Looks like there's still a subpackage that does not install:

| Problem: conflicting requests
|   - nothing provides (crate(instant/wasm-bindgen) >= 0.1.0 with crate(instant/wasm-bindgen) < 0.2.0~) needed by rust-cached+wasm-devel-0.48.1-1.fc41.noarch

You should be able to drop the "wasm" feature as well, we don't support building for WASM in Fedora.

PS: You might want to use the mock "--postinstall" flag when doing local test builds. It would show you these issues immediately ...

Comment 19 Davide Cavalca 2024-06-03 23:14:52 UTC
I need this too so I put up an updated version in https://bugzilla.redhat.com/show_bug.cgi?id=2290308

Comment 20 Fabio Valentini 2024-10-13 20:15:27 UTC

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


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