Skip to content

Commit

Permalink
Fix nasa#1946, Correct typos in code comments
Browse files Browse the repository at this point in the history
  • Loading branch information
thnkslprpt authored and astrogeco committed Sep 11, 2021
1 parent 8d4511b commit fcca73a
Show file tree
Hide file tree
Showing 88 changed files with 275 additions and 275 deletions.
2 changes: 1 addition & 1 deletion cmake/Makefile.sample
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ $(FILETGTS):
$(MAKE) -C $(O)/$(ARCH) $(@)
endif

# The "prep" step requires extra options that are specified via enviroment variables.
# The "prep" step requires extra options that are specified via environment variables.
# Certain special ones should be passed via cache (-D) options to CMake.
# These are only needed for the "prep" target but they are computed globally anyway.
PREP_OPTS :=
Expand Down
6 changes: 3 additions & 3 deletions cmake/arch_build.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
##################################################################

# define a custom property to track dependencies on CFE module targets.
# users should not typically maniplate this directly
# users should not typically manipulate this directly
define_property(TARGET PROPERTY CFE_MODULE_DEPENDENCIES
BRIEF_DOCS
"A set of CFE module dependencies"
Expand Down Expand Up @@ -206,7 +206,7 @@ function(add_cfe_tables APP_NAME TBL_SRC_FILES)
message("NOTE: Selected ${TBL_SRC} as source for ${APP_NAME}.${TBLWE} on ${TGT}")

# NOTE: On newer CMake versions this should become an OBJECT library which makes this simpler.
# On older versions one may not referece the TARGET_OBJECTS property from the custom command.
# On older versions one may not reference the TARGET_OBJECTS property from the custom command.
# As a workaround this is built into a static library, and then the desired object is extracted
# before passing to elf2cfetbl. It is roundabout but it works.
add_library(${TABLE_LIBNAME} STATIC ${TBL_SRC})
Expand Down Expand Up @@ -293,7 +293,7 @@ endfunction(add_cfe_coverage_dependency)
#
function(add_cfe_coverage_test MODULE_NAME UNIT_NAME TESTCASE_SRC UT_SRCS)

# A consistent name convention for all targets generated by this funtion
# A consistent name convention for all targets generated by this function
set(TEST_NAME "coverage-${MODULE_NAME}-${UNIT_NAME}")
set(OBJECT_TARGET "${TEST_NAME}-object")
set(RUNNER_TARGET "${TEST_NAME}-testrunner")
Expand Down
4 changes: 2 additions & 2 deletions cmake/global_functions.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ endfunction(generate_config_includefile)
# MISSION_APPS: list of all applications in this build
# MISSION_PSPMODULES: list of all psp modules in this build
#
# Additionally for each architechture in TGTSYS_LIST:
# Additionally for each architecture in TGTSYS_LIST:
# TGTSYS_<arch>: list of CPU names that utilize the same architecture
# TGTSYS_<arch>_APPS: list of apps for the architecture
# TGTSYS_<arch>_STATICAPPS: list of static apps for the architecture
Expand Down Expand Up @@ -211,7 +211,7 @@ function(read_targetconfig)

set(BUILD_CONFIG ${TOOLCHAIN_NAME} ${${CPUNAME}_PLATFORM})

# convert to a the string which is safe for a variable name
# convert to a string which is safe for a variable name
string(REGEX REPLACE "[^A-Za-z0-9]" "_" SYSVAR "${BUILD_CONFIG}")

# save the unmodified name for future reference
Expand Down
6 changes: 3 additions & 3 deletions cmake/mission_build.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ function(generate_build_version_templates)
VERBATIM
)

