Bug 546666 - Review Request: selenium-core - A DHTML test execution framework
Summary: Review Request: selenium-core - A DHTML test execution framework
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Marek Mahut
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 546667
TreeView+ depends on / blocked
 
Reported: 2009-12-11 16:16 UTC by Lubomir Rintel
Modified: 2010-01-04 21:47 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-01-04 21:47:34 UTC
Type: ---
Embargoed:
mmahut: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Lubomir Rintel 2009-12-11 16:16:20 UTC
SPEC: http://v3.sk/~lkundrak/SPECS/selenium-core.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/selenium-core-1.0.2-0.1.20091211svn.fc12.src.rpm
scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1869933

Description:

Selenium Core is the engine of both, Selenium IDE and Selenium RC (driven
mode), but it also can be deployed on the desired application server.

Comment 1 Lubomir Rintel 2009-12-11 16:18:41 UTC
rpmlint is quitet.

local mockbuild succeeded (hopefully the scratch build will as well)

Comment 2 Lubomir Rintel 2009-12-11 20:21:48 UTC
Requires:       ant
Requires:       commons-logging
Requires:       servlet

I'm missing these ^^^ they will be added on next package spin (once reviewed)

Comment 3 Marek Mahut 2010-01-03 11:20:19 UTC
- Package meets naming and packaging guidelines
- Spec file matches base package name.
- Meets Packaging Guidelines.
- Licenses field in spec matches the content of tarball
- License file included in package matches used code (the code is provided under 3 different licenses)
- BuildRequires are correct (tests will depend on selenium-server once imported, also see comment #2)
- Packages %doc files don't affect runtime.
- Rpmlint is clean.
- Builds in mock for rawhide.

This package looks sane, approved.

Comment 4 Lubomir Rintel 2010-01-03 18:42:05 UTC
Dziękuję, Marek.

New Package CVS Request
=======================
Package Name: selenium-core
Short Description: A DHTML test execution framework
Owners: lkundrak
Branches: F-11 F-12 EL-5

Comment 5 Kevin Fenzi 2010-01-04 20:16:25 UTC
cvs done.

Comment 6 Lubomir Rintel 2010-01-04 21:47:34 UTC
Thank you. Imported and built.


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