Bug 2227277 - Review Request: cdecl - C gibberish translator
Summary: Review Request: cdecl - C gibberish translator
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tom Rix
QA Contact: Fedora Extras Quality Assurance
URL: https://github.com/paul-j-lucas/cdecl/
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-07-28 15:24 UTC by Artur Frenszek-Iwicki
Modified: 2024-06-06 01:38 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2024-06-06 01:38:12 UTC
Type: ---
Embargoed:
trix: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 6220374 to 6362997 (2.47 KB, patch)
2023-09-01 14:02 UTC, Fedora Review Service
no flags Details | Diff
The .spec file difference from Copr build 6362997 to 6971775 (1.64 KB, patch)
2024-01-29 19:38 UTC, Fedora Review Service
no flags Details | Diff
The .spec file difference from Copr build 6971775 to 7042156 (749 bytes, patch)
2024-02-21 10:56 UTC, Fedora Review Service
no flags Details | Diff
The .spec file difference from Copr build 7042156 to 7097401 (663 bytes, patch)
2024-03-02 15:13 UTC, Fedora Review Service
no flags Details | Diff

Description Artur Frenszek-Iwicki 2023-07-28 15:24:39 UTC
spec: https://suve.fedorapeople.org/review/cdecl-14.2-1/cdecl.spec
srpm: https://suve.fedorapeople.org/review/cdecl-14.2-1/cdecl-14.2-1.fc38.src.rpm
koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=104021474

Description: cdecl is a program that translates C language declarations into pseudo-English, like "int(*fn[2])(void*)" into "declare fn as an array 2 of pointer to function (pointer to void) returning integer". It can also do the inverse, generating C declarations from English phrases.

Fedora Account System Username: suve

Comment 1 Fedora Review Service 2023-07-28 15:31:33 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6220374
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2227277-cdecl/fedora-rawhide-x86_64/06220374-cdecl/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 3 Fedora Review Service 2023-09-01 14:02:24 UTC
Created attachment 1986594 [details]
The .spec file difference from Copr build 6220374 to 6362997

Comment 4 Fedora Review Service 2023-09-01 14:02:26 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6362997
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2227277-cdecl/fedora-rawhide-x86_64/06362997-cdecl/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 7 Tom Rix 2023-12-31 13:37:46 UTC
I am concerned about the relicensing.
This is public domain.

Comment 8 Artur Frenszek-Iwicki 2023-12-31 13:57:25 UTC
I've brought the question up on the legal mailing list, and the answer there was that this should be OK to package.
https://lists.fedoraproject.org/archives/list/legal@lists.fedoraproject.org/thread/QBYRCIMQDXAD2ZKUWKKYSTDB6T6GW2SO/

We can still open a ticket on the license-data GitLab, or mark this as blocked by FE-LEGAL, for further review.

Comment 9 Tom Rix 2023-12-31 14:33:13 UTC
Ok, sound reasonable.  Since this is complicated, I think you should add the link and the entire text to the spec file.
repost spec and srpm and I will go forward with the review.

Comment 10 Artur Frenszek-Iwicki 2024-01-29 19:29:16 UTC
Huh. I've updated the spec and then forgot to make a comment here.
Anyway. Updated to v16.1.

spec: https://suve.fedorapeople.org/review/cdecl-16.1-1/cdecl.spec
srpm: https://suve.fedorapeople.org/review/cdecl-16.1-1/cdecl-16.1-1.fc39.src.rpm
koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=112560612

Comment 11 Fedora Review Service 2024-01-29 19:38:12 UTC
Created attachment 2011904 [details]
The .spec file difference from Copr build 6362997 to 6971775

Comment 12 Fedora Review Service 2024-01-29 19:38:15 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/6971775
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2227277-cdecl/fedora-rawhide-x86_64/06971775-cdecl/fedora-review/review.txt

Found issues:

- Not a valid SPDX expression 'GPL-3.0-or-later AND LGPL-2.1-or-later AND LicenseRef-Fedora-PublicDomain'.
  Read more: https://fedoraproject.org/wiki/Changes/SPDX_Licenses_Phase_1

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 14 Fedora Review Service 2024-02-21 10:56:14 UTC
Created attachment 2017961 [details]
The .spec file difference from Copr build 6971775 to 7042156

Comment 15 Fedora Review Service 2024-02-21 10:56:16 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7042156
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2227277-cdecl/fedora-rawhide-x86_64/07042156-cdecl/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 17 Fedora Review Service 2024-03-02 15:13:03 UTC
Created attachment 2019788 [details]
The .spec file difference from Copr build 7042156 to 7097401

Comment 18 Fedora Review Service 2024-03-02 15:13:05 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/7097401
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2227277-cdecl/fedora-rawhide-x86_64/07097401-cdecl/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 20 Tom Rix 2024-05-20 19:45:15 UTC
Thanks for documenting the license discussion.
Approved.

Comment 21 Fedora Admin user for bugzilla script actions 2024-05-21 04:53:50 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/cdecl

Comment 22 Fedora Update System 2024-05-21 05:20:38 UTC
FEDORA-2024-4ff196a896 (cdecl-16.3-1.fc40) has been submitted as an update to Fedora 40.
https://bodhi.fedoraproject.org/updates/FEDORA-2024-4ff196a896

Comment 23 Fedora Update System 2024-05-22 01:11:48 UTC
FEDORA-2024-4ff196a896 has been pushed to the Fedora 40 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-4ff196a896 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-4ff196a896

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 24 Fedora Update System 2024-05-26 02:33:49 UTC
FEDORA-2024-4f37316d6a has been pushed to the Fedora 40 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-4f37316d6a`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-4f37316d6a

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 25 Fedora Update System 2024-05-29 03:45:52 UTC
FEDORA-2024-3e6bb779ce has been pushed to the Fedora 40 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2024-3e6bb779ce`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2024-3e6bb779ce

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 26 Fedora Update System 2024-06-06 01:38:12 UTC
FEDORA-2024-3e6bb779ce (cdecl-16.4.1-1.fc40) has been pushed to the Fedora 40 stable repository.
If problem still persists, please make note of it in this bug report.


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