Bug 458085

Summary: Review Request: nes_ntsc - Provides a NES NTSC video filtering library
Product: [Fedora] Fedora Reporter: David Timms <dtimms>
Component: Package ReviewAssignee: Hans de Goede <hdegoede>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, hdegoede, huzaifas, musuruan, notting
Target Milestone: ---Flags: hdegoede: fedora-review+
huzaifas: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-09-12 14:58:15 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:
Attachments:
Description Flags
diff between original dribble spec0.2.0 and this spec. none

Description David Timms 2008-08-06 13:59:18 UTC
Spec URL: http://members.iinet.net.au/~timmsy/nes_ntsc/nes_ntsc.spec
SRPM URL: 
http://members.iinet.net.au/~timmsy/nes_ntsc/nes_ntsc-0.2.2-1.fc9.src.rpm

Description:
NES NTSC video filter library. Pixel artifacts and color mixing play an 
important role in NES games console graphics. Accepts pixels in native 6-bit
NES palette format, or a 9-bit format that includes the three color emphasis
bits in PPU register $2001. Can also output an RGB palette for use in a 
regular blitter

Comment 1 David Timms 2008-08-06 14:11:39 UTC
Created attachment 313566 [details]
diff between original dribble spec0.2.0 and this spec.

I request some assistance regarding lib sonames. This library has previously been packaged and used by out of Fedora repo applications as libnes_ntsc.so.0.2.0

Have I gone about the soname change in an effective way ?
Should I keep the original 0.2.0 soname instead ?

Comment 2 David Timms 2008-08-06 14:12:06 UTC
I plan on up/packaging upstreams other *_ntsc libs. The author mentions the latest release include a source cleanup and refactoring, so that some files are now identical between the 3x *_ntsc sources. Would it be acceptable/worth the effort to create a single libntsc package, that includes all three upstream source packages, with sub packages for -nes, -snes, -sms ?

Comment 3 Andrea Musuruane 2008-08-06 15:39:12 UTC
(In reply to comment #2)
> I plan on up/packaging upstreams other *_ntsc libs. The author mentions the
> latest release include a source cleanup and refactoring, so that some files are
> now identical between the 3x *_ntsc sources. Would it be acceptable/worth the
> effort to create a single libntsc package, that includes all three upstream
> source packages, with sub packages for -nes, -snes, -sms ?

IMHO it is not worth the trouble and you could have a serious drawback. What if upstream update just -sms? You should make a new release for all three - and this is something unneeded by both end users (who would download a release for -nes and -snes for nothing) and for infrastructure (who would build a release for -nes and -snes for nothing).

Separate packages is the way to go if upstream have separate independent sources.

Comment 4 David Timms 2008-08-06 21:49:34 UTC
(In reply to comment #3)
...
> Separate packages is the way to go if upstream have separate independent
> sources.
Thanks, I thought that might be the case.

Comment 5 Hans de Goede 2008-09-10 08:13:21 UTC
Full review done, no problems found: Approved!

Comment 6 David Timms 2008-09-11 08:07:37 UTC
Excellent, thanks Hans.

New Package CVS Request
=======================
Package Name: nes_ntsc
Short Description: Provides a NES NTSC video filtering library
Owners: dtimms.au
Branches: F-8 F-9
InitialCC:

Comment 7 David Timms 2008-09-11 08:21:57 UTC
One further question: when a reference spec is used as the basis for a Fedora spec, is there a preference on keeping the original changelog entries ?
I see value in acknowledging previous contributors and issues that occurred in getting the spec to it's pre-fedora state. On the other hand, is it allowed to include those old changelog entries ?

Comment 8 Andrea Musuruane 2008-09-11 08:35:50 UTC
(In reply to comment #7)
> One further question: when a reference spec is used as the basis for a Fedora
> spec, is there a preference on keeping the original changelog entries ?
> I see value in acknowledging previous contributors and issues that occurred in
> getting the spec to it's pre-fedora state. On the other hand, is it allowed to
> include those old changelog entries ?

It is fine to keep the original entries. For example, I kept the old entries from hatari when I migrated it from Dribble to Fedora.

Comment 9 Huzaifa S. Sidhpurwala 2008-09-11 08:59:01 UTC
cvs done

Comment 10 Fedora Update System 2008-09-12 12:45:50 UTC
nes_ntsc-0.2.2-1.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/nes_ntsc-0.2.2-1.fc9

Comment 11 Fedora Update System 2008-09-12 12:45:53 UTC
nes_ntsc-0.2.2-1.fc8 has been submitted as an update for Fedora 8.
http://admin.fedoraproject.org/updates/nes_ntsc-0.2.2-1.fc8

Comment 12 Rakesh Pandit 2008-09-12 12:56:18 UTC
Correcting the status and assigning the request to j.w.r.degoede

Thanks,

Comment 13 Fedora Update System 2008-09-25 00:03:05 UTC
nes_ntsc-0.2.2-1.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 14 Fedora Update System 2008-09-25 00:23:55 UTC
nes_ntsc-0.2.2-1.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.