Bug 2329823 - Review Request: GENie - Project generator tool
Summary: Review Request: GENie - Project generator tool
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Simone Caronni
QA Contact: Fedora Extras Quality Assurance
URL: https://github.com/bkaradzic/GENie
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-12-02 08:49 UTC by Benson Muite
Modified: 2025-01-11 01:42 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2025-01-11 01:25:28 UTC
Type: ---
Embargoed:
negativo17: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 8332924 to 8453995 (1.74 KB, patch)
2024-12-29 20:04 UTC, Fedora Review Service
no flags Details | Diff

Description Benson Muite 2024-12-02 08:49:40 UTC
spec: https://download.copr.fedorainfracloud.org/results/fed500/GENie/fedora-rawhide-x86_64/08332917-GENie/GENie.spec
srpm: https://download.copr.fedorainfracloud.org/results/fed500/GENie/fedora-rawhide-x86_64/08332917-GENie/GENie-1181-1.fc42.src.rpm

description:
GENie (pronounced as Jenny) is project generator tool. It automagically
generates project from Lua script, making applying the same settings for
multiple projects easy.

fas: fed500

Reproducible: Always

Comment 1 Fedora Review Service 2024-12-02 08:54:28 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8332924
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2329823-genie/fedora-rawhide-x86_64/08332924-GENie/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 2 Simone Caronni 2024-12-28 11:46:40 UTC
1) Please drop the first block, is not needed at all:

## START: Set by rpmautospec
## (rpmautospec version 0.7.3)
## RPMAUTOSPEC: autorelease, autochangelog
%define autorelease(e:s:pb:n) %{?-p:0.}%{lua:
    release_number = 1;
    base_release_number = tonumber(rpm.expand("%{?-b*}%{!?-b:1}"));
    print(release_number + base_release_number - 1);
}%{?-e:.%{-e*}}%{?-s:.%{-s*}}%{!?-n:%{?dist}}
## END: Set by rpmautospec

2) Please adjust version, since you are already declaring commit/date/shortcommit, use them in the version field.

Version: 1181^%{date}.%{shortcommit}

3) Please pass "-p" to install commands in the install section. You can also squeeze in one line omitting mkdir:

install -p -m 755 bin/linux/genie %{buildroot}%{_bindir}/genie

4) In the %changelog section, remove everything and just use %autochangelog to have the changelog automatically populated with git commit messages:

%changelog
%autochangelog

Comment 3 Simone Caronni 2024-12-28 11:48:11 UTC
Btw, since the project does not provide any tag or release, you might as want package the latest snapshot which is at version 1187. Thanks.

Comment 4 Benson Muite 2024-12-29 18:27:23 UTC
Thanks for the feedback. Addressed all comments and updated.


Spec URL: https://fed500.fedorapeople.org/GENie.spec
SRPM URL: https://fed500.fedorapeople.org/GENie-1187^20241207.93f6621-1.fc42.src.rpm

Comment 5 Fedora Review Service 2024-12-29 18:27:41 UTC
There seems to be some problem with the following file.
SRPM URL: https://fed500.fedorapeople.org/GENie-1187%5E20241207.93f6621-1.fc42.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 6 Benson Muite 2024-12-29 20:01:15 UTC
[fedora-review-service-build]

Comment 7 Fedora Review Service 2024-12-29 20:04:48 UTC
Created attachment 2064161 [details]
The .spec file difference from Copr build 8332924 to 8453995

Comment 8 Fedora Review Service 2024-12-29 20:04:50 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8453995
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2329823-genie/fedora-rawhide-x86_64/08453995-GENie/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 9 Simone Caronni 2025-01-01 11:12:33 UTC
Package approved.

Comment 10 Fedora Admin user for bugzilla script actions 2025-01-02 11:08:21 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/GENie

Comment 11 Benson Muite 2025-01-02 12:58:15 UTC
Thanks for the review.

Comment 12 Fedora Update System 2025-01-02 14:56:58 UTC
FEDORA-2025-fe353b50ec (GENie-1187^20241207.93f6621-2.fc40) has been submitted as an update to Fedora 40.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-fe353b50ec

Comment 13 Fedora Update System 2025-01-02 14:57:29 UTC
FEDORA-2025-5096ebaac7 (GENie-1187^20241207.93f6621-2.fc41) has been submitted as an update to Fedora 41.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-5096ebaac7

Comment 14 Benson Muite 2025-01-02 19:30:48 UTC
https://release-monitoring.org/project/376202/

Comment 15 Fedora Update System 2025-01-03 03:58:05 UTC
FEDORA-2025-fe353b50ec 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-2025-fe353b50ec \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-fe353b50ec

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

Comment 16 Fedora Update System 2025-01-03 05:03:57 UTC
FEDORA-2025-5096ebaac7 has been pushed to the Fedora 41 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-5096ebaac7 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-5096ebaac7

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

Comment 17 Fedora Update System 2025-01-11 01:25:28 UTC
FEDORA-2025-fe353b50ec (GENie-1187^20241207.93f6621-2.fc40) has been pushed to the Fedora 40 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 18 Fedora Update System 2025-01-11 01:42:04 UTC
FEDORA-2025-5096ebaac7 (GENie-1187^20241207.93f6621-2.fc41) has been pushed to the Fedora 41 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.