Bug 857992 - Review Request: JQuery - Fast, concise library that simplifies how you use JavaScript
Summary: Review Request: JQuery - Fast, concise library that simplifies how you use Ja...
Keywords:
Status: CLOSED DUPLICATE of bug 1078371
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 805587 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-09-17 17:24 UTC by Gregor Tätzner
Modified: 2014-03-19 16:56 UTC (History)
12 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-03-19 16:51:11 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Gregor Tätzner 2012-09-17 17:24:59 UTC
Another try of getting jquery into fedora ;)...

Spec URL: http://brummbq.fedorapeople.org/jquery.spec
SRPM URL: http://brummbq.fedorapeople.org/jquery-1.7.2-2.fc17.src.rpm
Description: jQuery is a fast, concise, JavaScript library that simplifies how you
traverse HTML documents, handle events, perform animations, and add
Ajax interactions to your web pages. jQuery is designed to change the
way that you write JavaScript.
Fedora Account System Username: brummbq

Comment 1 Gregor Tätzner 2012-09-17 17:26:43 UTC
*** Bug 805587 has been marked as a duplicate of this bug. ***

Comment 2 Matthias Runge 2012-09-17 20:26:12 UTC
I'm asking myself currently, if it is really desirable to make a package. 
Reason: many web based frameworks and other packages bundle jquery in any form. Mostly, they are tailored to a specific jquery version, so unbundling is not really an option. 

When jquery becomes a package: what happens with the packages bundling (more or less) silently jquery? Take django as example. It's upstream voted against unbundling; Security reasons are also not that problem, because it's client side, and should/will be interpreted in a sandbox.

To be more productive:
I'd define a <Directory> in your httpd-conf snipped, and to make it more readable, I'd put that everything in a separate file.

Comment 3 Gregor Tätzner 2012-09-17 21:11:27 UTC
Well, I'm not sure...bundling is bundling. Do we have an official position about javascript libraries? But if upstream is really shipping unmodified jquery libs, I don't see why we couldn't provide packages for that...in the worst case in form of parallel-installable packages containing different jquery versions.

Anyway if you would be able to obtain a global exception for jquery* bundling I would drop this package happily :)

Comment 4 Felix Kaechele 2012-09-22 12:01:00 UTC
As of now there is no best practice for bundling (or unbundling) JavaScript libraries.
The issue raises a few questions: For example, what _may_ be bundled, what _must not_ be bundled.
Think of jQuery plugins: Do we package each plugin seperately? What if a webapp bundles a plugin that is only part of that particular webapp?
Do we unbundle it as well?
I guess we need to fundamentally discuss this. I'm unsure if we can make this in the F18 timeframe.

Comment 5 Matthias Runge 2012-09-26 08:46:47 UTC
jquery: latest version is (currently) 1.8.2. , 1.8.1 was released on Aug 30, so you'l need to update this package.

In my experience, esp. web developers are pretty ignorant in bundling/copying external code. And also, most of them don't care for backwards compatibility.

For example: Django developers: they said about the request to unbundle: Not having external dependencies is a design choice of Django.
https://code.djangoproject.com/ticket/17982
Django still uses jquery 1.4.2, and I don't see, this will change in the near future.

jquery's release notes list some changes affecting backwards compatibility:
http://blog.jquery.com/2012/08/09/jquery-1-8-released/

When breaking other's packages by upgrading to newer versions, you're also requested to fix those issues. I'm sure, I don't want to do that.

