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

Remove out-of-tree patches #99

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion applications/freertos_iot_libraries_tests/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,6 @@ target_link_libraries(freertos-libraries-integration-tests-config
coremqtt
helpers-logging
freertos-libraries-integration-tests-mqtt
freertos-libraries-integration-tests-ota-pal
freertos-libraries-integration-tests-qualification
freertos-libraries-integration-tests-transport
unity
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@

#include "transport_interface_test.h"
#include "qualification_test.h"
#include "ota_pal_test.h"
#include "mqtt_test.h"

#include "demo_config.h"
Expand Down Expand Up @@ -128,11 +127,6 @@ void SetupMqttTestParam( MqttTestParam_t * pTestParam )
}
}

void SetupOtaPalTestParam( OtaPalTestParam_t * pTestParam )
{
pTestParam->pageSize = 4096;
}

typedef struct Task_t
{
TaskHandle_t handle;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,11 +55,6 @@
*/
#define TRANSPORT_INTERFACE_TEST_ENABLED ( 1 )

/**
* @brief Configuration to enable the OTA PAL test.
*/
#define OTA_PAL_TEST_ENABLED ( 1 )

/**
* @brief Configuration to enable the OTA End-to-end test.
*/
Expand Down
29 changes: 0 additions & 29 deletions applications/freertos_iot_libraries_tests/test_param_config.h
Original file line number Diff line number Diff line change
Expand Up @@ -186,35 +186,6 @@
* #define TEST_START_DELAY_MS 5000
*/

#define OTA_RSA_SHA1 1
#define OTA_RSA_SHA256 2
#define OTA_ECDSA_SHA256 3

/**
* @brief Certificate type for OTA PAL test.
* Valid options are: OTA_RSA_SHA1, OTA_RSA_SHA256, OTA_ECDSA_SHA256.
*/
#define OTA_PAL_TEST_CERT_TYPE OTA_RSA_SHA256

/**
* @brief Path to cert for OTA test PAL. Used to verify signature.
* If applicable, the device must be pre-provisioned with this certificate. Please see
* test/common/ota/test_files for the set of certificates.
*/
#define OTA_PAL_CERTIFICATE_FILE "ecdsa-sha256-signer.crt.pem"

/**
* @brief Some devices have a hard-coded name for the firmware image to boot.
*/
#define OTA_PAL_FIRMWARE_FILE "non_secure image"

/**
* @brief Some boards OTA PAL layers will use the file names passed into it for the
* image and the certificates because their non-volatile memory is abstracted by a
* file system. Set this to 1 if that is the case for your device.
*/
#define OTA_PAL_USE_FILE_SYSTEM 0

/**
* @brief The PKCS #11 supports RSA key function.
*
Expand Down
15 changes: 4 additions & 11 deletions applications/helpers/provisioning/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -23,17 +23,10 @@ else()
fri-bsp
)

if(FREERTOS_LIBRARIES_INTEGRATION_TESTS EQUAL 1)
set(
CODE_SIGNING_PUBLIC_KEY_PEM_PATH
${freertos_libraries_integration_tests_SOURCE_DIR}/src/ota/test_files/tfm-rsa-sha256-signer.crt.pem
)
else()
set(
CODE_SIGNING_PUBLIC_KEY_PEM_PATH
${AWS_OTA_SIGNATURE_PUBLIC_KEY_PATH}
)
endif()
set(
CODE_SIGNING_PUBLIC_KEY_PEM_PATH
${AWS_OTA_SIGNATURE_PUBLIC_KEY_PATH}
)

add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/aws_clientcredential_keys.h
DEPENDS ${AWS_CLIENT_PRIVATE_KEY_PEM_PATH}
Expand Down
6 changes: 0 additions & 6 deletions components/aws_iot/corepkcs11/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,11 +9,5 @@ if(CMAKE_CROSSCOMPILING)
"Path to corePKCS11 source code"
)

include(ApplyPatches)

set(PATCH_FILES_DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/integration/patches")
set(PATCH_FILES "${PATCH_FILES_DIRECTORY}/0001-Do-not-define-__PASTE-macro-for-Arm-compiler.patch")
iot_reference_arm_corstone3xx_apply_patches("${corepkcs11_SOURCE_DIR}" "${PATCH_FILES}")

add_subdirectory(integration)
endif()

This file was deleted.

2 changes: 1 addition & 1 deletion components/aws_iot/corepkcs11/library
Submodule library updated 147 files
6 changes: 0 additions & 6 deletions components/security/mbedtls/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,6 @@ else ()
set(ENABLE_PROGRAMS OFF CACHE BOOL "" FORCE)
set(ENABLE_TESTING OFF CACHE BOOL "" FORCE)

include(ApplyPatches)

set(PATCH_FILES_DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/integration/patches")
set(PATCH_FILES "${PATCH_FILES_DIRECTORY}/0001-md-Fix-guards-for-functions-and-headers.patch")
iot_reference_arm_corstone3xx_apply_patches("${mbedtls_SOURCE_DIR}" "${PATCH_FILES}")

add_subdirectory(library)

add_subdirectory(integration)
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,6 @@ include(ApplyPatches)
set(PATCH_FILES_DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/integration/patches")
set(PATCH_FILES
"${PATCH_FILES_DIRECTORY}/0001-Reduce-the-TRANSPORT_TEST_BUFFER_WRITABLE_LENGTH.patch"
"${PATCH_FILES_DIRECTORY}/0002-closefile-validsignature-test-Fix-test-bugs.patch"
"${PATCH_FILES_DIRECTORY}/0003-Introduce-user-defined-MQTT-keep-alive-value.patch"
)
iot_reference_arm_corstone3xx_apply_patches("${freertos_libraries_integration_tests_SOURCE_DIR}" "${PATCH_FILES}")

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ add_library(freertos-libraries-integration-tests-config INTERFACE)
include(${freertos_libraries_integration_tests_SOURCE_DIR}/qualification_test.cmake)
include(${freertos_libraries_integration_tests_SOURCE_DIR}/src/mqtt_test.cmake)
include(${freertos_libraries_integration_tests_SOURCE_DIR}/src/pkcs11_test.cmake)
include(${freertos_libraries_integration_tests_SOURCE_DIR}/src/ota_pal_test.cmake)
include(${freertos_libraries_integration_tests_SOURCE_DIR}/src/transport_interface_test.cmake)

add_library(freertos-libraries-integration-tests-mqtt
Expand All @@ -31,20 +30,6 @@ target_compile_definitions(freertos-libraries-integration-tests-mqtt
PRIVATE
MQTT_KEEP_ALIVE_INTERVAL_SECONDS=60
)
add_library(freertos-libraries-integration-tests-ota-pal
${OTA_PAL_TEST_SOURCES}
)
target_include_directories(freertos-libraries-integration-tests-ota-pal
PUBLIC
${OTA_PAL_TEST_INCLUDE_DIRS}
)
target_link_libraries(freertos-libraries-integration-tests-ota-pal
PRIVATE
freertos-libraries-integration-tests-config
freertos-ota-pal-psa
ota-for-aws-iot-embedded-sdk
unity
)

add_library(freertos-libraries-integration-tests-pkcs11
${PKCS11_TEST_SOURCES}
Expand Down Expand Up @@ -82,7 +67,6 @@ target_link_libraries(freertos-libraries-integration-tests-qualification
coremqtt
freertos-libraries-integration-tests-config
freertos-libraries-integration-tests-mqtt
freertos-libraries-integration-tests-ota-pal
freertos-libraries-integration-tests-pkcs11
)

Expand Down

This file was deleted.

Loading
Loading