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

Error caused by uk/time_types.h #29

Open
czq693497091 opened this issue Feb 13, 2023 · 4 comments
Open

Error caused by uk/time_types.h #29

czq693497091 opened this issue Feb 13, 2023 · 4 comments

Comments

@czq693497091
Copy link

In file included from /data1/deepl/workdir/libs/lib-newlib/include/sys/_types.h:37:0,
from /data1/deepl/workdir/apps/nginx/build/libnewlibc/origin/newlib-2.5.0.20170922/newlib/libc/include/machine/endian.h:5,
from /data1/deepl/workdir/apps/nginx/build/libnewlibc/origin/newlib-2.5.0.20170922/newlib/libc/include/sys/param.h:10,
from /data1/deepl/workdir/libs/lib-newlib/include/sys/param.h:4,
from /data1/deepl/workdir/unikraft/include/uk/essentials.h:48,
from /data1/deepl/workdir/unikraft/lib/ukdebug/include/uk/print.h:41,
from /data1/deepl/workdir/unikraft/plat/common/x86/trace.c:34:
/data1/deepl/workdir/libs/lib-newlib/include/uk/_types.h:44:10: fatal error: uk/time_types.h: No such file or directory
#include <uk/time_types.h>
^~~~~~~~~~~~~~~~~

failed to compile the nginx and other libs are ok.

@istr
Copy link

istr commented Feb 26, 2023

See unikraft/unikraft@2d4dea9

This might need to be removed here as well.

@istr
Copy link

istr commented Feb 26, 2023

@czq693497091 how did you try to build that?

A build using kraft configure -p kvm -m x86_64, kraft prepare, kraft build works for me with a fresh, clean checkout of all dependencies.

@wattake
Copy link

wattake commented Feb 28, 2023

I also got this error when using the make and qemu-system-x86_64 tools instead of the kraft tool.

@istr
Copy link

istr commented Feb 28, 2023

@wattake yes. It seems that the make based build is not up to speed with kraft. If I get that right, support for libnew will be dropped. See unikraft/lib-redis#7 (review)

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

No branches or pull requests

3 participants