Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 176528 - Review Request: MochiKit: A lightweight JavaScript library
Summary: Review Request: MochiKit: A lightweight JavaScript library
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: David Lawrence
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-12-24 09:51 UTC by Ignacio Vazquez-Abrams
Modified: 2009-02-14 00:49 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-02-15 23:41:30 UTC
Type: ---
Embargoed:
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Ignacio Vazquez-Abrams 2005-12-24 09:51:11 UTC
Spec Name or Url: http://fedora.ivazquez.net/files/extras/MochiKit.spec
SRPM Name or Url: http://fedora.ivazquez.net/files/extras/MochiKit-1.1-1.src.rpm
Description: MochiKit is a highly-documented and well-tested suite of JavaScript libraries that will help you get stuff done, fast.

Comment 1 Jason Tibbitts 2006-02-02 15:23:14 UTC
FYI, 1.2 is out.

rpmlint says:
W: MochiKit invalid-license MIT/Academic

The code is indeed under both MIT and the Academic Free license (
http://www.opensource.org/licenses/afl-2.1.php ) and so meets the packaging
guidelines but I don't know how to satisfy rpmlint.

Are there selinux isues with pointing the web server into /usr/share?  Perhaps
/var/www would be a better location?  (It kind of gives me indigestion to put
such things under /var, but there is precedent in httpd-manual.)

Comment 2 Ignacio Vazquez-Abrams 2006-02-02 15:33:26 UTC
(In reply to comment #1)
> Are there selinux isues with pointing the web server into /usr/share?  Perhaps
> /var/www would be a better location?  (It kind of gives me indigestion to put
> such things under /var, but there is precedent in httpd-manual.)

Something I discovered (which gives *me* indigestion...) is that httpd has full
read access to usr_t. Yes, that's right, Apache can read almost anything under /usr.

I will update the package to 1.2 when I get a chance.

Comment 3 Ignacio Vazquez-Abrams 2006-02-08 06:34:32 UTC
Updated.

http://fedora.ivazquez.net/files/extras/MochiKit-1.2-1.src.rpm

Comment 4 Jason Tibbitts 2006-02-10 23:32:58 UTC
Sorry for taking so long....

After seeing the Cacti package go in, I'll drop any objection I had to storing
the files in /usr/share.  And rpmlint only complains about the license, which is
not an issue since the license is valid.

So:
No rpmlint blockers, just the end-of-line warning.
Package meets naming and packaging guidelines.
License is acceptable and matches License: tag.
Specfile is properly named, legible, well-written, well-commented and uses
macros consistently.
Source file matches upstream.
Package builds and installs on FC3 and FC4.

I'd just like clarification on one thing.  It's not common practise to install
package tests under %doc (or even to install them at all).  I understand that
it's not really possible to execute those tests in any meaningful way at build
time, but I wonder what your reasoning is behind including them in the final
package.  Examples are already included, so the tests don't really add much in
the way of documentation.

Comment 5 Ignacio Vazquez-Abrams 2006-02-12 06:55:40 UTC
You're right, it doesn't make much sense. Updated.

Comment 6 Jason Tibbitts 2006-02-13 01:11:40 UTC
Cool.  Approved.

Comment 7 Ignacio Vazquez-Abrams 2006-02-15 23:41:30 UTC
Built under FC4 and devel.

Comment 8 Adam Miller 2009-02-14 00:31:10 UTC
Package Change Request
======================
Package Name: MochiKit
New Branches: EL-4 EL-5
Owners: maxamillion

Comment 9 Kevin Fenzi 2009-02-14 00:49:36 UTC
cvs done.


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