Skip to content

Commit c2109c1

Browse files
committed
Merge branch '140-native-binaries' into 276-rename-omp-to-esmf
2 parents 0432fe5 + d2bd78c commit c2109c1

File tree

3 files changed

+8
-9
lines changed

3 files changed

+8
-9
lines changed

.github/workflows/pull_request_check.yml

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,15 @@ jobs:
1010
strategy:
1111
fail-fast: true
1212
matrix:
13-
os: [windows-latest, ubuntu-latest]
13+
os: [windows-latest, ubuntu-20.04]
1414
steps:
1515
- name: Checkout repository
1616
uses: actions/checkout@v3
1717

1818
- name: Setup JDK
1919
uses: graalvm/setup-graalvm@v1
2020
with:
21-
version: '22.3.0'
21+
version: '22.3.1'
2222
java-version: '17'
2323
components: 'native-image,js'
2424
github-token: ${{ secrets.GITHUB_TOKEN }}
@@ -45,7 +45,7 @@ jobs:
4545
disk-root: "C:"
4646

4747
- name: Set Swap Space (Linux)
48-
if: matrix.os == 'ubuntu-latest'
48+
if: matrix.os == 'ubuntu-20.04'
4949
uses: pierotofy/set-swap-space@master
5050
with:
5151
swap-size-gb: 12
@@ -62,7 +62,7 @@ jobs:
6262

6363
- name: Upload executable jar
6464
# We only need one OS job to upload the jars
65-
if: matrix.os == 'ubuntu-latest'
65+
if: matrix.os == 'ubuntu-20.04'
6666
uses: actions/upload-artifact@v2
6767
with:
6868
name: samm-cli-jar
@@ -77,9 +77,11 @@ jobs:
7777
# upload-artifact will upload the one(s) that are actually present
7878
path: |
7979
tools/samm-cli/target/samm.exe
80+
tools/samm-cli/target/*.dll
81+
tools/samm-cli/target/lib/
8082
8183
- name: Upload binary (Linux/Mac)
82-
if: matrix.os == 'ubuntu-latest' || matrix.os == 'macos-latest'
84+
if: matrix.os == 'ubuntu-20.04' || matrix.os == 'macos-latest'
8385
uses: actions/upload-artifact@v2
8486
with:
8587
name: samm-cli-binary-${{ matrix.os }}

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ Are you having trouble with ESMF SDK? We want to help!
5656

5757
The top level elements of the SDK structure are all carried out as Maven multimodule projects.
5858
Building the SDK requires a Java 17-compatible [GraalVM JDK](https://www.graalvm.org/) with the 'js'
59-
component installed. If you with to build the samm-cli (see below), you will also have to have the
59+
component installed. If you want to build the samm-cli (see below), you will also have to have the
6060
'native-image' GraalVM component installed.
6161

6262
To build the SDK core components, run the following command:

core/esmf-aspect-model-validator/src/main/java/org/eclipse/esmf/aspectmodel/validation/services/AspectModelValidator.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -107,9 +107,6 @@ public List<Violation> validateModel( final Try<VersionedModel> versionedModel )
107107
// To catch false positives, also try to load the model
108108
final Try<List<ModelElement>> modelElements = AspectModelLoader.getElements( model );
109109
if ( modelElements.isFailure() && !(modelElements.getCause() instanceof InvalidRootElementCountException) ) {
110-
final ByteArrayOutputStream out = new ByteArrayOutputStream();
111-
model.getModel().write( out, "ttl" );
112-
LOG.debug( out.toString( StandardCharsets.UTF_8 ) );
113110
return List.of( new ProcessingViolation(
114111
"Validation succeeded, but an error was found while processing the model. "
115112
+ "This indicates an error in the model validation; please consider reporting this issue including the model "

0 commit comments

Comments
 (0)