Skip to content
This repository was archived by the owner on Aug 11, 2021. It is now read-only.

Commit 232c1af

Browse files
committed
[tests,DeltaSettingsTest2] Fixes #333. Fix DeltaSettingsTest2 example on Linux.
1 parent c14cc5f commit 232c1af

File tree

4 files changed

+14
-3
lines changed

4 files changed

+14
-3
lines changed

Changelog.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
13-May-2017 Fixes #333. Fix DeltaSettingsTest2 example on Linux.
2+
13
13-May-2017 Fixes #332. Mingw assembler support now working. AssemblerTest1 example also now builds with Mingw.
24

35
11-May-2017 ======== Version 1.1.1 beta 2 Release ========

tests/DeltaSettingsTest2/bam/DeltaSettingsTest2.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
<Package name="Clang" version="Xcode8" default="true" />
1111
<Package name="Gcc" version="4.8" default="true" />
1212
<Package name="Gcc" version="5.4" />
13+
<Package name="MakeFileBuilder" />
1314
<Package name="NativeBuilder" />
1415
<Package name="VisualC" version="10.0" />
1516
<Package name="VisualC" version="11.0" />

tests/DeltaSettingsTest2/bam/Scripts/DeltaSettingsTest2.cs

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,9 @@ protected override void
5757
gccCompiler.AllWarnings = true;
5858
gccCompiler.ExtraWarnings = true;
5959
gccCompiler.Pedantic = true;
60+
61+
var compiler = settings as C.ICommonCompilerSettings;
62+
compiler.DisableWarnings.AddUnique("long-long"); // DeltaSettingsTest2/source/main.c:37:10: error: ISO C90 does not support ‘long long’ [-Werror=long-long]
6063
}
6164

6265
var clangCompiler = settings as ClangCommon.ICommonCompilerSettings;
@@ -66,7 +69,7 @@ protected override void
6669
clangCompiler.ExtraWarnings = true;
6770
clangCompiler.Pedantic = true;
6871

69-
// this is intentionally common, as it's part of pedantic, which is disabled
72+
// this is intentionally common to all source, as it's part of pedantic, which is disabled
7073
// for a specific file
7174
// the issue arises for the other file and the order of this warning suppression
7275
// and enabling 'pedantic' on the Xcode generated command line
@@ -83,6 +86,11 @@ protected override void
8386
{
8487
clangCompiler.Pedantic = false; // DeltaSettingsTest2/source/literal.c:35:9: error: string literal of length 510 exceeds maximum length 509 that C90 compilers are required to support [-Werror,-Woverlength-strings]
8588
}
89+
var gccCompiler = settings as GccCommon.ICommonCompilerSettings;
90+
if (null != gccCompiler)
91+
{
92+
gccCompiler.Pedantic = false; // DeltaSettingsTest2/source/literal.c:35:9: error: string length ‘510’ is greater than the length ‘509’ ISO C90 compilers are required to support [-Werror=overlength-strings]
93+
}
8694
}));
8795

8896
if (this.Linker is VisualCCommon.LinkerBase)

tests/bamtests.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,8 @@ def configure_repository():
109109
linux={"Native": [gcc64, gcc32], "MakeFile": [gcc64, gcc32]},
110110
osx={"Native": [clang64, clang32], "MakeFile": [clang64, clang32], "Xcode": [clang64, clang32]})
111111
configs["DeltaSettingsTest2"] = TestSetup(win={"Native": [visualc64, visualc32], "VSSolution": [visualc64, visualc32]},
112-
linux={"Native": [gcc64, gcc32]},
113-
osx={"Native": [clang64, clang32], "Xcode": [clang64, clang32]})
112+
linux={"Native": [gcc64, gcc32], "MakeFile": [gcc64, gcc32]},
113+
osx={"Native": [clang64, clang32], "MakeFile": [clang64, clang32], "Xcode": [clang64, clang32]})
114114
configs["AssemblerTest1"] = TestSetup(win={"Native": [visualc64, visualc32, mingw32], "VSSolution": [visualc64, visualc32]},
115115
linux={"Native": [gcc64, gcc32], "MakeFile": [gcc64, gcc32]},
116116
osx={"Native": [clang64, clang32], "MakeFile": [clang64, clang32], "Xcode": [clang64, clang32]})

0 commit comments

Comments
 (0)