Bug 1094415

Summary: Review Request: jai-imageio-core-standalone - Java Advanced Imaging Image I/O Tools API core (standalone)
Product: [Fedora] Fedora Reporter: gil cattaneo <puntogil>
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: medium    
Version: rawhideCC: mattdm, orion, package-review
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-10-30 09:15:43 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:

Description gil cattaneo 2014-05-05 15:50:27 UTC
Spec URL: http://gil.fedorapeople.org/jai-imageio-core-standalone.spec
SRPM URL: http://gil.fedorapeople.org/jai-imageio-core-standalone-1.2-0.1.pre.dr.b04.2013.04.23.fc19.src.rpm
Description: 
Java Advanced Imaging Image I/O Tools API core, but without the classes 
involved with javax.media.jai dependencies and codecLibJIIO, meaning that
this library can be distributed under the modified BSD license and should be
GPL compatible.
Fedora Account System Username: gil

Comment 1 Orion Poplawski 2014-05-06 19:43:30 UTC
Looks like github based projects need to use the commit hash: https://fedoraproject.org/wiki/Packaging:SourceURL#Github

Comment 2 gil cattaneo 2014-05-07 04:24:06 UTC
(In reply to Orion Poplawski from comment #1)
> Looks like github based projects need to use the commit hash:
> https://fedoraproject.org/wiki/Packaging:SourceURL#Github

for "immutability, availability, uniqueness ... referring to the sources" 
should be sufficient to use the existing archive.
maybe this rule should be used with the repositories that do not have the latest functionality of git*
and more than one package (even my), in http://pkgs.fedoraproject.org/cgit/ don't practice this rule

The really problem with this package is the bundled jj2000 library( http://jpeg2000.epfl.ch/ , https://code.google.com/p/jj2000/ ), but
this one is a customization of the original. Maybe is necessary create/request an FPC exception.

Comment 3 Orion Poplawski 2014-05-08 22:44:42 UTC
I'm not sure I'm happy with it, but it is the most immutable way.

As for jj2000, the usual thing we try to do is to get upstream to accept the changes if possible so we can use the system version.

Comment 4 gil cattaneo 2014-05-09 06:18:01 UTC
(In reply to Orion Poplawski from comment #3)
> I'm not sure I'm happy with it, but it is the most immutable way.
> 
> As for jj2000, the usual thing we try to do is to get upstream to accept the
> changes if possible so we can use the system version.
form google jj200 site: "The code here originally came from version 5.1 of JJ2000 as downloaded from this website: http://jpeg2000.epfl.ch/. However, that code is incomplete, has bugs, has performance issues, and has memory-usage issues. So a Google Code project was created so that modifications and improvements could be made." 
More project (e.g. http://www.unidata.ucar.edu/software/tds/ aka thredds aka netcdf-java submodule grib ) use a proper mod of the same library. your suggestion s not applicable for various reasons, in addition to those listed in the upstream site

Comment 6 gil cattaneo 2014-10-30 09:15:43 UTC
Try to use jai-imageio-core 1.2-0.15.20100217cvs