Bug 2362744

Summary: Review Request: openCOLLADA - Collada Exporters
Product: [Fedora] Fedora Reporter: Brigham Keys <bkeys>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: NEW --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rawhideCC: dan, package-review
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: https://github.com/RemiArnaud/OpenCOLLADA/
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:

Comment 1 Dan HorĂ¡k 2025-04-28 17:04:46 UTC
The files/directories listed in the %files sections should be more specific, see https://docs.fedoraproject.org/en-US/packaging-guidelines/#_explicit_lists

Comment 2 Brigham Keys 2025-05-03 03:43:41 UTC
Hi Dan,
I added the files explicitly like you asked:
https://download.copr.fedorainfracloud.org/results/bkeys/openCOLLADA/fedora-42-aarch64/08992617-openCOLLADA/openCOLLADA-master-2.fc42.src.rpm
Let me know if I did it right, or if there is anything else that needs correcting about the SPEC file.

Comment 3 Fedora Review Service 2025-05-03 05:28:31 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8992714
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2362744-opencollada/fedora-rawhide-x86_64/08992714-openCOLLADA/fedora-review/review.txt

Found issues:

- pcre-devel is deprecated, you must not depend on it.
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/deprecating-packages/
- A package with this name already exists. Please check https://src.fedoraproject.org/rpms/openCOLLADA
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Naming/#_conflicting_package_names

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.