# Content for build info - these vars can be evaulated right now, no need to defer
# Content for build info - these vars can be evaluated right now, no need to defer
set(GENERATED_FILE_HEADER "/* Automatically generated from CMake build system */")
string(CONCAT GENERATED_FILE_CONTENT
"const char CFE_MISSION_NAME[] = \"${MISSION_NAME}\";\n"
Expand Down Expand Up @@ -414,7 +414,7 @@ function(process_arch TARGETSYSTEM)
set(BUILD_CONFIG ${BUILD_CONFIG_${TARGETSYSTEM}})
list(GET BUILD_CONFIG 0 ARCH_TOOLCHAIN_NAME)
list(REMOVE_AT BUILD_CONFIG 0)
# convert to a the string which is safe for a directory name
# convert to a string which is safe for a directory name
string(REGEX REPLACE "[^A-Za-z0-9]" "_" ARCH_CONFIG_NAME "${BUILD_CONFIG}")
set(ARCH_BINARY_DIR "${CMAKE_BINARY_DIR}/${ARCH_TOOLCHAIN_NAME}/${ARCH_CONFIG_NAME}")
file(MAKE_DIRECTORY "${ARCH_BINARY_DIR}" "${ARCH_BINARY_DIR}/inc")
Expand All @@ -424,7 +424,7 @@ function(process_arch TARGETSYSTEM)
# Note - A warning is issued if you pass CMAKE_TOOLCHAIN_FILE to an already-configured build area
# so an extra check is added to see if this is an initial run or a re-run by checking for a CMakeCache file.
if (NOT ARCH_TOOLCHAIN_NAME STREQUAL "native" AND NOT EXISTS "${ARCH_BINARY_DIR}/CMakeCache.txt")
# Find the toolchain file - allow a file in the mission defs dir to supercede one in the compile dir
# Find the toolchain file - allow a file in the mission defs dir to supersede one in the compile dir
if (EXISTS "${MISSION_DEFS}/toolchain-${ARCH_TOOLCHAIN_NAME}.cmake")
set(TOOLCHAIN_FILE "${MISSION_DEFS}/toolchain-${ARCH_TOOLCHAIN_NAME}.cmake")
elseif(EXISTS "${CFE_SOURCE_DIR}/cmake/toolchain-${ARCH_TOOLCHAIN_NAME}.cmake")
Expand Down
4 changes: 2 additions & 2 deletions cmake/mission_defaults.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
# a set of include directories and compile options for all modules,
# including the ubiquitous "cfe.h" header file and all it depends on).
# NOTE: these interfaces are really what defines "CFE core" - changing
# this list is not recommend, as these interface names are important.
# this list is not recommended, as these interface names are important.
set(MISSION_CORE_INTERFACES
core_api # this is the "public" core API that apps use, includes cfe.h and all core headers
core_private # this is the "private" interface that core apps use, but not used by apps
Expand All @@ -21,7 +21,7 @@ set(MISSION_CORE_INTERFACES
# The "MISSION_CORE_MODULES" will be built and statically linked as part
# of the CFE core executable on every target. These can be used to amend
# or override parts of the CFE core on a mission-specific basis.
# NOTE: Everthing in this list becomes part of the "core_api" interface above.
# NOTE: Everything in this list becomes part of the "core_api" interface above.
# Missions may add/remove/replace components in this list as needed.
set(MISSION_CORE_MODULES
"es"
Expand Down
2 changes: 1 addition & 1 deletion cmake/sample_defs/cpu1_cfe_es_startup.scr
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ CFE_APP, sch_lab, SCH_Lab_AppMain, SCH_LAB_APP, 80, 16384, 0x0, 0;
! 1. Object Type -- CFE_APP for an Application, or CFE_LIB for a library.
! 2. Path/Filename -- This is a cFE Virtual filename, not a vxWorks device/pathname
! 3. Entry Point -- This is the "main" function for Apps.
! 4. CFE Name -- The cFE name for the the APP or Library
! 4. CFE Name -- The cFE name for the APP or Library
! 5. Priority -- This is the Priority of the App, not used for Library
! 6. Stack Size -- This is the Stack size for the App, not used for the Library
! 7. Load Address -- This is the Optional Load Address for the App or Library. Currently not implemented
Expand Down
Loading

0 comments on commit fcca73a

Please sign in to comment.