From Fedora Project Wiki
(gzip-js)
m (Kevin moved page Changes/Openldap 25/jQuery to Talk:Changes//jQuery: undoing previous move)
 
(4 intermediate revisions by 4 users not shown)
Line 1: Line 1:
* '''npm(bowercopy)''' does not appear to be used during the build process. It also pulls in '''npm(bower)''' which is quite a significant dependency tree. I also don't think '''npm(jsonlint)''' and '''npm(jscs-checker)''' are strictly necessary for the package. They also have a sizeable dependency tree, and these kind of checks are for upstream developers anyway so they don't really provide much benefit during the package build process. Just trying to cut our workload a little... [[User:Jamielinux|Jamielinux]] ([[User talk:Jamielinux|talk]]) 19:58, 29 March 2014 (UTC)
* '''npm(bowercopy)''' does not appear to be used during the build process. It also pulls in '''npm(bower)''' which is quite a significant dependency tree. I also don't think '''npm(jsonlint)''' and '''npm(jscs-checker)''' are strictly necessary for the package. They also have a sizeable dependency tree, and these kind of checks are for upstream developers anyway so they don't really provide much benefit during the package build process. Just trying to cut our workload a little... [[User:Jamielinux|Jamielinux]] ([[User talk:Jamielinux|talk]]) 19:58, 29 March 2014 (UTC)
* '''npm(gzip-size)''' is also not particularly useful during the build process. The <code>compare_size</code> task compares <code>jquery.js</code> with <code>jquery-min.js</code>, and if '''npm(gzip-size)''' is available then it also compares the gzipped sizes. I'm not interested in packaging this set of dependencies for the reasons below. Of course if anybody else wants to package them then feel free! [[User:Jamielinux|Jamielinux]] ([[User talk:Jamielinux|talk]]) 13:55, 30 March 2014 (UTC)
 
** Well, bower would be NTH in general, but definitely down to drop it from jquery if it's unnecessary.  --[[User:Patches|Patches]] ([[User talk:Patches|talk]]) 00:06, 31 March 2014 (UTC)
 
* '''npm(gzip-js)''' is also not particularly useful during the build process. The <code>compare_size</code> task compares <code>jquery.js</code> with <code>jquery-min.js</code>, and if '''npm(gzip-js)''' is available then it also compares the gzipped sizes. I'm not interested in packaging this set of dependencies for the reasons below. Of course if anybody else wants to package them then feel free! [[User:Jamielinux|Jamielinux]] ([[User talk:Jamielinux|talk]]) 13:55, 30 March 2014 (UTC)
** We need a bundling exception for '''npm(deflate-js)''' which has a fork of [http://www.onicos.com/staff/iz/amuse/javascript/expert/deflate.txt deflate.txt] and [http://www.onicos.com/staff/iz/amuse/javascript/expert/inflate.txt inflate.txt].
** We need a bundling exception for '''npm(deflate-js)''' which has a fork of [http://www.onicos.com/staff/iz/amuse/javascript/expert/deflate.txt deflate.txt] and [http://www.onicos.com/staff/iz/amuse/javascript/expert/inflate.txt inflate.txt].
** '''npm(gzip-js)''' carries this warning: '''''"it should NOT be used on the server for production code. It also does not comply 100% with the standard, yet."'''''
** '''npm(gzip-js)''' carries this warning: '''''"it should NOT be used on the server for production code. It also does not comply 100% with the standard, yet."'''''
** '''npm(deflate-js)''' carries this warning: '''''"This module is not meant to be run on node for any production code. Currently deflate does not pass all tests, but inflate does. This should not be used for compressing data yet in production."'''''
** '''npm(deflate-js)''' carries this warning: '''''"This module is not meant to be run on node for any production code. Currently deflate does not pass all tests, but inflate does. This should not be used for compressing data yet in production."'''''
** The jquery Gruntfile.js can be [http://jamielinux.fedorapeople.org/gruntjs/jquery-1.11.0-Do-not-use-gzip-js-during-the-build-process.patch easily patched] to not use gzip-js.
** The jquery Gruntfile.js can be [http://jamielinux.fedorapeople.org/gruntjs/jquery-1.11.0-Do-not-use-gzip-js-during-the-build-process.patch easily patched] to not use gzip-js.
: Total agreement here, thanks for looking into this and coming up with a patch.  --[[User:Patches|Patches]] ([[User talk:Patches|talk]]) 00:06, 31 March 2014 (UTC)
== testing ==
"Try a converted web application and make sure it still works.  :-) "
It's really not that easy, is it?  This places a burden on the webapp maintainer to retest with every system-updated copy of jquery, unless jquery provides a backward compatibility guarantee as strong as shared libraries.
[[User:Fche|Fche]] ([[User talk:Fche|talk]]) 20:48, 9 July 2015 (UTC)

Latest revision as of 20:09, 25 October 2021

  • npm(bowercopy) does not appear to be used during the build process. It also pulls in npm(bower) which is quite a significant dependency tree. I also don't think npm(jsonlint) and npm(jscs-checker) are strictly necessary for the package. They also have a sizeable dependency tree, and these kind of checks are for upstream developers anyway so they don't really provide much benefit during the package build process. Just trying to cut our workload a little... Jamielinux (talk) 19:58, 29 March 2014 (UTC)
    • Well, bower would be NTH in general, but definitely down to drop it from jquery if it's unnecessary. --Patches (talk) 00:06, 31 March 2014 (UTC)
  • npm(gzip-js) is also not particularly useful during the build process. The compare_size task compares jquery.js with jquery-min.js, and if npm(gzip-js) is available then it also compares the gzipped sizes. I'm not interested in packaging this set of dependencies for the reasons below. Of course if anybody else wants to package them then feel free! Jamielinux (talk) 13:55, 30 March 2014 (UTC)
    • We need a bundling exception for npm(deflate-js) which has a fork of deflate.txt and inflate.txt.
    • npm(gzip-js) carries this warning: "it should NOT be used on the server for production code. It also does not comply 100% with the standard, yet."
    • npm(deflate-js) carries this warning: "This module is not meant to be run on node for any production code. Currently deflate does not pass all tests, but inflate does. This should not be used for compressing data yet in production."
    • The jquery Gruntfile.js can be easily patched to not use gzip-js.
Total agreement here, thanks for looking into this and coming up with a patch. --Patches (talk) 00:06, 31 March 2014 (UTC)

testing

"Try a converted web application and make sure it still works.  :-) "

It's really not that easy, is it? This places a burden on the webapp maintainer to retest with every system-updated copy of jquery, unless jquery provides a backward compatibility guarantee as strong as shared libraries.

Fche (talk) 20:48, 9 July 2015 (UTC)