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

Fix build with gcc 13.1.1 #184

Merged
merged 1 commit into from
Jun 11, 2023
Merged

Fix build with gcc 13.1.1 #184

merged 1 commit into from
Jun 11, 2023

Conversation

FabioLolix
Copy link
Contributor

Hello, with the update of GCC to v13.1.1 <cstdint> need to be added otherwise the build fail

i.e.

[ 81%] Linking C static library libappimage_shared.a
[ 81%] Built target libappimage_shared
In file included from /home/fabio/Dev/Github/PKGBUILD-AUR_fix/n/libappimage/src/libappimage-1.0.4-5/src/libappimage/utils/hashlib.cpp:9:
/home/fabio/Dev/Github/PKGBUILD-AUR_fix/n/libappimage/src/libappimage-1.0.4-5/src/libappimage/utils/hashlib.h:19:25: error: 'uint8_t' was not declared in this scope
   19 |             std::vector<uint8_t> md5(std::istream& data);
      |                         ^~~~~~~
/home/fabio/Dev/Github/PKGBUILD-AUR_fix/n/libappimage/src/libappimage-1.0.4-5/src/libappimage/utils/hashlib.h:1:1: note: 'uint8_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
  +++ |+#include <cstdint>
    1 | #pragma once
/home/fabio/Dev/Github/PKGBUILD-AUR_fix/n/libappimage/src/libappimage-1.0.4-5/src/libappimage/utils/hashlib.h:19:32: error: template argument 1 is invalid
   19 |             std::vector<uint8_t> md5(std::istream& data);
      |                                ^
/home/fabio/Dev/Github/PKGBUILD-AUR_fix/n/libappimage/src/libappimage-1.0.4-5/src/libappimage/utils/hashlib.h:19:32: error: template argument 2 is invalid
/home/fabio/Dev/Github/PKGBUILD-AUR_fix/n/libappimage/src/libappimage-1.0.4-5/src/libappimage/utils/hashlib.h:26:25: error: 'uint8_t' was not declared in this scope
   26 |             std::vector<uint8_t> md5(const std::string& data);
      |                         ^~~~~~~

@TheAssassin
Copy link
Member

Is this a fix for #574?

@FabioLolix
Copy link
Contributor Author

Is this a fix for #574?

Not related to TheAssassin/AppImageLauncher#574, this is related to nx-software-center which fails to build

@TheAssassin TheAssassin merged commit 1e0515b into AppImageCommunity:master Jun 11, 2023
7 checks passed
@TheAssassin
Copy link
Member

Thanks!

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