[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-08-18 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Comment #30 from Dave Malcolm [EMAIL PROTECTED]  2008-08-18 09:54:41 EDT 
---
Unfortunately, there are about 15kloc of JavaScript in there, which are heavily
dependent on ExtJS.  A port of Testopia to a different js lib would be a major
engineering effort, and a fork from upstream.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-08-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450013


Jon Stanley [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CANTFIX




--- Comment #29 from Jon Stanley [EMAIL PROTECTED]  2008-08-06 19:11:19 EDT 
---
Had a mini-conversation with spot on IRC today: (edited to show relevant parts,
there was another conversation ongoing).

11:26  jds2001 btw, extjs 2.2 was released on monday
11:26  jds2001 with the FLOSS exception
11:26  jds2001 need spot to opine on that :)
11:27  spot do not want
11:27  spot RH Legal says stay far far far away from ExtJS. Exception, or no
exception.

As a result of this, closing this review CANTFIX.

However, it would be great if we could port this to use some other JavaScript
library, since that's really the only problem with it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-08-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Comment #28 from Jon Stanley [EMAIL PROTECTED]  2008-08-05 22:36:08 EDT 
---
Note that ExtJS published a new version of their exception today over at
http://extjs.com/products/floss-exception.php

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-08-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Comment #27 from Jon Stanley [EMAIL PROTECTED]  2008-08-04 09:00:04 EDT 
---
Yep, but as I read that exception, it will only apply to new versions of ExtJS.
So here's what we'd need to do:

1) Package the new ExtJS in Fedora (Testopia bundles ExtJS with the tarball,
however, we don't allow private copies of libs in packages)

2) Either wait for upstream or help upstream port to the new version of ExtJS
that this exception applies to (probably not hard - might not even need
porting)

After that, we can have Testopia!

Spot - you never commented on the legal status of that exception, were it to
become final - here's our opportunity to influence it :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-08-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450013


