[Bug 457343] Review Request: jquery - Fast, concise library that simplifies how you use javascript

bugzilla at redhat.com bugzilla at redhat.com
Wed Aug 31 12:44:14 UTC 2011


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=457343

Stijn Hoop <stijn at sandcat.nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |stijn at sandcat.nl

--- Comment #11 from Stijn Hoop <stijn at sandcat.nl> 2011-08-31 08:44:10 EDT ---
While this is surely a more general problem, I just happened upon this and
this:

> but I guess you need to then include "js/sizzle.js" before including
> "js/jquery.js" in your application. Would that be an acceptable requirement
> for packagers?

most certainly is not going to fly in the real world. Are you really suggesting
that every webapp needs to be patched by packagers to do this? Because
upstreams will certainly refuse such a patch anyway...

If this package makes it through and it *requires* modifications to packaged
webapps to make use of it, I would not be surprised if the number of packaged
webapps simply drops (but hey, please prove me wrong).

IMHO, comparing a javascript "library" to a .so is simply wrong to begin with,
and bundling here should be allowed. The only thing to make sure is that
security issues can be identified ASAP; which involves making it easy to
identify every .srpm that bundles a specific jquery version.

-- 
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.



More information about the package-review mailing list