Bug 518486 - Review Request: eclipse-testframework - Eclipse Test Framework
Summary: Review Request: eclipse-testframework - Eclipse Test Framework
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Andrew Overholt
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-08-20 15:23 UTC by Alexander Kurtakov
Modified: 2009-08-21 18:51 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-08-21 18:51:50 UTC
Type: ---
Embargoed:
overholt: fedora-review+
dennis: fedora-cvs+


Attachments (Terms of Use)

Description Alexander Kurtakov 2009-08-20 15:23:21 UTC
Spec URL: http://akurtakov.fedorapeople.org/eclipse-testframework.spec
SRPM URL: http://akurtakov.fedorapeople.org/eclipse-testframework-3.5.0-1.fc11.src.rpm
Description: Eclipse Test Framework. Used in conjunction with Eclipse JUnit tests.

Comment 1 Andrew Overholt 2009-08-20 15:38:22 UTC
OK naming, licensing
OK files
OK macros
OK clean, sections
? can we get a better URL
? should it depend upon eclipse >= 3.5?
? do we need to symlink to the JUnit bundles in the installation?
X please version the tarball and its contents
X we need an R3_5 tag passed to the fetch script.  otherwise, ...
OK source (md5sum doesn't match, but contents do)

X rpmlint is not clean:

$ rpmlint eclipse-testframework-3.5.0-1.fc11.noarch.rpm
eclipse-testframework.noarch: E: non-standard-dir-perm /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130 0775
eclipse-testframework.noarch: E: non-standard-dir-perm /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130/src/org.eclipse.test.performance_3.5.0.200908201130 0775
eclipse-testframework.noarch: W: spurious-executable-perm /usr/share/doc/eclipse-testframework-3.5.0/license.html
eclipse-testframework.noarch: E: non-standard-dir-perm /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130/META-INF 0775
eclipse-testframework.noarch: E: non-standard-dir-perm /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test_3.2.0/META-INF 0775
eclipse-testframework.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.easymock_2.4.0.v20090202-0900.jar ../../../../../java/easymock2.jar
eclipse-testframework.noarch: E: non-standard-dir-perm /usr/share/eclipse/dropins/testframework/eclipse/features/org.eclipse.test_3.5.0.200908201130 0775
eclipse-testframework.noarch: E: non-standard-dir-perm /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130/src 0775
eclipse-testframework.noarch: E: non-standard-dir-perm /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test_3.2.0 0775
eclipse-testframework.noarch: E: non-standard-dir-perm /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130/src/org.eclipse.test_3.2.0 0775
eclipse-testframework.noarch: W: spurious-executable-perm /usr/share/doc/eclipse-testframework-3.5.0/epl-v10.html
1 packages and 0 specfiles checked; 8 errors, 3 warnings.

Comment 2 Alexander Kurtakov 2009-08-21 12:20:26 UTC
(In reply to comment #1)
> OK naming, licensing
> OK files
> OK macros
> OK clean, sections
> ? can we get a better URL
I can't find any better URL
> ? should it depend upon eclipse >= 3.5?
Yes, fixed.
> ? do we need to symlink to the JUnit bundles in the installation?
They will be resolved from the eclipse-jdt (added as requires).
> X please version the tarball and its contents
Done
> X we need an R3_5 tag passed to the fetch script.  otherwise, ...
Done
> OK source (md5sum doesn't match, but contents do)
> 
> X rpmlint is not clean:
> 
> $ rpmlint eclipse-testframework-3.5.0-1.fc11.noarch.rpm
> eclipse-testframework.noarch: E: non-standard-dir-perm
> /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130
> 0775
> eclipse-testframework.noarch: E: non-standard-dir-perm
> /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130/src/org.eclipse.test.performance_3.5.0.200908201130
> 0775
> eclipse-testframework.noarch: W: spurious-executable-perm
> /usr/share/doc/eclipse-testframework-3.5.0/license.html
> eclipse-testframework.noarch: E: non-standard-dir-perm
> /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130/META-INF
> 0775
> eclipse-testframework.noarch: E: non-standard-dir-perm
> /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test_3.2.0/META-INF
> 0775
> eclipse-testframework.noarch: W: dangling-relative-symlink
> /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.easymock_2.4.0.v20090202-0900.jar
> ../../../../../java/easymock2.jar
> eclipse-testframework.noarch: E: non-standard-dir-perm
> /usr/share/eclipse/dropins/testframework/eclipse/features/org.eclipse.test_3.5.0.200908201130
> 0775
> eclipse-testframework.noarch: E: non-standard-dir-perm
> /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130/src
> 0775
> eclipse-testframework.noarch: E: non-standard-dir-perm
> /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test_3.2.0
> 0775
> eclipse-testframework.noarch: E: non-standard-dir-perm
> /usr/share/eclipse/dropins/testframework/eclipse/plugins/org.eclipse.test.source_3.5.0.200908201130/src/org.eclipse.test_3.2.0
> 0775
> eclipse-testframework.noarch: W: spurious-executable-perm
> /usr/share/doc/eclipse-testframework-3.5.0/epl-v10.html
> 1 packages and 0 specfiles checked; 8 errors, 3 warnings.  
Done. I wasn't seeing all the errors you listed. But the only rpmlint warning i see is dangling-relative-symlink which is normal.

New sources:
Spec URL: http://akurtakov.fedorapeople.org/eclipse-testframework.spec
SRPM URL:
http://akurtakov.fedorapeople.org/eclipse-testframework-3.5.0-2.fc11.src.rpm

Comment 3 Andrew Overholt 2009-08-21 12:29:24 UTC
Thanks for the fixes.  Approved.

Comment 4 Alexander Kurtakov 2009-08-21 12:40:28 UTC
New Package CVS Request
=======================
Package Name: eclipse-testframework 
Short Description: Eclipse Test Framework
Owners: akurtakov overholt
Branches: 
InitialCC:

Comment 5 Dennis Gilmore 2009-08-21 18:25:03 UTC
CVS Done

Comment 6 Andrew Overholt 2009-08-21 18:51:50 UTC
Built:

http://koji.fedoraproject.org/koji/buildinfo?buildID=128643


Note You need to log in before you can comment on or make changes to this bug.