Skip to content

Commit 1d903ee

Browse files
committed
src: fix compilation errors after bump to 24.04
1 parent a8067ab commit 1d903ee

File tree

4 files changed

+5
-2
lines changed

4 files changed

+5
-2
lines changed

Diff for: docker/faabric-base.dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ RUN apt update \
1111
wget \
1212
# LLVM APT Repo config
1313
&& wget -qO- https://apt.llvm.org/llvm-snapshot.gpg.key | tee /etc/apt/trusted.gpg.d/apt.llvm.org.asc \
14-
&& add-apt-repository -y "deb http://apt.llvm.org/jammy/ llvm-toolchain-jammy-${LLVM_VERSION_MAJOR} main"
14+
&& add-apt-repository -y "deb http://apt.llvm.org/noble/ llvm-toolchain-noble-${LLVM_VERSION_MAJOR} main"
1515

1616
# Install APT packages
1717
RUN apt update && apt install -y \

Diff for: docker/faabric.dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ ARG FAABRIC_VERSION
55
SHELL ["/bin/bash", "-c"]
66

77
# Flag to say we're in a container
8-
ENV FAABRIC_DOCKER="on"
8+
ENV FAABRIC_DOCKER "on"
99

1010
# Put the code in place
1111
WORKDIR /code

Diff for: include/faabric/util/bytes.h

+1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
#pragma once
22

33
#include <algorithm>
4+
#include <cstdint>
45
#include <iomanip>
56
#include <list>
67
#include <span>

Diff for: src/planner/Planner.cpp

+2
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,7 @@ static void releaseHostMpiPort(std::shared_ptr<Host> host, int mpiPort)
120120
throw std::runtime_error("Requested to free unavailable MPI port!");
121121
}
122122

123+
#ifndef NDEBUG
123124
static void printHostState(std::map<std::string, std::shared_ptr<Host>> hostMap,
124125
const std::string& logLevel = "debug")
125126
{
@@ -147,6 +148,7 @@ static void printHostState(std::map<std::string, std::shared_ptr<Host>> hostMap,
147148
SPDLOG_ERROR("Unrecognised log level: {}", logLevel);
148149
}
149150
}
151+
#endif
150152

151153
// ----------------------
152154
// Planner

0 commit comments

Comments
 (0)