Bug 567419

Summary: Review Request: jps - Gant based build framework and DSL
Product: [Fedora] Fedora Reporter: Lubomir Rintel <lkundrak>
Component: Package ReviewAssignee: Michal Fojtik <mfojtik>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: fedora-package-review, mfojtik, notting
Target Milestone: ---Flags: mfojtik: fedora-review+
j: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard: F13FeatureIDEA
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-03-23 08:01:33 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: 565844    

Description Lubomir Rintel 2010-02-22 22:14:02 UTC
SPEC: http://v3.sk/~lkundrak/SPECS/jps.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/jps-0-1.20100222git.fc14.src.rpm

Description:

Gant based build framework + dsl, with declarative project structure
definition and automatic IntelliJ IDEA projects build.

Comment 1 Lubomir Rintel 2010-02-22 22:15:16 UTC
Has a circular dependency with Idea (bug #565844)

Comment 2 Lubomir Rintel 2010-02-23 04:40:43 UTC
Bootstrap enabled, OpenJDK used for regex library.

scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2007631
SPEC: http://v3.sk/~lkundrak/SPECS/jps.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/jps-0-1.20100222git.fc12.src.rpm

Comment 3 Michal Fojtik 2010-02-25 14:35:57 UTC
* rpmlint

rpmlint shows 1 warning, but seems like this package is OK.

* Correctly named
* Correctly versioned, packing the latest version
* Source checksum correct
* SPEC file clean and legible
* American english used
* Documentation files marked correctly
* License correct and permitted in Fedora
* Builds fine in mock

Comment 4 Michal Fojtik 2010-02-25 14:51:38 UTC
* Correctly versioned, packing the latest version

Oh I forgot: Version 0 ?

Comment 5 Lubomir Rintel 2010-02-25 15:00:30 UTC
(In reply to comment #4)
> * Correctly versioned, packing the latest version
> 
> Oh I forgot: Version 0 ?    

Upstream does not have any stable versioning yet, so using this to warrant smooth version bump once upstream adds a version number.

Upstream was contacted about this.

Comment 6 Lubomir Rintel 2010-02-25 17:00:26 UTC
(Won't import/build this until when intellij-idea is approved as well, since they are dependent on each other.)

New Package CVS Request
=======================
Package Name: jsl
Short Description: Gant based build framework and DSL
Owners: lkundrak
Branches: F-13

Comment 7 Jason Tibbitts 2010-02-25 17:55:05 UTC
The bug summary and the requested package name don't seem to agree.  Not only
that, but the requested package name doesn't seem to appear anywhere else in
the ticket.  What's the proper package name?

Comment 8 Lubomir Rintel 2010-02-25 17:59:19 UTC
(In reply to comment #7)
> What's the proper package name?

Uh, sorry I messed that up. The correct request:

New Package CVS Request
=======================
Package Name: jps
Short Description: Gant based build framework and DSL
Owners: lkundrak
Branches: F-13

Also: got a reply from upstream: "Let it be 0.1" regarding the next release version number. Will fix up spec upon import.

Comment 9 Jason Tibbitts 2010-02-25 18:17:48 UTC
CVS done (by process-cvs-requests.py).

Comment 10 Lubomir Rintel 2010-03-23 08:01:33 UTC
Imported and built.
Thank you.