Bug 302351

Summary: Review Request: higlayout - Easy to use and powerful layout manager for Java
Product: [Fedora] Fedora Reporter: Hans de Goede <hdegoede>
Component: Package ReviewAssignee: Ville Skyttä <ville.skytta>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting
Target Milestone: ---Flags: ville.skytta: fedora-review+
kevin: 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: 2007-09-28 19:12:11 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 Hans de Goede 2007-09-23 19:56:47 UTC
Spec URL: http://people.atrpms.net/~hdegoede/higlayout.spec
SRPM URL: http://people.atrpms.net/~hdegoede/higlayout-1.0-1.fc8.src.rpm
Description:
Easy to use and powerful layout manager for Java.

Comment 1 Ville Skyttä 2007-09-25 17:52:51 UTC
"BR: java-devel" should be replaced by "BR: java-javadoc" (even if sinjdoc
doesn't do crosslinked javadocs at the moment) - java-gcj-compat-devel provides
java-devel.  Otherwise looks good to me, feel free to import and fix before the
first build.

Comment 2 Hans de Goede 2007-09-26 21:12:28 UTC
(In reply to comment #1)
> "BR: java-devel" should be replaced by "BR: java-javadoc" (even if sinjdoc
> doesn't do crosslinked javadocs at the moment) - java-gcj-compat-devel provides
> java-devel.  Otherwise looks good to me, feel free to import and fix before the
> first build.

Nothing provides java-javadoc, thats why I put java-devel there, but you are
right, that is the same as java-gcj-compat-devel. So I'll take that out before
import.

Thanks for the review!

Comment 3 Hans de Goede 2007-09-26 21:14:18 UTC
New Package CVS Request
=======================
Package Name:      higlayout
Short Description: Easy to use and powerful layout manager for Java
Owners:            jwrdegoede
Branches:          F-7 devel
InitialCC:         <empty>
Cvsextras Commits: Yes


Comment 4 Ville Skyttä 2007-09-26 22:03:59 UTC
(In reply to comment #2)
> Nothing provides java-javadoc

$ rpm -q --whatprovides java-javadoc
java-1.5.0-gcj-javadoc-1.5.0.0-14.fc7.x86_64


Comment 5 Hans de Goede 2007-09-27 11:35:23 UTC
(In reply to comment #4)
> (In reply to comment #2)
> > Nothing provides java-javadoc
> 
> $ rpm -q --whatprovides java-javadoc
> java-1.5.0-gcj-javadoc-1.5.0.0-14.fc7.x86_64
> 

Your right, I could swear I tried "yum install java-javadoc" after getting an
rpmbuild BuildRequires error for it and yum didn't do anything, ah well, I will
fix this before import.


Comment 6 Kevin Fenzi 2007-09-27 16:31:53 UTC
cvs done.

Comment 7 Hans de Goede 2007-09-28 15:07:37 UTC
Imported and build, closing.