Skip to content

Commit

Permalink
remove remaining prefix tree mentions
Browse files Browse the repository at this point in the history
  • Loading branch information
ivarprudnikov committed Sep 5, 2024
1 parent fd0a463 commit 5dbfa2d
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 19 deletions.
1 change: 0 additions & 1 deletion .github/workflows/build-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@ jobs:
# Additional CMake flags.
PLATFORM: ${{ matrix.platform.name }}
CMAKE_BUILD_TYPE: Debug
ENABLE_PREFIX_TREE: OFF
ENABLE_CLANG_TIDY: ON
BUILD_TESTS: ${{ matrix.platform.unit_tests_enabled }}
# These make CI print all the errors, rather than stopping early.
Expand Down
6 changes: 0 additions & 6 deletions .pipelines/pullrequest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -94,16 +94,13 @@ extends:
RunTests: false
Environment:
CMAKE_BUILD_TYPE: Debug
ENABLE_PREFIX_TREE: OFF
ENABLE_CLANG_TIDY: ON
InstallPackages: [ libcurl4-openssl-dev, clang-tidy-10 ]

- template: .pipelines/local_virtual_build.yml@self
parameters:
Name: test_virtual
DisplayName: Build and test (Virtual)
Environment:
ENABLE_PREFIX_TREE: OFF
FunctionalTestArguments: '--enable-faketime'
InstallPackages: [ libcurl4-openssl-dev, faketime ]

Expand All @@ -119,7 +116,6 @@ extends:
PLATFORM: virtual
CXXFLAGS: -ferror-limit=0
NINJA_FLAGS: -k 0
ENABLE_PREFIX_TREE: OFF
steps:
- checkout: SCITT
path: s/
Expand Down Expand Up @@ -172,7 +168,6 @@ extends:
--env PLATFORM="sgx" \
--env CMAKE_BUILD_TYPE="Release" \
--env CCF_UNSAFE="OFF" \
--env ENABLE_PREFIX_TREE="OFF" \
--env BUILD_TESTS="OFF" \
--env ENABLE_CLANG_TIDY="OFF" \
--env NINJA_FLAGS="-k 0" \
Expand All @@ -199,7 +194,6 @@ extends:
PLATFORM: sgx
CXXFLAGS: -ferror-limit=0
NINJA_FLAGS: -k 0
ENABLE_PREFIX_TREE: OFF
steps:
- checkout: SCITT
path: s/
Expand Down
5 changes: 0 additions & 5 deletions app/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,6 @@ set(CMAKE_EXPORT_COMPILE_COMMANDS ON)

find_package(ccf_${COMPILE_TARGET} 5.0.0 REQUIRED)


if (ENABLE_PREFIX_TREE)
add_compile_definitions(ENABLE_PREFIX_TREE)
endif ()

include(${CMAKE_CURRENT_SOURCE_DIR}/cmake/version.cmake)
include(${CMAKE_CURRENT_SOURCE_DIR}/cmake/target_link_system_libraries.cmake)

Expand Down
1 change: 0 additions & 1 deletion app/src/constants.h
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ namespace scitt
const std::string QueryParameterError = "QueryParameterError";
const std::string PayloadTooLarge = "PayloadTooLarge";
const std::string UnknownFeed = "UnknownFeed";
const std::string NoPrefixTree = "NoPrefixTree";
const std::string NotFound = "NotFound";
const std::string OperationExpired = "OperationExpired";
const std::string PolicyError = "PolicyError";
Expand Down
4 changes: 0 additions & 4 deletions app/src/main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -789,10 +789,6 @@ namespace scitt
register_service_endpoints(context, *this);
register_operations_endpoints(
context, *this, authn_policy, post_entry_continuation);

#ifdef ENABLE_PREFIX_TREE
PrefixTreeFrontend::init_handlers(context, *this);
#endif
}
};
} // namespace scitt
Expand Down
2 changes: 0 additions & 2 deletions build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ set -ex
CMAKE_BUILD_TYPE=${CMAKE_BUILD_TYPE:-Release}
PLATFORM=${PLATFORM:-sgx}
CCF_UNSAFE=${CCF_UNSAFE:-OFF}
ENABLE_PREFIX_TREE=${ENABLE_PREFIX_TREE:-OFF}
BUILD_TESTS=${BUILD_TESTS:-ON}
ENABLE_CLANG_TIDY=${ENABLE_CLANG_TIDY:-OFF}
NINJA_FLAGS=${NINJA_FLAGS:-}
Expand Down Expand Up @@ -61,7 +60,6 @@ CC="$CC" CXX="$CXX" \
-DATTESTED_FETCH_MRENCLAVE_HEX="${ATTESTED_FETCH_MRENCLAVE_HEX}" \
-DCOMPILE_TARGET="${PLATFORM}" \
-DCCF_UNSAFE="${CCF_UNSAFE}" \
-DENABLE_PREFIX_TREE="${ENABLE_PREFIX_TREE}" \
-DBUILD_TESTS="${BUILD_TESTS}" \
-DLVI_MITIGATIONS=OFF \
-DCMAKE_INSTALL_PREFIX=$install_dir \
Expand Down

0 comments on commit 5dbfa2d

Please sign in to comment.