[Bug 1414087] New: Review Request: Gearboy - Game Boy and Game Boy Color emulator

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1414087

            Bug ID: 1414087
           Summary: Review Request: Gearboy - Game Boy and Game Boy Color
                    emulator
           Product: Fedora
           Version: rawhide
         Component: Package Review
          Severity: medium
          Assignee: nobody@xxxxxxxxxxxxxxxxx
          Reporter: dmoerner@xxxxxxxxx
        QA Contact: extras-qa@xxxxxxxxxxxxxxxxx
                CC: package-review@xxxxxxxxxxxxxxxxxxxxxxx



Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/dmoerner/Gearboy/Gearboy.git/plain/Gearboy.spec?id=91cac5b448bbdee305b3a040bb2b11108a6537ff
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/dmoerner/Gearboy/fedora-25-x86_64/00500763-Gearboy/Gearboy-2.3.1-1.fc25.src.rpm

Description: 
Gearboy is a Nintendo Game Boy and Game Boy Color emulator written in
C++. The emulator prioritizes readable, simple source code and high
compatibility across a variety of platforms and architectures. Gearboy
features highly accurate CPU emulation, instruction timing, and memory
timing. It also includes memory bank controllers, accurate LCD
controller emulation, mix frames, sound emulation, and compressed ROM
support.

Fedora Account System Username: dmoerner

Successful koji scratch-build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=17313799

This is my first package, and I need a sponsor. 

Gearboy is an emulator compatible with Fedora's new, more relaxed emulator
policy.[1] It is GPLv3+ and contains no non-free components. It needs only game
ROMS supplied by the user and not included in the package.

I chose to name it "Gearboy" rather than "gearboy" because upstreame
consistently uses the capitalized form, including in documentation and in the
binary name. (The only exception is actually git tagging, as noted in the spec
file.) If "gearboy" would be preferred, it would be trivial to rename the files
during install.

Upstream has been very responsive; they merged a number of pull requests I
submitted, including man, desktop, and appdata files that I wrote.[2]

I checked the package with mock, rpmlint, and fedora-review many times, and
believe it conforms to the package requirements.

To potential sponsors:

I already introduced myself informally on the games mailing list[3], and I will
introduce myself again on the devel list. Here is some relevant information
that might help you decide to sponsor me:

I have read the packaging guidelines many times. Gearboy is actually the third
package I have created while trying to follow the packaging guidelines. (The
first, hamlet, an SML interpreter, still requires some patches to satisfy FHS
which I have not yet written. I hope to submit a review-request for it when I
do that. The second, higan, another emulator, has some licensing issues that I
came to believe make it unsuitable for inclusion in Fedora.)

I completed a sample review for the minetestmapper package.[4] I thought the
package was in very good shape, so it turned out to be very simple. (I'm sure I
missed something, though!) But it did give me an opportunity to find what I
believe is a small bug in fedora-review[5], and to write and submit a manpage
for the package upstream.[6]

Finally, back when I was involved in Debian development, they asked you to
triage a so-called "release-critical" bug, like a FTBFS. I informally did this
for a recent bug in guile found during the rawhide readline-7 rebuild.[7] (It
turned out to be non-reproducible.)

I also hang out in IRC as dmoerner, you are welcome to quiz me or ask me to do
additional things! Thanks!

[1]
https://fedoraproject.org/wiki/Licensing:SoftwareTypes?rd=Licensing/SoftwareTypes#Emulators
[2] https://github.com/drhelius/Gearboy/pull/42
[3]
https://lists.fedoraproject.org/archives/list/games@xxxxxxxxxxxxxxxxxxxxxxx/thread/LPEW5CCGK6JRGDCV3CTRXGGDTAD2EHO5/
[4] https://bugzilla.redhat.com/show_bug.cgi?id=1410651#c1
[5] https://bugzilla.redhat.com/show_bug.cgi?id=1414065
[6] https://github.com/minetest/minetestmapper/pull/43
[7] https://bugzilla.redhat.com/show_bug.cgi?id=1412931#c1

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux