Age | Commit message (Collapse) | Author | Lines |
|
|
|
|
|
|
|
Fixes #159
|
|
Migrate off of soon-to-be-removed CI image `macos-12`
|
|
|
|
mpartel/dependabot/github_actions/codespell-project/actions-codespell-2.1
Actions(deps): Bump codespell-project/actions-codespell from 2.0 to 2.1
|
|
Bumps [codespell-project/actions-codespell](https://github.com/codespell-project/actions-codespell) from 2.0 to 2.1.
- [Release notes](https://github.com/codespell-project/actions-codespell/releases)
- [Commits](https://github.com/codespell-project/actions-codespell/compare/v2.0...v2.1)
---
updated-dependencies:
- dependency-name: codespell-project/actions-codespell
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
|
|
All runners should now have KVM.
|
|
|
|
|
|
Not sure what changed in the runners to make gcc-13 unavailable.
|
|
|
|
For #140
|
|
|
|
|
|
Apparently this became necessary at some point.
|
|
Cover macOS in CI + make CI use the right compiler + fix more compile warnings
|
|
Make CI reject spelling errors using codespell + fix five typos
|
|
|
|
|
|
codespell report was:
> # git ls-files | xargs codespell
> src/bindfs.1:136: mimicing ==> mimicking
> src/bindfs.1:237: excecute ==> execute
> src/bindfs.1:445: efficent ==> efficient
> src/bindfs.1:526: accessable ==> accessible
> tests/test_bindfs.rb:976: hax ==> hex # not agreeing on a fix here
|
|
|
|
|
|
|
|
Related commit:
https://github.com/libfuse/libfuse/commit/6c66dac9f229d0035f83ae63d9033e4f51d9c5c8
|
|
Symptom with Clang 17:
> tests/odirect_write.c:35:12: warning: variable 'total_size' set but not used [-Wunused-but-set-variable]
> 35 | size_t total_size = 0;
> | ^
|
|
Symptom with Clang 15:
> In file included from userinfo.c:20:
> ./userinfo.h:38:27: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
> void invalidate_user_cache(); /* safe to call from signal handler */
> ^
> void
> [many more]
|
|
Symptom with GCC 13:
> bindfs.c:1470:12: error: 'bindfs_flock' defined but not used [-Werror=unused-function]
> static int bindfs_flock(const char *path, struct fuse_file_info *fi, int op)
> ^~~~~~~~~~~~
> bindfs.c:1458:12: error: 'bindfs_lock' defined but not used [-Werror=unused-function]
> static int bindfs_lock(const char *path, struct fuse_file_info *fi, int cmd,
> ^~~~~~~~~~~
|
|
Symptom with Apple GCC:
> bindfs.c:1677:26: error: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'ssize_t' (aka 'long') [-Werror,-Wsign-compare]
> } while (len < res);
> ~~~ ^ ~~~
|
|
|
|
|
|
|
|
|
|
Fix compile warnings (including a serious one) + cover FUSE 2 in CI + add `-Wextra` to `configure.ac`
|
|
|
|
The fix is a near 1:1 copy of what add_chmod_rule_to_permchain
already does about the same problem.
Symptom was:
> src/permchain.c: In function ‘add_octal_rule_to_permchain’:
> src/permchain.c:151:71: error: unused parameter ‘end’ [-Werror=unused-parameter]
> 151 | static int add_octal_rule_to_permchain(const char *start, const char *end,
> |
|
|
|
|
|
|
|
|
Symptom was:
> tests/odirect_write.c: In function ‘main’:
> tests/odirect_write.c:58:17: error: comparison of integer expressions of different signedness: ‘ssize_t’ {aka ‘long int’} and ‘size_t’ {aka ‘long unsigned int’} [-Werror=sign-compare]
> 58 | if (res != buf_size) {
> | ^~
|
|
Started out with this warning:
> src/misc.c: In function ‘grow_memory_block’:
> src/misc.c:333:25: error: comparison of unsigned expression in ‘< 0’ is always false [-Werror=type-limits]
> 333 | if (new_cap < 0) { // Overflow
> | ^
|
|
The symptom:
> src/misc.c: In function ‘path_starts_with’:
> src/misc.c:169:45: error: operand of ‘?:’ changes signedness from ‘long int’ to ‘long unsigned int’ due to unsignedness of other operand [-Werror=sign-compare]
> 169 | size_t path_part_len = path_slash ? path_slash - path_part : path_len - (path_part - path);
> | ^~~~~~~~~~~~~~~~~~~~~~
|
|
|
|
For example:
> src/bindfs.c: In function ‘getattr_common’:
> src/bindfs.c:449:26: error: comparison of integer expressions of different signedness: ‘uid_t’ {aka ‘unsigned int’} and ‘int’ [-Werror=sign-compare]
> 449 | if (settings.new_uid != -1)
> | ^~
>
|
|
|
|
|
|
|
|
|
|
|