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

[BUG] Build error with fs_pseudofile.c on master #13756

Closed
1 task done
adriendesp opened this issue Sep 30, 2024 · 1 comment · Fixed by #13759
Closed
1 task done

[BUG] Build error with fs_pseudofile.c on master #13756

adriendesp opened this issue Sep 30, 2024 · 1 comment · Fixed by #13759
Labels
Arch: arm Issues related to ARM (32-bit) architecture Area: File System File System issues

Comments

@adriendesp
Copy link
Contributor

adriendesp commented Sep 30, 2024

Description / Steps to reproduce the issue

Where : In the file fs/vfs/fs_pseudofile.c, in method pseudofile_munmap, since #13443
When : When CONFIG_PSEUDOFS_FILE=y is enabled
What : The following build error is returned :

libm/lib_nanl.c vfs/fs_pseudofile.c: In function 'pseudofile_munmap': vfs/fs_pseudofile.c:367:7: error: 'ret' undeclared (first use in this function) 367 | ret = OK; | ^~~

Also, it seems that this return variable isn't really relevant in the way it is implemented.

On which OS does this issue occur?

[OS: Linux]

What is the version of your OS?

Ubuntu 22.04.3

NuttX Version

master

Issue Architecture

[Arch: arm]

Issue Area

[Area: File System]

Verification

  • I have verified before submitting the report.
@github-actions github-actions bot added Arch: arm Issues related to ARM (32-bit) architecture Area: File System File System issues labels Sep 30, 2024
@adriendesp adriendesp changed the title [BUG] Master-broken : Build error with fs_pseudofile.c [BUG] Build error with fs_pseudofile.c on master Sep 30, 2024
@Donny9
Copy link
Contributor

Donny9 commented Sep 30, 2024

@adriendesp hello, you can try again with PR #13759.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arch: arm Issues related to ARM (32-bit) architecture Area: File System File System issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants