Uploaded image for project: 'Ranger'
  1. Ranger
  2. RANGER-541

Address 0.5 release packaging feedback

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Critical
    • Resolution: Fixed
    • 0.5.0
    • 0.6.0
    • None
    • Important

    Description

      Feedback from Justin Mclean: http://s.apache.org/NBH

      The LICENSE notes that you using JQuery Pine Notify which is triple licensed under GPL, LGPL and MPL. MPL is a category B license and as such needs to be handled with care [6]. However I’m not sure that it is actually bundled in the software - can you confirm this. If it is not it can be removed from the LICENSE.

      Permissive licenses such as Apache and MIT do not normally get mentioned in the NOTICE file [4] as the NOTICE file places a burden on downstream projects can these please be removed.

      The following seem to be missing from the LICENSE

      • font awesome (MIT + SIL) see security-admin/src/main/webapp/fonts/fontawesome/fontawesome-webfont.svg and ranger-0.5.0/security-admin/src/main/webapp/fonts/fontawesome/FontAwesome.* + ranger-0.5.0/security-admin/src/main/webapp/fonts/fontopensans/open-sans*
      • backbone forms (MIT) see ranger-0.5.0/security-admin/src/main/webapp/libs/bower/backbone-forms/*
      • select2 (MIT) see security-admin/src/main/webapp/libs/bower/select2/select2.css
      • bootstrap (MIT) see ranger-0.5.0/security-admin/src/main/webapp/themejs/1.3.0/bootstrap.min.js
      • QUnit (MIT) see security-admin/src/main/webapp/libs/bower/globalize/test/qunit/qunit.js
      • jsDump (BSD -part of QUnit) see security-admin/src/main/webapp/libs/bower/globalize/test/qunit/qunit.js
      • Sizzle.js (part of jQuery) see security-admin/src/main/webapp/libs/bower/globalize/examples/browser/jquery-1.4.4.js

      Also VisualSearch.js could be placed with the other MIT licenses.

      There is also no need to list Apache licensed software in LICENSE, however it’s not an licensing error, and up to you if you want to leave them there.

      For the next release can you please fix the following:

      • Add incubating to the release name [1]
      • Place the release in the correct place [2][3]
      • Put the contents of hashes in a standard format (making it easier to check)
      • Consider adding apache to release artefact name
      • Correct years in NOTICE file
      • Remove unnecessary information from NOTICE
      • Add missing licenses to LICENSE

      Note that the first two items are marked as MUST in the incubator policy.

      Thanks,
      Justin

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              bosco Bosco
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: