Skip to content

Commit

Permalink
tests: prevent inclusion of time.h in baremetal compiles
Browse files Browse the repository at this point in the history
baremetal compiles should not include time.h, as MBEDTLS_HAVE_TIME is
undefined. To test this, provide an overriding include directory that
has a time.h which throws a meaningful error if included.

Signed-off-by: Daniel Axtens <[email protected]>
  • Loading branch information
daxtens committed Sep 2, 2020
1 parent 49cbda9 commit 3bd69b9
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 3 deletions.
18 changes: 18 additions & 0 deletions tests/include/baremetal-override/time.h
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
/*
* Copyright The Mbed TLS Contributors
* SPDX-License-Identifier: Apache-2.0
*
* Licensed under the Apache License, Version 2.0 (the "License"); you may
* not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
* WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

#error "time.h included in a configuration without MBEDTLS_HAVE_TIME"
4 changes: 1 addition & 3 deletions tests/scripts/all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -644,8 +644,6 @@ pre_check_tools () {
"$@" scripts/output_env.sh
}



################################################################
#### Basic checks
################################################################
Expand Down Expand Up @@ -1168,7 +1166,7 @@ component_build_crypto_full () {
component_build_crypto_baremetal () {
msg "build: make, crypto only, baremetal config"
scripts/config.py crypto_baremetal
make CFLAGS='-O1 -Werror'
make CFLAGS="-O1 -Werror -I$(pwd)/tests/include/baremetal-override/"
if_build_succeeded are_empty_libraries library/libmbedx509.* library/libmbedtls.*
}

Expand Down

0 comments on commit 3bd69b9

Please sign in to comment.