openwrt / luci

LuCI - OpenWrt Configuration Interface
Apache License 2.0
6.36k stars 2.53k forks source link

NGINX File dependency error #1948

Closed fuqiang03 closed 6 years ago

fuqiang03 commented 6 years ago

qq 20180714132914 qq 20180714132933

build000 commented 6 years ago

@hnyman @Ansuel

Ansuel commented 6 years ago

are you sure you have updated the package feeds?

(Check if the makefile of nginx is equal to the one upstream)

build000 commented 6 years ago

Situation to openwrt-18.06...no master. Yes - updated feeds. example new:

slawek@machina:~/Pulpit/Projekty/openwrt-18.06-witi256m$ git pull
Already up to date.
slawek@machina:~/Pulpit/Projekty/openwrt-18.06-witi256m$ make package/symlinks V=s
./scripts/feeds update -a
Updating feed 'packages' from 'https://git.openwrt.org/feed/packages.git;openwrt-18.06' ...
Already up to date.
Create index file './feeds/packages.index' 
Updating feed 'luci' from 'https://git.openwrt.org/project/luci.git;openwrt-18.06' ...
remote: Counting objects: 21, done.
remote: Compressing objects: 100% (18/18), done.
remote: Total 21 (delta 13), reused 0 (delta 0)
Unpacking objects: 100% (21/21), done.
From https://git.openwrt.org/project/luci
   4b267cecf..9724d7661  openwrt-18.06 -> origin/openwrt-18.06
Updating 4b267cecf..9724d7661
Fast-forward
 applications/luci-app-adblock/po/ru/adblock.po | 100 ++++++++++++++++++++++++++++++++++++++-------------------------------------
 applications/luci-app-upnp/po/ru/upnp.po       |  14 +++++------
 modules/luci-base/po/ru/base.po                |   4 +--
 3 files changed, 60 insertions(+), 58 deletions(-)
Create index file './feeds/luci.index' 
Updating feed 'routing' from 'https://git.openwrt.org/feed/routing.git;openwrt-18.06' ...
Already up to date.
Create index file './feeds/routing.index' 
Updating feed 'telephony' from 'https://git.openwrt.org/feed/telephony.git;openwrt-18.06' ...
Already up to date.
Create index file './feeds/telephony.index' 
./scripts/feeds install -a
Collecting package info: done
WARNING: Makefile 'package/feeds/luci/luci-nginx/Makefile' has a dependency on 'nginx-mod-luci', which does not exist
WARNING: Makefile 'package/feeds/luci/luci-ssl-nginx/Makefile' has a dependency on 'nginx-ssl', which does not exist
WARNING: Makefile 'package/feeds/luci/luci-ssl-nginx/Makefile' has a dependency on 'nginx-mod-luci-ssl', which does not exist
Installing all packages from feed packages.
Installing all packages from feed luci.
Installing all packages from feed routing.
Installing all packages from feed telephony.
slawek@machina:~/Pulpit/Projekty/openwrt-18.06-witi256m$ 
Ansuel commented 6 years ago

openwrt-18.06 doesn't have the new changes to nginx... think that's the problem... instead luci have the new package....

(so i think luci package should be dropped or nginx and uwsgi changes should be pushed upstream)

build000 commented 6 years ago

So i think to revert this commit https://github.com/openwrt/luci/commit/885f993593d13cd7828e5f4b4b27a4b46d7918ab if actually not a "nginx changes should be pushed upstream"...

Ansuel commented 6 years ago

actually no... luci-nginx should not be there if uwsgi and nginx changes are not present in 18.06...

i really don't know if is possible to push the new nginx version to 18.06

think is better to wait for hnyman about this...

build000 commented 6 years ago

...exactly... well ... there is no change for 18.06... So i think to revert this commit 885f993 from 18.06 xD

jow- commented 6 years ago

https://github.com/openwrt/luci/commit/14c73faf313a85f9a70b21b1afd84a47e28a93c0

rossiniscarface commented 5 years ago

Looks like in the 18.06-snapshots there are still luci-nginx and luck-nginx-ssl packages built. https://downloads.openwrt.org/releases/18.06-SNAPSHOT/packages/arm_cortex-a5_neon-vfpv4/luci/luci-nginx_git-18.201.34715-c813598-1_all.ipk

hnyman commented 5 years ago

Looks like in the 18.06-snapshots there are still luci-nginx and luck-nginx-ssl packages built.

They are not built any more, but the old binaries have not been deleted. @jow-