Remove jshint as the license is not ASL compatible

Bug #1270764 reported by Matthias Runge
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Critical
Matthias Runge

Bug Description

We bundle jshint in horizon, but don't include the license, despite it says:

The above copyright notice and this permission notice shall be
included in all copies or substantial portions of the Software.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/67799

Changed in horizon:
assignee: nobody → Matthias Runge (mrunge)
status: Confirmed → In Progress
Revision history for this message
Matthias Runge (mrunge) wrote : Re: jshint license missing

turns out to be worse. jshint license uses this silly:

"The Software shall be used for Good, not Evil."

Which makes it incompatible.

Changed in horizon:
status: In Progress → Triaged
Revision history for this message
Matthias Runge (mrunge) wrote :

we need to drop that asap.

Revision history for this message
Matthias Runge (mrunge) wrote :

License clarification requested upstream: https://github.com/jshint/jshint/issues/1481

Revision history for this message
Matthias Runge (mrunge) wrote :

Original author said, he won't change jshint license, so it must be considered as non-free.

summary: - jshint license missing
+ license of jshint is not ASL compatible
summary: - license of jshint is not ASL compatible
+ Remove jshint as the license is not ASL compatible
Thierry Carrez (ttx)
Changed in horizon:
milestone: icehouse-3 → icehouse-2
Matthias Runge (mrunge)
Changed in horizon:
assignee: Matthias Runge (mrunge) → nobody
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/68268

Changed in horizon:
assignee: nobody → Matthias Runge (mrunge)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/68268
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=277b12affc3fe5fc21a8540acf82cfecad9cbbe0
Submitter: Jenkins
Branch: master

commit 277b12affc3fe5fc21a8540acf82cfecad9cbbe0
Author: Matthias Runge <email address hidden>
Date: Tue Jan 21 22:27:16 2014 +0100

    Revert "JSHint integration"

    This reverts commit 7711a78ac4365a04d49a69ca5238b7c2fc045f15.

    Closes-Bug: #1270764
    Change-Id: I259384716089b1cdb2c66e6777d54848de5fa58d

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: icehouse-2 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.