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

compilation error using GCC 13.2 #9334

Closed
denisbertini opened this issue Sep 1, 2023 · 2 comments
Closed

compilation error using GCC 13.2 #9334

denisbertini opened this issue Sep 1, 2023 · 2 comments

Comments

@denisbertini
Copy link

trying to compile the latest version ( 1.14.1) gives a compilation error:

base/uct_md.c:559:1: error: conflicting types for 'uct_md_mem_advise' due to enum/integer mismatch; have 'ucs_status_t(struct uct_md *, void *, void *, size_t,  unsigned int)' {aka 'ucs_status_t(struct uct_md *, void *, void *, long unsigned int,  unsigned int)'} [-Werror=enum-int-mismatch]
  559 | uct_md_mem_advise(uct_md_h md, uct_mem_h memh, void *addr, size_t length,
      | ^~~~~~~~~~~~~~~~~
In file included from base/uct_component.h:11,
                 from base/uct_md.h:14,
                 from base/uct_md.c:13:
/var/tmp/ucx-1.14.1/src/uct/api/uct.h:2542:14: note: previous declaration of 'uct_md_mem_advise' with type 'ucs_status_t(struct uct_md *, void *, void *, size_t,  uct_mem_advice_t)' {aka 'ucs_status_t(struct uct_md *, void *, void *, long unsigned int,  uct_mem_advice_t)'}
 2542 | ucs_status_t uct_md_mem_advise(uct_md_h md, uct_mem_h memh, void *addr,
      |              ^~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
make[3]: *** [Makefile:877: base/libuct_la-uct_md.lo] Error 1
make[3]: *** Waiting for unfinished jobs....
make[3]: Leaving directory '/var/tmp/ucx-1.14.1/src/uct'
make[2]: *** [Makefile:1138: all-recursive] Error 1
make[2]: Leaving directory '/var/tmp/ucx-1.14.1/src/uct'
make[1]: *** [Makefile:753: all-recursive] Error 1
make[1]: Leaving directory '/var/tmp/ucx-1.14.1'
make: *** [Makefile:618: all] Error 2

one should fix the type mismatch.

@yosefe
Copy link
Contributor

yosefe commented Sep 3, 2023

@denisbertini can you pls check if https://github.com/openucx/ucx/releases/tag/v1.15.0-rc4 works for you?

@denisbertini
Copy link
Author

works now with this version thanks !

@shamisp shamisp closed this as completed Sep 10, 2023
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