Conversation
docker-compose.build.yml
Outdated
| - ./dist:/dist | ||
| - ./jacoco:/jacoco | ||
| command: > | ||
| sh -c "cp /root/.m2/repository/org/evomaster/evomaster-client-java-instrumentation/5.0.3-SNAPSHOT/evomaster-client-java-instrumentation-5.0.3-SNAPSHOT.jar /dist/evomaster-agent.jar && |
There was a problem hiding this comment.
you will need to update scripts/version.py to handle this hardcoded 5.0.3-SNAPSHOT
scripts/dist-docker.sh
Outdated
| @@ -0,0 +1,399 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
there is an extra feature we need here.
by default, nothing about EvoMaster should be built or required (only the JAR of the SUTs are built, with no agent and no EM drivers).
we can have an input option to build those as well (off by default)
scripts/dist-docker.sh
Outdated
| echo "Tip: Use './scripts/dist-docker.sh <JDK> <TOOL>' for incremental builds" | ||
| echo " Example: ./scripts/dist-docker.sh 8 gradle" | ||
| exit 0 | ||
| fi |
There was a problem hiding this comment.
also, we should remove this. should be no interactive when running the script. should alwasy override if dist already exist. this also makes things easier to run in other scripts (eg on CI)
scripts/dist-docker.sh
Outdated
| @@ -0,0 +1,399 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
I guess this file can be now removed?
| echo 'JDK 21 Maven build completed' | ||
|
|
||
| copy-additional-files: | ||
| copy-jacoco: |
There was a problem hiding this comment.
could still be called copy-additional-files as maybe in future there might be more to add
| echo 'Additional files copied'" | ||
| echo 'Jacoco files copied'" | ||
|
|
||
| copy-evomaster-agent: |
There was a problem hiding this comment.
this includes as well the drivers. so maybe copy-evomaster-files?
There was a problem hiding this comment.
Actually, there is no need to use an additional service named copy-evomaster-files. I updated the logic.
2 participants
No description provided.