The nivo-slider releases for 2.7 and 2.7.1 actually nest the release a directory deep. This was not the case from my experience with 2.6, but that version of the library is not in the whitelisted urls for external libraries. http://drupal.org/project/drupalorg_whitelist
Example library extraction
jeff@feynman:/tmp$ unzip nivo-slider2.7.1.zip
Archive: nivo-slider2.7.1.zip
creating: __MACOSX/
creating: __MACOSX/nivo-slider/
inflating: __MACOSX/nivo-slider/._jquery.nivo.slider.js
inflating: __MACOSX/nivo-slider/._jquery.nivo.slider.pack.js
inflating: __MACOSX/nivo-slider/._nivo-slider.css
creating: __MACOSX/nivo-slider/demo/
inflating: __MACOSX/nivo-slider/demo/._demo.html
creating: __MACOSX/nivo-slider/demo/scripts/
inflating: __MACOSX/nivo-slider/demo/scripts/._jquery-1.7.1.min.js
creating: nivo-slider/
inflating: nivo-slider/README
creating: nivo-slider/demo/
inflating: nivo-slider/demo/demo.html
creating: nivo-slider/demo/images/
inflating: nivo-slider/demo/images/dev7logo.png
inflating: nivo-slider/demo/images/nemo.jpg
inflating: nivo-slider/demo/images/toystory.jpg
inflating: nivo-slider/demo/images/up.jpg
inflating: nivo-slider/demo/images/walle.jpg
creating: nivo-slider/demo/scripts/
inflating: nivo-slider/demo/scripts/jquery-1.7.1.min.js
inflating: nivo-slider/demo/style.css
inflating: nivo-slider/jquery.nivo.slider.js
inflating: nivo-slider/jquery.nivo.slider.pack.js
inflating: nivo-slider/license.txt
inflating: nivo-slider/nivo-slider.css
creating: nivo-slider/themes/
creating: nivo-slider/themes/default/
inflating: nivo-slider/themes/default/arrows.png
inflating: nivo-slider/themes/default/bullets.png
inflating: nivo-slider/themes/default/default.css
inflating: nivo-slider/themes/default/loading.gif
creating: nivo-slider/themes/orman/
inflating: nivo-slider/themes/orman/arrows.png
inflating: nivo-slider/themes/orman/bullets.png
inflating: nivo-slider/themes/orman/loading.gif
inflating: nivo-slider/themes/orman/orman.css
inflating: nivo-slider/themes/orman/readme.txt
inflating: nivo-slider/themes/orman/ribbon.png
inflating: nivo-slider/themes/orman/slider.png
creating: nivo-slider/themes/pascal/
inflating: nivo-slider/themes/pascal/bullets.png
inflating: nivo-slider/themes/pascal/controlnav.png
inflating: nivo-slider/themes/pascal/featured.png
inflating: nivo-slider/themes/pascal/loading.gif
inflating: nivo-slider/themes/pascal/pascal.css
inflating: nivo-slider/themes/pascal/readme.txt
inflating: nivo-slider/themes/pascal/ribbon.png
inflating: nivo-slider/themes/pascal/slider.png
This results in
nivo-slider/nivo-slider/
nivo-slider/__MACOSX/
This in general isn't an issue if the process is manual as a human will likely spot this and un-mangle, however if this is automated a packager will *correctly* extract the library such that the directory structure is 'libraries/nivo-slider/nivo-slider'. This is one extra directory structure.
IMO this is an issue with the way the nivo-slider library is being packaged and we should try to get the maintainer to adjust how the package is zipped so that we avoid having any wonky workarounds in the drupal module.
If the maintainer is not interested in adjusting this then I believe that this module should be updated to reflect the structure of the archive package.
See also: #1465816: nivo slider js library packaging/build issues
Comments
Comment #0.0
jgraham CreditAttribution: jgraham commentedFixing link to other issue
Comment #1
peronas CreditAttribution: peronas commentedHey jgraham,
Ideally this would be resolved on the packaging end as you said. Before I make any special accommodations in the module I'll try and reach out to the library maintainer and see if it is just a temporary oversight. Given that it hasn't unpacked this way in past releases that may be entirely possible.
Comment #2
jgraham CreditAttribution: jgraham commentedFollowed up via github https://github.com/gilbitron/Nivo-Slider/issues/270
Comment #2.0
jgraham CreditAttribution: jgraham commentedFixing link to other issue