Bug 1945938

Summary: Review Request: rgbds - An assembly toolchain for the Nintendo Game Boy & Game Boy Color
Product: [Fedora] Fedora Reporter: Jan Drögehoff <sentrycraft123>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rawhideCC: eclipseo, package-review
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2021-04-03 16:04:32 UTC 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: 177841, 1944764    

Description Jan Drögehoff 2021-04-02 19:03:17 UTC
Spec URL: https://pagure.io/rgbds-rpm/raw/0adf657ecd48d77c72b302f8547bab78e3e6ee44/f/rgbds.spec
SRPM URL: https://releases.pagure.org/rgbds-rpm/rgbds-0.4.2-1.fc33.src.rpm

Description:
RGBDS (Rednex Game Boy Development System) is a free assembler/linker package
for the Game Boy and Game Boy Color.

It consists of:

* rgbasm (assembler)
* rgblink (linker)
* rgbfix (checksum/header fixer)
* rgbgfx (PNG‐to‐2bpp graphics converter)

Fedora Account System Username: sentry
This is my first Fedora package so I'm looking for a sponsor

rpmlint output:
> rgbds.src: W: spelling-error Summary(en_US) toolchain -> tool chain, tool-chain, touchline
> rgbds.src: W: spelling-error %description -l en_US rgbasm -> orgasm
> rgbds.src: W: spelling-error %description -l en_US rgblink -> blink
> rgbds.src: W: spelling-error %description -l en_US rgbfix
> rgbds.src: W: spelling-error %description -l en_US checksum -> check sum, check-sum, checks um
> rgbds.src: W: spelling-error %description -l en_US rgbgfx

Comment 1 Robert-André Mauchin 🐧 2021-04-02 23:47:04 UTC
 - No review yet but please also set the default ldflags:

LDFLAGS=%{build_ldflags}

 - Please BR 'make' explicitly

BuildRequires:  make

Comment 2 Jan Drögehoff 2021-04-02 23:57:17 UTC
Totally forgot about that, thanks for pointing that out

Spec URL: https://pagure.io/rgbds-rpm/raw/fe057a23e365af103cfe9c82ff5c9e5f0483590f/f/rgbds.spec
SRPM URL: https://releases.pagure.org/rgbds-rpm/rgbds-0.4.2-1.fc33.src1.rpm

Comment 3 Robert-André Mauchin 🐧 2021-04-03 16:04:32 UTC
It seems already packaged: https://src.fedoraproject.org/rpms/rgbds