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

[stable-v2.0] fix build on gcc 13.x and some typo #69

Merged
merged 8 commits into from
Oct 30, 2023

Conversation

Fantu
Copy link
Contributor

@Fantu Fantu commented Oct 29, 2023

today while I was preparing a blksnap usage test on debian unstable the compilation was failed due to the error already reported in #65 so I did a quick search and made this fix which should solve the build issue with gcc 13 or higher

the PR include also commits of #66 because I keep for fast check it with circleci build on ubuntu 23.10

also fixed a typo on a test script extension and some typo spotted with codespell

dependabot bot and others added 6 commits October 21, 2023 14:42
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to
4.
- [Release notes](https://github.com/actions/checkout/releases)
-
[Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
restored other arch container to bullseye as workaround for build error:
/home/runner/work/blksnap/blksnap/lib/blksnap/TrackerCtl.cpp: In
member function 'void blksnap::CTrackerCtl::ReadCbtMap(unsigned int,
unsigned int, uint8_t*)':
/home/runner/work/blksnap/blksnap/lib/blksnap/TrackerCtl.cpp:93:5:
sorry, unimplemented: non-trivial designated initializers not supported
       };
       ^

probably related to this:
https://stackoverflow.com/questions/31215971/non-trivial-designated-initializers-not-supported
but I not checked more for now
@Fantu Fantu changed the title fix build on gcc 13.x: error uint8_t not declared [stable-v2.0] fix build on gcc 13.x: error uint8_t not declared Oct 29, 2023
@Fantu Fantu changed the title [stable-v2.0] fix build on gcc 13.x: error uint8_t not declared [stable-v2.0] fix build on gcc 13.x and some typo Oct 29, 2023
@SergeiShtepa SergeiShtepa merged commit a34b36e into veeam:stable-v2.0 Oct 30, 2023
5 checks passed
@Fantu Fantu deleted the v2-fix-build branch November 1, 2023 09:37
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 this pull request may close these issues.

2 participants