Comment 6 Gregor Tätzner 2012-09-26 18:38:45 UTC
(In reply to comment #4)
> Think of jQuery plugins: Do we package each plugin seperately? What if a
> webapp bundles a plugin that is only part of that particular webapp?
> Do we unbundle it as well?
> I guess we need to fundamentally discuss this. I'm unsure if we can make
> this in the F18 timeframe.

Regarding jQuery plugins I think it does not harm if we put everything together in one package? Of course if a js lib is unique/forked it doesn't make sense to distribute it separately.


(In reply to comment #5)
> jquery: latest version is (currently) 1.8.2. , 1.8.1 was released on Aug 30,
> so you'l need to update this package.

If I'm obliged to build it from source I've a problem because nodejs is still not in fedora. Or would it be ok to use the ready-to-use version from the website?

> 
> In my experience, esp. web developers are pretty ignorant in
> bundling/copying external code. And also, most of them don't care for
> backwards compatibility.
> 
> For example: Django developers: they said about the request to unbundle: Not
> having external dependencies is a design choice of Django.
> https://code.djangoproject.com/ticket/17982
> Django still uses jquery 1.4.2, and I don't see, this will change in the
> near future.

agreed, in such cases we should respect the decision of upstream.

> jquery's release notes list some changes affecting backwards compatibility:
> http://blog.jquery.com/2012/08/09/jquery-1-8-released/
> 
> When breaking other's packages by upgrading to newer versions, you're also
> requested to fix those issues. I'm sure, I don't want to do that.

I think we must provide different jquery packages right from the beginning i.e.: jquery1.7; jquery1.8 and use them explicitly as deps in the packages. If the maintainer knows for sure that his app supports the next jquery release he can bump it manually and go to a "higher" jquery package.

Comment 7 T.C. Hollingsworth 2013-01-17 09:09:40 UTC
Node.js is now available in Fedora 18 and Rawhide and should no longer block this package.

Comment 8 Gregor Tätzner 2013-01-18 13:27:14 UTC
(In reply to comment #7)
> Node.js is now available in Fedora 18 and Rawhide and should no longer block
> this package.

aye, node.js is a big step forward but I need also npm to install 'grunt'. The JQuery buildsystem has changed quite a bit in recent releases.

Comment 9 Gregor Tätzner 2013-04-12 10:17:29 UTC
@T.C.
Are you planning to package nodejs-grunt* (also grunt-init, grunt-cli). I've looked briefly over them and I worry that not all nodejs deps for grunt are fulfilled in fedora 18.

Comment 10 T.C. Hollingsworth 2013-04-25 08:51:46 UTC
I need a test case to finish up npm2rpm, so might as well kill two birds with one stone.  ;-)

For the record, the missing deps are:
└─┬ grunt.1
  ├── colors.0-1
  ├── dateformat.2-1.2.3
  ├── eventemitter2.11
  ├─┬ findup-sync.2
  │ └── lodash.1
  ├── hooker.3
  ├── iconv-lite.8
  ├─┬ js-yaml.4
  │ └─┬ argparse.13
  │   ├── underscore.4
  │   └── underscore.string.1
  ├── lodash.2
  └── underscore.string.0rc

I can probably get these up for review next week sometime.

Comment 11 T.C. Hollingsworth 2013-04-25 09:05:03 UTC
Oops, forgot about grunt-cli and grunt-init.  The full list of packages is really:
├─┬ grunt.1
│ ├── colors.0-1
│ ├── dateformat.2-1.2.3
│ ├── eventemitter2.11
│ ├─┬ findup-sync.2
│ │ └── lodash.1
│ ├── hooker.3
│ ├── iconv-lite.8
│ ├─┬ js-yaml.4
│ │ └─┬ argparse.13
│ │   ├── underscore.4
│ │   └── underscore.string.1
│ ├── lodash.2
│ └── underscore.string.0rc
├─┬ grunt-cli.7
│ ├─┬ findup-sync.2
│ │ └── lodash.1
│ └── resolve.1
└─┬ grunt-init.0
  ├── colors.0-1
  ├── hooker.3
  └─┬ prompt.12
    ├── async.22
    ├── pkginfo.0
    └─┬ winston.11
      ├─┬ loggly.11
      │ ├── request.203
      │ └── timespan.0
      └── stack-trace.6

If you want to help out with getting a head start on the reviews that will be needed, one of grunt-init's deps (winston) is already awaiting review in bug 911060.

Comment 12 Jamie Nguyen 2013-06-23 18:06:20 UTC
I didn't realise you'd made a fancy list of dependencies T.C.! I would have used it if I'd bothered to look.. Anyway I've packaged grunt, grunt-cli and grunt-init and they're all now up for review :)

Comment 13 T.C. Hollingsworth 2013-06-25 21:27:15 UTC
(In reply to Jamie Nguyen from comment #12)
> I didn't realise you'd made a fancy list of dependencies T.C.! I would have
> used it if I'd bothered to look.. Anyway I've packaged grunt, grunt-cli and
> grunt-init and they're all now up for review :)

That list was actually generated by npm.  To get it, do something like:
mkdir /tmp/foo
cd /tmp/foo
npm install bar
npm ls

Comes in handy when figuring out exactly how much work something will be to package.  ;-)

Comment 14 T.C. Hollingsworth 2013-09-19 12:20:11 UTC
devDependencies for jquery that still need to be packaged:
"archiver": "~0.4.9",
"grunt-compare-size": "~0.4.0",
"grunt-contrib-watch": "~0.5.3",
"grunt-git-authors": "~1.2.0",
"grunt-jsonlint": "~1.0.0",
"gzip-js": "0.3.2",
"testswarm": "~1.1.0",
"requirejs": "~2.1.8"

Also, this one is ineligible for Fedora due to its use of the "evil" JSON license, and will need to be patched out:
"grunt-contrib-jshint": "~0.6.4",

It's only used in the test suite so that's not a big deal.

Comment 15 Vít Ondruch 2014-02-03 14:03:26 UTC
Can't see sizzle between dependencies you list ...

Comment 16 T.C. Hollingsworth 2014-03-19 16:51:11 UTC
This bug is now being closed duplicate of the review request for js-jquery1, which provides the latest version of the 1.x jQuery branch.  Most web applications depend on this version.

If your application depends on jQuery 2.x, please manually change its dependency to bug 1078368.  Thanks!

*** This bug has been marked as a duplicate of bug 1078371 ***


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