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

Failed to build v2022.06.20 #1800

Closed
ftiasch opened this issue Jun 22, 2022 · 2 comments
Closed

Failed to build v2022.06.20 #1800

ftiasch opened this issue Jun 22, 2022 · 2 comments

Comments

@ftiasch
Copy link

ftiasch commented Jun 22, 2022

Building against the release v2022.06.20 gives me the following error:

[ 40%] Building CXX object CMakeFiles/folly_base.dir/folly/experimental/symbolizer/ElfCache.cpp.o
In file included from /home/ftiasch/Documents/aur/folly/src/folly-2022.06.20.00/folly/experimental/io/IoUringBackend.cpp:25:
/home/ftiasch/Documents/aur/folly/src/folly-2022.06.20.00/folly/experimental/io/IoUringBackend.h: In member function ‘void folly::IoUringBackend::IoSqe::prepCancel(io_uring_sqe*, folly::IoUringBackend::IoSqe*)’:
/home/ftiasch/Documents/aur/folly/src/folly-2022.06.20.00/folly/experimental/io/IoUringBackend.h:637:35: error: invalid conversion from ‘uint64_t’ {aka ‘long unsigned int’} to ‘void*’ [-fpermissive]
  637 |       ::io_uring_prep_cancel(sqe, reinterpret_cast<uint64_t>(cancel_sqe), 0);
      |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                   |
      |                                   uint64_t {aka long unsigned int}
In file included from /home/ftiasch/Documents/aur/folly/src/folly-2022.06.20.00/folly/experimental/io/IoUringBackend.h:44:
/usr/include/liburing.h:442:73: note:   initializing argument 2 of ‘void io_uring_prep_cancel(io_uring_sqe*, void*, int)’
  442 | static inline void io_uring_prep_cancel(struct io_uring_sqe *sqe, void *user_data,
      |                                                                   ~~~~~~^~~~~~~~~

It seems that we shouldn't cast a pointer to uint64_t to fit in a void*.

Additionally, uname -r shows 5.18.5-arch1-1.

@Orvid
Copy link
Contributor

Orvid commented Jun 22, 2022

3317744 should be the fix for this, though it hasn't made it into a release yet.

@yfeldblum
Copy link
Contributor

Should have made it into a release by now.

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