Greg DeKoenigsberg [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
Customer Facing||---




--- Comment #26 from Greg DeKoenigsberg [EMAIL PROTECTED]  2008-08-04 
01:14:57 EDT ---
Any chance that the current situation can be clarified here?

Are we simply waiting for the draft document here:

http://extjs.com/products/floss-exception.php

...to become final?  What's the status?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-07-22 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-07-22 08:17 EST ---
There's a draft of the FLOSS exception available at
http://extjs.com/products/floss-exception.php

It looks like ExtJS 2.2 will be licensed like this (which looks fine to me, but
IANAL)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-07-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-07-15 13:52 EST ---
extjs.com have hinted at allowing an exception for FLOSS yet GPL-incompatible
applications at
http://extjs.com/blog/2008/04/27/open-source-license-exception-for-extensions/

In the hope that this reaches fruition I've filed a review request for the GPLv3
version of extjs as bug 455474.



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-07-08 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-07-08 15:08 EST ---
A 6384971 byte zipfile is downloadable from the extjs website from this URL:
http://extjs.com/deploy/ext-2.0.2.zip
with this m5dsum:
afb2619b828f8b6d947aff3c02340159  ext-2.0.2.zip

I've uploaded the LICENSE.txt from that zipfile to
https://bugzilla.redhat.com/attachment.cgi?id=311307

The md5sums of the extjs files in the testopia tarball don't all match that of
the zipfile from extjs.com, so I don't think we can attain option 2 this way.  

Aiming for option 1, I've packaged the content of the zipfile up for review 
here:
https://bugzilla.redhat.com/show_bug.cgi?id=454480
naming it extjs-lgpl, since the license appears to be the LGPL, however, I am
not a lawyer.

I've tested removing the extjs files from testopia and replacing with a symlink
to the extjs-lgpl payload, and it works, providing that I delete the .htacess
file.  We could add a Requires to the testopia package accordingly.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-07-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-07-07 14:46 EST ---
Lawyer says that, to the extent that ExtJS is licensed under GPLv3, there
is a license incompatibility here because of GPLv3/MPL incompatibility,
and thus the risk of violation of ExtJS's copyright. If LGPLv3 is the license,
on the other hand, there is no compatibility problem.

At one point, ExtJS was released under LGPLv3, but they've since changed to
release under GPLv3. Unfortunately, the testopia code doesn't give any
indication of when they pulled in their local copy of ExtJS, and all of the code
refers back to the website for licensing, which says GPLv3, so we can only
safely assume GPLv3.

The options here are:

1. Find a checkout of ExtJS that we know was released under the LGPLv3 and
replace what is in testopia with that.
2. Prove that the ExtJS bits in testopia were released under the LGPLv3 license.
3. Move testopia to use OpenExt (http://sourceforge.net/projects/openext/)
4. Don't put testopia in Fedora.

This package is legal blocked until one of those outcomes is reached.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-07-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-07-07 16:14 EST ---
There is a discussion of licensing issues upstream at
https://bugzilla.mozilla.org/show_bug.cgi?id=430138

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-07-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-07-03 12:00 EST ---
Upstream released 2.0-RC2; have uploaded new spec to:
http://dmalcolm.fedorapeople.org/testopia.spec
and new SRPM to:
http://dmalcolm.fedorapeople.org/testopia-3.0.4-1_5.tr2.0_RC2.fc10.src.rpm

From the ChangeLog:
* Wed Jun 25 2008 David Malcolm [EMAIL PROTECTED] - 3.0.4-1_5.tr2.0_RC2
- 2.0-RC2; needs perl-JSON = 2.10
- introduce -selftests subpackage to avoid having main package pull in all
of the various dependencies of the upstream Testopia test suite
- add perl(testopia::t::Constants) to the unwanted autogenerated requirements


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-07-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-07-03 19:27 EST ---
Upstream released 2.0 final today; have uploaded new spec to:
http://dmalcolm.fedorapeople.org/testopia.spec
and new SRPM to:
http://dmalcolm.fedorapeople.org/testopia-3.0.4-1_6.tr2.0.fc10.src.rpm

From the %changelog:
* Thu Jul  3 2008 David Malcolm [EMAIL PROTECTED] - 3.0.4-1_6.tr2.0
- 2.0
- add import_example.csv to -doc subpackage


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-25 14:17 EST ---
I've reworked things to try to make this parallel-installable with bugzilla, and
use /testopia on the server
Updated specfile at same location: 
http://dmalcolm.fedorapeople.org/testopia.spec
and SRPM at:
http://dmalcolm.fedorapeople.org/testopia-3.0.4-1_4.tr2.0_RC1.fc10.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-17 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-17 18:36 EST ---
I've been experimenting with the python client-side helper module for XML-RPC
interface in the contrib subpackage, and moved it to its own -python subpackage,
installing it to site-libs so that it's accessible easily with import testopia
in client code.  Some notes on doing this are at:
https://fedoraproject.org/wiki/QA/Testopia/XmlRpc

Updated specfile at same location: 
http://dmalcolm.fedorapeople.org/testopia.spec
and SRPM at:
http://dmalcolm.fedorapeople.org/testopia-3.0.4-1_3.tr2.0_RC1.fc10.src.rpm


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013


Bug 450013 depends on bug 450412, which changed state.

Bug 450412 Summary: please build in EL-5
https://bugzilla.redhat.com/show_bug.cgi?id=450412

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||WORKSFORME



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-11 14:29 EST ---
Re perl-GD-Graph3d: doh!

Updated specfile at same location: 
http://dmalcolm.fedorapeople.org/testopia.spec
and SRPM: 
http://dmalcolm.fedorapeople.org/testopia-3.0.4-1_2.tr2.0_RC1.fc10.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-10 08:12 EST ---
I don't think the fact that it doesn't work is entirely your fault :). I found
(via filing at b.m.o) that the latest version of perl-JSON doesn't work with
Testopia. Greg is aiming to having this resolved by the time 2.0 final comes
out, but it's a little hairy, In the meanwhile, I've posted F9 and EL-5 versions
of what does work at http://i.am.jds2001.org/temp

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-10 08:14 EST ---
Oh, and the install at http://i.am.jds2001.org/bugzilla/ is alive and kicking
with this new module (and you have admin rights to that bugzilla installation).
The one thing that I've found is that if you add a component to an environment,
for some reason you can't rename it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-10 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013


[EMAIL PROTECTED] changed:

   What|Removed |Added

   External Bug||http://bugzilla.mozilla.org/
Tracker URL||show_bug.cgi?id=409309




--- Additional Comments From [EMAIL PROTECTED]  2008-06-10 16:39 EST ---
Thanks; the bug you filed on mozilla.org _was_ the issue I'm seeing; for later
reference you filed it on b.m.o as 437828, resolved as dup of 409309; adding
reference to latter to this bug


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-09 13:50 EST ---
Problem was in my patching of bz_locations; 
extensionsdir was pointing at: /var/lib/bugzilla/extensions (empty) rather than:
/usr/share/bugzilla/extensions (which has
testopia/code/db_schema-abstract_schema.pl); hence not finding the testopia
extenstion and thus not extending the schema to cover the Testopia tables in
Bugzilla/DB/Schema.pm:_initialize

Working on a revised version, and on the other issues.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-09 18:03 EST ---
Created an attachment (id=308756)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=308756action=view)
extjs license as it appears in testopia tarball

