search

opentechinstitute / commotion-feed

Feed of Commotion components to be pulled in by OpenWRT
GNU Affero General Public License v3.0
5 stars 11 forks source link

luci-commotion-apps does not install assets properly #55

Closed areynold closed 10 years ago

areynold commented 10 years ago

Since the repo merge, luci-commotion-apps fails to install images, stylesheets, and javascript properly. The package assets are available in buildroot's staging directory and are actually available in /www/luci-commotion/apps/, but are not listed by opkg files luci-commotion-apps. The files should be installed in /www/luci-static/commotion/.

This is the list of files installed on the router: /usr/lib/lua/luci/controller/commotion/apps_controller.lua /usr/lib/lua/luci/view/commotion/apps_view.htm /usr/lib/lua/luci/model/cbi/commotion/app_settings.lua /usr/lib/lua/luci/view/themes/commotion/apps_apps.htm /usr/lib/lua/luci/view/commotion/apps_settings.htm /usr/lib/lua/luci/view/commotion/apps_display.htm /usr/lib/lua/luci/view/themes/commotion/apps_categories.htm /usr/lib/lua/luci/view/commotion/apps_form.htm /usr/lib/lua/luci/view/commotion/apps_admin_display.htm

This is the full list of files that should be installed: /www/luci-static/commotion/code.png /www/luci-static/commotion/equals.png /www/luci-static/commotion/radio.png /www/luci-static/commotion/apps_form.js /www/luci-static/commotion/chat.png /www/luci-static/commotion/audio.png /www/luci-static/commotion/apps.js /usr/lib/lua/luci/controller/commotion/apps_controller.lua /usr/lib/lua/luci/view/commotion/apps_view.htm /www/luci-static/commotion/x.png /usr/lib/lua/luci/model/cbi/commotion/app_settings.lua /usr/lib/lua/luci/view/themes/commotion/apps_apps.htm /usr/lib/lua/luci/view/commotion/apps_settings.htm /www/luci-static/commotion/write.png /www/luci-static/commotion/email.png /www/luci-static/commotion/commotion-faded2.png /www/luci-static/commotion/calendar.png /www/luci-static/commotion/internet.png /usr/lib/lua/luci/view/commotion/apps_display.htm /usr/lib/lua/luci/view/themes/commotion/apps_categories.htm /www/luci-static/commotion/check.png /www/luci-static/commotion/commotion-faded.png /www/luci-static/commotion/downloads.png /usr/lib/lua/luci/view/commotion/apps_form.htm /www/luci-static/commotion/apps.css /www/luci-static/commotion/bann.png /www/luci-static/commotion/admin_apps.js /www/luci-static/commotion/photo.png /www/luci-static/commotion/patternizer.min.js /usr/lib/lua/luci/view/commotion/apps_admin_display.htm

areynold commented 10 years ago

Asset source path was wrong in the Makefile. Assets are now available, but apps are not being listed.

Edit: The remaining issue seems to be caused by #44.

dismantl commented 10 years ago

Just to confirm, luci-commotion-apps are working fine, unless you are building with an upgraded version of OLSRd?

areynold commented 10 years ago

Right. The 1.1 branch should work, but master is built against olsrd 0.6.6, which does not work with commotion-apps properly.