Skip to content

Commit 2d4f750

Browse files
MRaybuckdcrowell77
authored andcommitted
Fix DCE compile issues
The dce_rc command was running an ls command without de-aliasing it cause some users to run into issues with the make not finding their extra files. TestSuite.H was missing an include that is picked up by chance in the hostboot compile. Change-Id: I4fe1d991f8991c7fba6e66e11f94070151adfb2e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/c/hostboot/+/172246 Tested-by: Jenkins Server <[email protected]> Reviewed-by: Daniel M Crowell <[email protected]>
1 parent c4227d1 commit 2d4f750

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

src/build/tools/dce/dce-extra-files/dce_rc

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
export DCE_EXTRA_FILES_LOCATION=$PROJECT_ROOT/src/build/tools/dce/dce-extra-files
88
# Will automatically gather all files in extra file location as a space-separated list. Compiler will filter out files
99
# that do not have .C or .H suffixes so that non-code files aren't made dependencies.
10-
export DCE_EXTRA_FILES=$(ls $DCE_EXTRA_FILES_LOCATION | awk -v var="$DCE_EXTRA_FILES_LOCATION/" '{printf "%s ", var $0 }')
10+
export DCE_EXTRA_FILES=$(\ls $DCE_EXTRA_FILES_LOCATION | awk -v var="$DCE_EXTRA_FILES_LOCATION/" '{printf "%s ", var $0 }')
1111
# If you need extra CXXFLAGS, space separated.
1212
export CXXFLAGS_DCE="-D__HOSTBOOT_MODULE=DCE"
1313
# If you need to add extra INCDIRs, space seperated.

src/include/usr/cxxtest/TestSuite.H

+1
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
#define __CXXTEST_TESTSUITE_H
3232

3333
#include <arch/magic.H>
34+
#include <limits.h>
3435

3536
// Set by code that wants to manually execute a testcase outside of the
3637
// normal CxxTest framework

0 commit comments

Comments
 (0)