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

Bug 176528

Summary: Review Request: MochiKit: A lightweight JavaScript library
Product: [Fedora] Fedora Reporter: Ignacio Vazquez-Abrams <ivazqueznet>
Component: Package ReviewAssignee: Jason Tibbitts <j>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-extras-list, j, maxamillion
Target Milestone: ---Flags: kevin: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-02-15 23:41:30 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 163779    

Description Ignacio Vazquez-Abrams 2005-12-24 09:51:11 UTC
Spec Name or Url:
SRPM Name or Url:
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 ( ) 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

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.

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.