Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes to support building in SDK #5

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Fixes to support building in SDK #5

wants to merge 5 commits into from

Conversation

mk-f
Copy link

@mk-f mk-f commented Mar 31, 2016

I recently tried to build juci inside the OpenWRT-SDK instead inside the whole Framework. Beside stumbling upon Issue mkschreder/juci#39 and #4 two Makefiles needed small fixes to compile trough.

@mkschreder
Copy link
Owner

Thanks.

I do however have a question mark around using unicode ncurses and whether that can have any side effects, but I'm adding your fix anyway because 1) libncurses is only a dependency for building luaposix and is not used by juci and 2) I'm considering removing luaposix completely some day in the future because it seems like almost an unnecessary dependency considering how little it is being used in the lua backend.

@mk-f
Copy link
Author

mk-f commented Apr 1, 2016

I tried to investigate this further with no luck, to be honest i don't know why inside the SDK luaposix requires the unicode variant, it certainly doesn't when building inside the complete framework (which also does not provide ncursesw in any way as far as i see).

Package luaposix33 is missing dependencies for the following libraries:
libncursesw.so.5
Makefile:57: recipe for target '/tmp/OpenWrt-SDK-at91-legacy_gcc-5.3.0_musl-1.1.14_eabi.Linux-x86_64/bin/at91/packages/juci/luaposix33_v33.3.1-1_at91.ipk' failed

However, the autoreconf-fix is required there for me to.

@mkschreder
Copy link
Owner

It could be that you are somehow building unicode version and configure
script of luaposix autodetects unicode lib as variant of libncurses and
links to it automatically. However, the package system detects that the lib
does not match the specified dependency in the package and complains about
it. For example this can happen when you autodetect lua in wrong way and
link to liblua5.X when your dependencies specify just liblua. Openwrt is
just trying to be smart and warns you that you may have missed to specify a
dependency in the package.
On 1 Apr 2016 21:40, "Fabian Müller-Knapp" [email protected] wrote:

I tried to investigate this further with no luck, to be honest i don't
know why inside the SDK luaposix requires the unicode variant, it certainly
doesn't when building inside the complete framework (which also does not
provide ncursesw in any way as far as i see).

Package luaposix33 is missing dependencies for the following libraries:
libncursesw.so.5
Makefile:57: recipe for target '/tmp/OpenWrt-SDK-at91-legacy_gcc-5.3.0_musl-1.1.14_eabi.Linux-x86_64/bin/at91/packages/juci/luaposix33_v33.3.1-1_at91.ipk' failed

However, the autoreconf-fix is required there for me to.


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#5 (comment)

@mk-f
Copy link
Author

mk-f commented Apr 4, 2016

In this case i think +libncurses +PACKAGE_libncursesw:libncursesw is a better solution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants