Bug 2421761

Summary: Review Request: rust-tree-sitter-clojure-orchard - Parser for Clojure files
Product: [Fedora] Fedora Reporter: Michel Lind <michel>
Component: Package ReviewAssignee: Fabio Valentini <decathorpe>
Status: ASSIGNED --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: decathorpe, package-review
Target Milestone: ---Flags: decathorpe: fedora-review?
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
URL: https://crates.io/crates/tree-sitter-clojure-orchard
Whiteboard:
Fixed In Version: Doc Type: ---
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2372928    

Description Michel Lind 2025-12-12 15:19:59 UTC
Spec URL: https://salimma.fedorapeople.org/rust-tree-sitter-clojure-orchard.spec
SRPM URL: https://salimma.fedorapeople.org/rust-tree-sitter-clojure-orchard-0.2.5-1.fc44.src.rpm

Description:
Parser for Clojure files.

Fedora Account System Username: salimma

Comment 1 Fedora Review Service 2025-12-12 15:33:37 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9903110
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2421761-rust-tree-sitter-clojure-orchard/fedora-rawhide-x86_64/09903110-rust-tree-sitter-clojure-orchard/fedora-review/review.txt

Found issues:

- No gcc, gcc-c++ or clang found in BuildRequires
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/C_and_C++/

Please know that there can be false-positives.

---
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 2025-12-22 15:29:58 UTC
The licensing here seems to be inconsistent.

The Cargo.toml license expression says "MIT" which would be OK, but the actual text in the LICENSE file is CC0-1.0, which wouldn't be.

Comment 3 Michel Lind 2025-12-22 20:40:39 UTC
Yeah, let me put up a PR changing it to MIT (which is what all the metadata files mention) and see what upstream says

Comment 4 Michel Lind 2025-12-22 21:01:28 UTC
Oh fun, the original project this is forked from is explicitly CC0-1.0 ... https://github.com/sogaiu/tree-sitter-clojure

Comment 5 Fabio Valentini 2025-12-22 22:46:18 UTC
Fun. So this probably won't be acceptable for a Fedora package, unless you want to as Fedora Legal for an exception.