The testopia/extjs/LICENSE.txt license as it appears in the testopia tarball
seems to indicate that it is LGPLv3.  However the license text on
http://extjs.com/products/license-faq.php seems to indicate GPLv3

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-09 22:11 EST ---
Updated SRPM here:
http://dmalcolm.fedorapeople.org/testopia-3.0.4-1_1.tr2.0_RC1.fc10.src.rpm
and specfile at:
http://dmalcolm.fedorapeople.org/testopia.spec

This is still a work-in-progress, I'm afraid; it uses the correct extensions
location and thus correctly populates the schema, but seeing some issues trying
to actually creating test plans and cases.

Fixing the extensions location revealed that Testopia adds a hard requirement on
GD-Graph3D which stops checkinstall.pl

It appears to only be referenced in the code at
./template/en/default/testopia/reports/completion.png.tmpl:USE my_graph
= GD.Graph.bars3d(275,250);
i.e. in one particular report, so shouldn't impact full functionality of the
package beyond that.  The above srpm patches out the requirement for now.
perl-GD-Graph3D doesn't seem to be packaged for Fedora yet.  Probably should fix
that before clearing review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-09 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-09 22:14 EST ---
Issue with extensionsdir filed against bugzilla package as bug 450636

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-08 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013


Bug 450013 depends on bug 450411, which changed state.

Bug 450411 Summary: please build in EL-5
https://bugzilla.redhat.com/show_bug.cgi?id=450411

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||182235
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-06-07 15:31 EST ---
OK - Package meets naming and packaging guidelines
OK? - Spec file matches base package name.
OK Spec has consistant macro usage.
OK Meets Packaging Guidelines.
SEE NOTES - License
MUSTFIX - License field in spec matches
MUSTFIX - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:

29847f8f095d56212e167a0bde1b2d70  testopia-2.0-RC1.tar.gz
29847f8f095d56212e167a0bde1b2d70  ../build/SOURCES/testopia-2.0-RC1.tar.gz
--
f55a3f3cde9cf1bf56492d18c8f7afe4  ../bugzilla-3.0.4.tar.gz
f55a3f3cde9cf1bf56492d18c8f7afe4  ../build/SOURCES/bugzilla-3.0.4.tar.gz

N/A - Package needs ExcludeArch
OK - BuildRequires correct
N/A - Spec handles locales/find_lang
N/A - Package is relocatable and has a reason to be.
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
OK- Package is code or permissible content.
OK - Doc subpackage needed/used.
OK - Packages %doc files don't affect runtime.

N/A - Headers/static libs in -devel subpackage.
N/A - Spec has needed ldconfig in post and postun
N/A - .pc files in -devel subpackage/requires pkgconfig
N/A - .so files in -devel subpackage.
N/A - -devel package Requires: %{name} = %{version}-%{release}
N/A - .la files are removed.

N/A - Package is a GUI app and has a .desktop file

OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
SEE NOTES - Package doesn't own any directories other packages own.
OK- Package owns all the directories it creates.
SEE NOTES- No rpmlint output:
[EMAIL PROTECTED] testopia]$ rpmlint testopia-3.0.4-1.tr2.0_RC1.fc10.noarch.rpm 
testopia.noarch: E: non-executable-script /usr/share/bugzilla/cron.daily 0644
testopia.noarch: E: non-standard-gid /etc/bugzilla apache
testopia.noarch: E: non-standard-dir-perm /etc/bugzilla 0750
testopia.noarch: W: invalid-license MPL

