Bug 2150545 - Review Request: xtideuniversalbios - XTIDE Universal BIOS
Summary: Review Request: xtideuniversalbios - XTIDE Universal BIOS
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 2150546 2150547
TreeView+ depends on / blocked
 
Reported: 2022-12-03 22:29 UTC by Davide Cavalca
Modified: 2022-12-07 01:35 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2022-12-05 16:49:26 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description Davide Cavalca 2022-12-03 22:29:59 UTC
Spec URL: https://dcavalca.fedorapeople.org/review/xtideuniversalbios/xtideuniversalbios.spec
SRPM URL: https://dcavalca.fedorapeople.org/review/xtideuniversalbios/xtideuniversalbios-2.0.0^20221002svn624-1.fc38.src.rpm

Description:
XTIDE Universal BIOS makes it possible to use modern large ATA hard disks or
Compact Flash cards on old PC's. You can then enjoy quiet or noiseless drives
with more capacity than you'll ever need for old computers.

XTIDE Universal BIOS (also known as simply "XUB") can be used on any IBM PC,
XT, AT or 100% compatible system.

Fedora Account System Username: dcavalca

Comment 1 Davide Cavalca 2022-12-03 22:30:02 UTC
This package built on koji:  https://koji.fedoraproject.org/koji/taskinfo?taskID=94897231

Comment 2 Davide Cavalca 2022-12-03 22:35:22 UTC
Open to suggestions for a better filesystem location for this, I'm not sure if %{_datadir} is the right place for x86 BIOS extensions and DOS binaries.

Comment 3 Neal Gompa 2022-12-03 22:54:34 UTC
Taking this review.

Comment 4 Neal Gompa 2022-12-03 22:57:52 UTC
(In reply to Davide Cavalca from comment #2)
> Open to suggestions for a better filesystem location for this, I'm not sure
> if %{_datadir} is the right place for x86 BIOS extensions and DOS binaries.

Normally, foreign platform binaries need a dedicated hierarchy, but in this case, I don't think it's not worth worrying about.

Comment 5 Neal Gompa 2022-12-03 23:00:23 UTC
(In reply to Neal Gompa from comment #4)
> (In reply to Davide Cavalca from comment #2)
> > Open to suggestions for a better filesystem location for this, I'm not sure
> > if %{_datadir} is the right place for x86 BIOS extensions and DOS binaries.
> 
> Normally, foreign platform binaries need a dedicated hierarchy, but in this
> case, I don't think it's not worth worrying about.

Blech, I mean it's not worth worrying about.

Comment 6 Neal Gompa 2022-12-03 23:02:15 UTC
Also, I've been poking through the source tree, and it seems like this is actually GPLv2+, rather than GPLv2...

Comment 7 Davide Cavalca 2022-12-03 23:08:09 UTC
You're right, the actual text the include is GPLv2, but a large part of the files have GPLv2+ headers.

Spec URL: https://dcavalca.fedorapeople.org/review/xtideuniversalbios/xtideuniversalbios.spec
SRPM URL: https://dcavalca.fedorapeople.org/review/xtideuniversalbios/xtideuniversalbios-2.0.0^20221002svn624-1.fc38.src.rpm

Changelog:
- update license tag

Comment 8 Neal Gompa 2022-12-03 23:26:21 UTC
Review notes:

* Package is named appropriately
* Licensing is correctly noted and license files are installed
* Package dependencies look sane
* No serious errors from rpmlint

PACKAGE APPROVED.

Comment 9 Gwyn Ciesla 2022-12-05 15:24:59 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/xtideuniversalbios

Comment 10 Fedora Update System 2022-12-05 16:48:26 UTC
FEDORA-2022-3d2cd12a8d has been submitted as an update to Fedora 38. https://bodhi.fedoraproject.org/updates/FEDORA-2022-3d2cd12a8d

Comment 11 Fedora Update System 2022-12-05 16:49:26 UTC
FEDORA-2022-3d2cd12a8d has been pushed to the Fedora 38 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 12 Fedora Update System 2022-12-05 19:44:45 UTC
FEDORA-2022-0b5746bee3 has been submitted as an update to Fedora 37. https://bodhi.fedoraproject.org/updates/FEDORA-2022-0b5746bee3

Comment 13 Fedora Update System 2022-12-06 01:32:34 UTC
FEDORA-2022-0b5746bee3 has been pushed to the Fedora 37 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2022-0b5746bee3 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2022-0b5746bee3

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

Comment 14 Fedora Update System 2022-12-07 01:35:58 UTC
FEDORA-2022-0b5746bee3 has been pushed to the Fedora 37 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.