Skip to content

Commit

Permalink
mstflint: fix compilation with gcc 13
Browse files Browse the repository at this point in the history
GCC 13 complained really hard about missing definitions for uintXX_t.
A one liner in mlxdpa elfio fixed this for me.

Tracked upstream: Mellanox/mstflint#916

Signed-off-by: Markus Theil <theil.markus@gmail.com>
  • Loading branch information
thillux committed Feb 6, 2024
1 parent 621798f commit 03684a3
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 0 deletions.
5 changes: 5 additions & 0 deletions pkgs/tools/misc/mstflint/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,11 @@ stdenv.mkDerivation rec {
hash = "sha256-P8XACcz6d8UTOhFFeTijfFOthBqnUghGlDj9K145sZ8=";
};

# needed to introduce this with GCC 13. Remove, when https://github.com/Mellanox/mstflint/pull/916 is upstream.
patches = [
./elf.patch
];

nativeBuildInputs = [
autoconf
automake
Expand Down
13 changes: 13 additions & 0 deletions pkgs/tools/misc/mstflint/elf.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
diff --git a/mlxdpa/elfio/elf_types.hpp b/mlxdpa/elfio/elf_types.hpp
index b99c35f3..b0823b55 100755
--- a/mlxdpa/elfio/elf_types.hpp
+++ b/mlxdpa/elfio/elf_types.hpp
@@ -23,6 +23,8 @@ THE SOFTWARE.
#ifndef ELFTYPES_H
#define ELFTYPES_H

+#include <cstdint>
+
#ifdef __cplusplus
namespace ELFIO {
#endif

0 comments on commit 03684a3

Please sign in to comment.