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

-Wint-conversion when compiling runtime.c #19

Open
danpla opened this issue Oct 19, 2023 · 3 comments · May be fixed by #20
Open

-Wint-conversion when compiling runtime.c #19

danpla opened this issue Oct 19, 2023 · 3 comments · May be fixed by #20

Comments

@danpla
Copy link

danpla commented Oct 19, 2023

Building the code at c9553b0 gives:

runtime.c: In function 'fusefs_main':
runtime.c:905:27: warning: initialization of 'long unsigned int' from 'char *' makes integer from pointer without a cast [-Wint-conversion]
  905 |             {"fsname=%s", "squashfuse", 0},
      |                           ^~~~~~~~~~~~
runtime.c:905:27: note: (near initialization for 'fuse_opts[2].offset')
runtime.c:906:28: warning: initialization of 'long unsigned int' from 'char *' makes integer from pointer without a cast [-Wint-conversion]
  906 |             {"subtype=%s", "squashfuse", 0},
@probonopd
Copy link
Member

How are you building it? Seems to build fine in this project's CI pipeline.

@danpla
Copy link
Author

danpla commented Oct 28, 2023

I used the standard

export ARCHITECTURE=x86_64
./chroot_build.sh

But these warnings are also visible in CI logs. Just make sure you select "View raw logs" from the gear menu before searching with Ctrl+F (otherwise, GitHub will dynamically load only the visible parts of the log, breaking the search).

probonopd added a commit that referenced this issue Oct 28, 2023
@probonopd probonopd linked a pull request Oct 28, 2023 that will close this issue
@probonopd
Copy link
Member

I see. Thanks for bringing this to our attention. Please do review the proposed fix.

@danpla danpla mentioned this issue Oct 29, 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

Successfully merging a pull request may close this issue.

2 participants