OK - final provides and requires are sane:
(include output of for i in *rpm; do echo $i; rpm -qp --provides $i; echo =; rpm
-qp --requires $i; echo; done

manually indented after checking each line.  I also remove the rpmlib junk and
anything provided by glibc.)
testopia-3.0.4-1.tr2.0_RC1.fc10.noarch.rpm
config(testopia) = 3.0.4-1.tr2.0_RC1.fc10
perl(Bugzilla)  
perl(Bugzilla::Attachment)  
perl(Bugzilla::Attachment::PatchReader)  
perl(Bugzilla::Auth)  
perl(Bugzilla::Auth::Login)  
perl(Bugzilla::Auth::Login::CGI)  
perl(Bugzilla::Auth::Login::Cookie)  
perl(Bugzilla::Auth::Login::Env)  
perl(Bugzilla::Auth::Login::Stack)  
perl(Bugzilla::Auth::Persist::Cookie)  
perl(Bugzilla::Auth::Verify)  
perl(Bugzilla::Auth::Verify::DB)  
perl(Bugzilla::Auth::Verify::LDAP)  
perl(Bugzilla::Auth::Verify::Stack)  
perl(Bugzilla::Bug)  
perl(Bugzilla::BugMail)  
perl(Bugzilla::CGI)  
perl(Bugzilla::Chart)  
perl(Bugzilla::Classification)  
perl(Bugzilla::Component)  
perl(Bugzilla::Config)  
perl(Bugzilla::Config::Admin)  
perl(Bugzilla::Config::Attachment)  
perl(Bugzilla::Config::Auth)  
perl(Bugzilla::Config::BugChange)  
perl(Bugzilla::Config::BugFields)  
perl(Bugzilla::Config::BugMove)  
perl(Bugzilla::Config::Common)  
perl(Bugzilla::Config::Core)  
perl(Bugzilla::Config::DependencyGraph)  
perl(Bugzilla::Config::GroupSecurity)  
perl(Bugzilla::Config::L10n)  
perl(Bugzilla::Config::LDAP)  
perl(Bugzilla::Config::MTA)  
perl(Bugzilla::Config::PatchViewer)  
perl(Bugzilla::Config::Query)  
perl(Bugzilla::Config::ShadowDB)  
perl(Bugzilla::Config::Testopia)  
perl(Bugzilla::Config::UserMatch)  
perl(Bugzilla::Constants)  
perl(Bugzilla::DB)  
perl(Bugzilla::DB::Mysql)  
perl(Bugzilla::DB::Pg)  
perl(Bugzilla::DB::Schema)  
perl(Bugzilla::DB::Schema::Mysql)  
perl(Bugzilla::DB::Schema::Pg)  
perl(Bugzilla::Error)  
perl(Bugzilla::Field)  
perl(Bugzilla::Flag)  
perl(Bugzilla::FlagType)  
perl(Bugzilla::Group)  
perl(Bugzilla::Hook)  
perl(Bugzilla::Install)  
perl(Bugzilla::Install::DB)  
perl(Bugzilla::Install::Filesystem)  
perl(Bugzilla::Install::Localconfig)  
perl(Bugzilla::Install::Requirements)  
perl(Bugzilla::Keyword)  
perl(Bugzilla::Mailer)  
perl(Bugzilla::Milestone)  
perl(Bugzilla::Object)  
perl(Bugzilla::Product)  
perl(Bugzilla::Search)  
perl(Bugzilla::Search::Quicksearch)  
perl(Bugzilla::Search::Saved)  
perl(Bugzilla::Series)  

[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-07 15:37 EST ---
Some things I forgot :)

5. The package owns directories that the bugzilla package proper owns, however,
there's a Conflicts, so I don't think that's a big problem.

6. The license text is not included as a %doc in the package

7. The subpackages don't require the base package

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||450411




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||450412




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-07 18:08 EST ---
This particular package doesn't actually install and work.  I've e-mailed Davve
about it, it exhibits various states of fail (though maybe it's the operator) 
:).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-07 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-07 20:32 EST ---
No, it was me; I broke the schema installation of the testopia-specific tables;
I'd only been testing on pre-created databases.  Will investigate; sorry

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-04 15:16 EST ---
Created an attachment (id=308382)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=308382action=view)
For reference, patch showing what I did to bugzilla.spec to get to
testopia.spec


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450013] Review Request: testopia - bugzilla extended to add test case management

2008-06-04 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: testopia - bugzilla extended to add test case 
management


https://bugzilla.redhat.com/show_bug.cgi?id=450013





--- Additional Comments From [EMAIL PROTECTED]  2008-06-04 15:43 EST ---
Successful scratch build here:
http://koji.fedoraproject.org/koji/taskinfo?taskID=646051

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review