https://bugzilla.redhat.com/show_bug.cgi?id=1115659
Bug ID: 1115659 Summary: Review Request: bootstrap - A front-end framework for developing projects on the web Product: Fedora Version: rawhide Component: Package Review Assignee: nobody@fedoraproject.org Reporter: rbean@redhat.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: http://ralph.fedorapeople.org//bootstrap.spec SRPM URL: http://ralph.fedorapeople.org//bootstrap-3.1.1-2.fc20.src.rpm
Description: A front-end framework for developing responsive, mobile first projects on the web.
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
Ralph Bean rbean@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |1115668, 1115671, 1115673, | |1115674, 1115665, 1115672, | |1115678, 1115670, 1115676, | |1115677
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1115665 [Bug 1115665] Review Request: nodejs-csscomb - CSS coding style formatter https://bugzilla.redhat.com/show_bug.cgi?id=1115668 [Bug 1115668] Review Request: nodejs-grunt-autoprefixer - Parse CSS and add vendor-prefixed CSS properties https://bugzilla.redhat.com/show_bug.cgi?id=1115670 [Bug 1115670] Review Request: nodejs-grunt-contrib-connect - Start a connect web server https://bugzilla.redhat.com/show_bug.cgi?id=1115671 [Bug 1115671] Review Request: nodejs-grunt-contrib-copy - Copy files and folders https://bugzilla.redhat.com/show_bug.cgi?id=1115672 [Bug 1115672] Review Request: nodejs-grunt-contrib-csslint - Lint CSS files https://bugzilla.redhat.com/show_bug.cgi?id=1115673 [Bug 1115673] Review Request: nodejs-grunt-contrib-cssmin - Compress CSS files https://bugzilla.redhat.com/show_bug.cgi?id=1115674 [Bug 1115674] Review Request: nodejs-grunt-contrib-less - Compile LESS files to CSS https://bugzilla.redhat.com/show_bug.cgi?id=1115676 [Bug 1115676] Review Request: nodejs-grunt-html-validation - W3C html validation grunt plugin https://bugzilla.redhat.com/show_bug.cgi?id=1115677 [Bug 1115677] Review Request: nodejs-grunt-jscs-checker - Grunt task for checking JavaScript Code Style with jscs https://bugzilla.redhat.com/show_bug.cgi?id=1115678 [Bug 1115678] Review Request: nodejs-grunt-lib-contrib - Common functionality shared across grunt-contrib tasks
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
Ralph Bean rbean@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |1115669
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1115669 [Bug 1115669] Review Request: nodejs-grunt-banner - Adds a simple banner to files
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
Jamie Nguyen jamielinux@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends On| |977126
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=977126 [Bug 977126] Review Request: nodejs-grunt-lib-contrib - Common functionality shared across grunt-contrib tasks
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115678, which changed state.
Bug 1115678 Summary: Review Request: nodejs-grunt-lib-contrib - Common functionality shared across grunt-contrib tasks https://bugzilla.redhat.com/show_bug.cgi?id=1115678
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |DUPLICATE
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115671, which changed state.
Bug 1115671 Summary: Review Request: nodejs-grunt-contrib-copy - Copy files and folders https://bugzilla.redhat.com/show_bug.cgi?id=1115671
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115672, which changed state.
Bug 1115672 Summary: Review Request: nodejs-grunt-contrib-csslint - Lint CSS files https://bugzilla.redhat.com/show_bug.cgi?id=1115672
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115669, which changed state.
Bug 1115669 Summary: Review Request: nodejs-grunt-banner - Adds a simple banner to files https://bugzilla.redhat.com/show_bug.cgi?id=1115669
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115673, which changed state.
Bug 1115673 Summary: Review Request: nodejs-grunt-contrib-cssmin - Compress CSS files https://bugzilla.redhat.com/show_bug.cgi?id=1115673
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115670, which changed state.
Bug 1115670 Summary: Review Request: nodejs-grunt-contrib-connect - Start a connect web server https://bugzilla.redhat.com/show_bug.cgi?id=1115670
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115676, which changed state.
Bug 1115676 Summary: Review Request: nodejs-grunt-html-validation - W3C html validation grunt plugin https://bugzilla.redhat.com/show_bug.cgi?id=1115676
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115674, which changed state.
Bug 1115674 Summary: Review Request: nodejs-grunt-contrib-less - Compile LESS files to CSS https://bugzilla.redhat.com/show_bug.cgi?id=1115674
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |RAWHIDE
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
Ralph Bean rbean@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |WONTFIX Last Closed| |2016-04-04 12:02:21
--- Comment #1 from Ralph Bean rbean@redhat.com --- I'm no longer pursuing this.
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115668, which changed state.
Bug 1115668 Summary: Review Request: nodejs-grunt-autoprefixer - Parse CSS and add vendor-prefixed CSS properties https://bugzilla.redhat.com/show_bug.cgi?id=1115668
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |WONTFIX
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115677, which changed state.
Bug 1115677 Summary: Review Request: nodejs-grunt-jscs-checker - Grunt task for checking JavaScript Code Style with jscs https://bugzilla.redhat.com/show_bug.cgi?id=1115677
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |WONTFIX
https://bugzilla.redhat.com/show_bug.cgi?id=1115659 Bug 1115659 depends on bug 1115665, which changed state.
Bug 1115665 Summary: Review Request: nodejs-csscomb - CSS coding style formatter https://bugzilla.redhat.com/show_bug.cgi?id=1115665
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution|--- |WONTFIX
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
--- Comment #2 from Dan Callaghan dcallagh@redhat.com --- Ralph, just wondering why you gave up on this? No longer needed, or too hard for some other reasons? Do you happen to remember what else would be needed to get this going again?
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
--- Comment #3 from Dan Callaghan dcallagh@redhat.com --- Specifically I was thinking it would be nice if resultsdb_frontend didn't reference off to some third party CDNs for downloading its CSS and JS dependencies... Looks like the Bootstrap bits are the only ones not already packaged in Fedora.
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
--- Comment #4 from Ralph Bean rbean@redhat.com --- Yeah, the dependencies to *build* bootstrap were enormous. I wrote a script (npm2spec[1]) to automate it, but getting through the hundreds of reviews took years. If I included buildtime dependencies to run the tests, there were over 1000 upstream projects that needed to be packaged. Even after I got some of these were approved, maintaining the nest of pinned-version deps between all the nodejs components turned out to be overwhelming.
I ended up building my own version of bootstrap and hosting a shared copy on fedoraproject.org (a kind of "local to fedora" CDN) or bundling it with the projects that needed it (this ended up being more common).
For resultsdb_frontend, I would recommend pointing to a copy of bootstrap using a URL that administrators can configure. By default, it can point to a CDN, but you can optionally host a local-to-your-site copy of bootstrap and configure resultsdb to request it from there.
[1] - https://github.com/ralphbean/npm2spec
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
--- Comment #5 from Ralph Bean rbean@redhat.com --- See this for the dep chain: https://bugzilla.redhat.com/showdependencytree.cgi?id=1115659&hide_resol...
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
--- Comment #6 from Dan Callaghan dcallagh@redhat.com --- Yeah, wow... I had a feeling it would be something nightmarish like that. I've had a bit of a taste of it myself from the handful of nodejs packages I'm maintaining.
The NPM ecosystem is in quite a sorry state IMHO, this trend towards publishing little four-line functions as their own entire separately-maintained, separately-versioned, separately-licensed "package" is dumb and just makes life painful for everyone...
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |ngompa13@gmail.com
--- Comment #7 from Neal Gompa ngompa13@gmail.com --- Just popping in to say that is a truly hideous dependency chain. I don't know how anyone could stand to deal with that...
Has anyone told the nodejs community that they're heading towards bad road?
https://bugzilla.redhat.com/show_bug.cgi?id=1115659
--- Comment #8 from Dan Callaghan dcallagh@redhat.com --- I thought the left-pad fiasco would have been a bit of a wake-up call for them, but it seems it wasn't...
package-review@lists.fedoraproject.org