Bug 2434105 - Review Request: rust-change-case - Transform a string between camelCase, PascalCase, Capital Case, and others
Summary: Review Request: rust-change-case - Transform a string between camelCase, Pasc...
Keywords:
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 2339830
TreeView+ depends on / blocked
 
Reported: 2026-01-28 18:38 UTC by Gwyn Ciesla
Modified: 2026-03-03 17:17 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Gwyn Ciesla 2026-01-28 18:38:48 UTC
SRPM:https://fedorapeople.org/~limb/review/rust-change-case/rust-change-case-0.2.0-1.fc44.src.rpm
SPEC:https://fedorapeople.org/~limb/review/rust-change-case/rust-change-case.spec

Description:
Transform a string between camelCase, PascalCase, Capital Case,
snake_case, param-case, CONSTANT_CASE and others.

Comment 1 Fedora Review Service 2026-01-28 21:55:54 UTC
There seems to be some problem with the following file.
SRPM URL: https://fedorapeople.org/~limb/review/rust-change-case/rust-change-case-0.2.0-1.fc44.src.rpm
Fetching it results in a 404 Not Found error.
Please make sure the URL is correct and publicly available.


---
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 Ben Beasley 2026-03-02 12:19:56 UTC
# FIXME: no license files detected
%doc %{crate_instdir}/README.md

This does have to be fixed, per https://docs.fedoraproject.org/en-US/packaging-guidelines/LicensingGuidelines/#_license_text.

The upstream repository https://github.com/sigoden/change-case/blob/master/Cargo.toml is archived, and the author recommends using https://crates.io/crates/convert_case instead.

I suggested switching to convert_case upstream in https://github.com/Cisco-Talos/clamav-signature-util/pull/12. Maybe you can do the same thing downstream and avoid packaging rust-change-case at all. You would have to coordinate an update of the existing rust-convert_case package, now at 0.8.0, in order to use the ccase!() macro as I did. The sole existing dependent package is rust-libspa, and it’s likely that it can be made to work with the latest rust-convert_case by simply patching its version bounds. Upstream for rust-libspa dropped the convert_case dependency altogether in https://gitlab.freedesktop.org/pipewire/pipewire-rs/-/commit/a8f365faf5292835650f9a49083e0c26c3de0bde, but that hasn’t been released yet.

Comment 3 Gwyn Ciesla 2026-03-03 17:17:27 UTC
Interesting. I've added myself to the PR, and I'll see where that goes. Thank you.


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