Bug 609011

Summary: Review Request: derby - Relational database implemented entirely in Java
Product: [Fedora] Fedora Reporter: Lubomir Rintel <lkundrak>
Component: Package ReviewAssignee: Marek Mahut <mmahut>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 13CC: akurtako, fedora-package-review, mefoster, mmahut, mzatko, notting, sebastian, sochotni
Target Milestone: ---Flags: mmahut: fedora-review+
j: 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: 2010-08-18 15:20:05 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: 609012    

Description Lubomir Rintel 2010-06-29 08:06:31 UTC
SPEC: http://v3.sk/~lkundrak/SPECS/derby.spec
SRPM: http://v3.sk/~lkundrak/gooddata-cli/derby-10.6.1.0-1.fc13.src.rpm

Description:

Apache Derby, an Apache DB sub-project, is a relational database implemented
entirely in Java. Some key advantages include a small footprint, conformance
to Java, JDBC, and SQL standards and embedded JDBC driver.

Comment 1 Alexander Kurtakov 2010-06-29 13:12:52 UTC
It would be good if you install maven poms and scripts for using tools.
Also there is https://bugzilla.redhat.com/show_bug.cgi?id=532520 which is a dubplicate of this one.

Comment 3 Lubomir Rintel 2010-07-27 01:49:27 UTC
*** Bug 532520 has been marked as a duplicate of this bug. ***

Comment 4 Lubomir Rintel 2010-07-27 09:19:28 UTC
            .--.
          .'    '.
         /  ~~~~  \
        ( __    __ )
       /|<o->  <o->|\   Oh noes
      ( |    ^^    | )  people over the internets
     _ ) \   __   / /   adding themselves to cc
    /##\  \_(__)_/ /    laughing at java tools packaging
   /####) )#\__// (
  /####( |##| |#\  \
 (#####| |##(_/##\_/\

Comment 5 Maros Zatko 2010-07-29 16:50:39 UTC
OK - rpmlint must be run on every package. The output should be posted in the review.
OK - The package must be named according to the  Package Naming Guidelines .
OK - The spec file name must match the base package %{name}, in the format %{name}.spec unless your package has an exemption.
?? - The package must meet the  Packaging Guidelines  .
OK - The package must be licensed with a Fedora approved license and meet the  Licensing Guidelines .
OK - The License field in the package spec file must match the actual license.
OK - If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package must be included in %doc.[4]
OK - The spec file must be written in American English.
OK - The spec file for the package MUST be legible.
OK - The sources used to build the package must match the upstream source, as provided in the spec URL. Reviewers should use md5sum for this task. If no upstream URL can be specified for this package, please see the  Source URL Guidelines  for how to deal with this.
* Source db-derby-10.6.1.0-src.tar.gz is the same as upstream
* Source derby-10.6.1.0.pom is the same as upstream
* Source derby-project-10.6.1.0.pom is the same as upstream
* Source derbyLocale_cs-10.6.1.0.pom is the same as upstream
* Source derbyLocale_de_DE-10.6.1.0.pom is the same as upstream
* Source derbyLocale_es-10.6.1.0.pom is the same as upstream
* Source derbyLocale_fr-10.6.1.0.pom is the same as upstream
* Source derbyLocale_hu-10.6.1.0.pom is the same as upstream
* Source derbyLocale_it-10.6.1.0.pom is the same as upstream
* Source derbyLocale_ja_JP-10.6.1.0.pom is the same as upstream
* Source derbyLocale_ko_KR-10.6.1.0.pom is the same as upstream
* Source derbyLocale_pl-10.6.1.0.pom is the same as upstream
* Source derbyLocale_pt_BR-10.6.1.0.pom is the same as upstream
* Source derbyLocale_ru-10.6.1.0.pom is the same as upstream
* Source derbyLocale_zh_CN-10.6.1.0.pom is the same as upstream
* Source derbyLocale_zh_TW-10.6.1.0.pom is the same as upstream
* Source derbyclient-10.6.1.0.pom is the same as upstream
* Source derbynet-10.6.1.0.pom is the same as upstream
* Source derbytools-10.6.1.0.pom is the same as upstream
FAIL - The package MUST successfully compile and build into binary rpms on at least one primary architecture.

Does not build in koji.
See http://koji.fedoraproject.org/koji/taskinfo?taskID=2356692

build.log from mock:
/var/tmp/rpm-tmp.rd7cdF: line 53: ant: command not found

Please add ant to dependencies.

This review was informal and is incomplete.

Comment 6 Lubomir Rintel 2010-07-30 13:00:37 UTC
(In reply to comment #5)
> FAIL - The package MUST successfully compile and build into binary rpms on at
> least one primary architecture.
> 
> Does not build in koji.
> See http://koji.fedoraproject.org/koji/taskinfo?taskID=2356692
> 
> build.log from mock:
> /var/tmp/rpm-tmp.rd7cdF: line 53: ant: command not found
> 
> Please add ant to dependencies.

Thank you. Fixed the BRs:

SPEC: http://v3.sk/~lkundrak/SPECS/derby.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/derby-10.6.1.0-3.fc13.src.rpm

Comment 7 Marek Mahut 2010-08-11 10:27:29 UTC
Internet People, this new package looks sane to me. APPROVED.

Comment 8 Lubomir Rintel 2010-08-13 13:24:38 UTC
Much thanks, Mar{ek,os} for reviewing this!

New Package SCM Request
=======================
Package Name: derby
Short Description: Relational database implemented entirely in java
Owners: lkundrak
Branches: f13 f14 el6

Comment 9 Jason Tibbitts 2010-08-13 16:09:18 UTC
Git done (by process-git-requests).

Comment 10 Lubomir Rintel 2010-08-18 15:20:05 UTC
______
_|___|__
___|___|__
_|___|___|______     Imported and built.
___|___|___|___|__
_|___|___|___|___|
___|___|___|___|___|
_|___|___|___|___|___