Skip to content

Commit 4f79845

Browse files
Merge 24.5 to develop
2 parents 95d4c03 + b34cdca commit 4f79845

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

SequenceAnalysis/build.gradle

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ dependencies {
163163
if (project.findProject(BuildUtils.getTestProjectPath(project.gradle)) != null && project.hasProperty("teamcity"))
164164
{
165165
project.evaluationDependsOn(BuildUtils.getTestProjectPath(project.gradle))
166-
def configDir = "${ServerDeployExtension.getServerDeployDirectory(project)}/config"
166+
def configDir = new File(ServerDeployExtension.getServerDeployDirectory(project), "config")
167167
def testProject = project.findProject(BuildUtils.getTestProjectPath(project.gradle))
168168
def createPipelineConfigTask = project.tasks.register("createPipelineConfig", Copy) {
169169
Copy task ->
@@ -183,13 +183,9 @@ if (project.findProject(BuildUtils.getTestProjectPath(project.gradle)) != null &
183183
return newLine
184184

185185
})
186-
task.destinationDir = new File(configDir)
186+
task.destinationDir = configDir
187187

188188
if (BuildUtils.useEmbeddedTomcat(project)) {
189-
task.doFirst {
190-
new File(new File(BuildUtils.getEmbeddedConfigPath(project)), "application.properties") << "\n${configDir}"
191-
}
192-
193189
rootProject.allprojects {
194190
task.mustRunAfter tasks.withType(DoThenSetup)
195191
}
@@ -199,7 +195,8 @@ if (project.findProject(BuildUtils.getTestProjectPath(project.gradle)) != null &
199195
dependsOn(createPipelineConfigTask)
200196
if (BuildUtils.useEmbeddedTomcat(project)) {
201197
it.doFirst {
202-
new File(new File(BuildUtils.getEmbeddedConfigPath(project)), "application.properties") << "\ncontext.pipelineConfig=${configDir}"
198+
new File(new File(BuildUtils.getEmbeddedConfigPath(project)), "application.properties")
199+
<< "\ncontext.pipelineConfig=${configDir.getAbsolutePath().replace("\\", "\\\\")}"
203200
}
204201
}
205202
}

SequenceAnalysis/src/org/labkey/sequenceanalysis/SequenceRemoteIntegrationTests.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,10 +83,12 @@ private File setupConfigDir(File outDir) throws IOException
8383

8484
path = path.replaceAll("\\\\", "/");
8585
line = line.replaceAll("@@SEQUENCEANALYSIS_TOOLS@@", path);
86+
_log.info("Writing to pipelineConfig.xml: " + line);
8687
}
8788
else if (line.contains("@@WORK_DIR@@"))
8889
{
8990
line = line.replaceAll("@@WORK_DIR@@", outDir.getPath().replaceAll("\\\\", "/"));
91+
_log.info("Writing to pipelineConfig.xml: " + line);
9092
}
9193

9294
writer.println(line);
@@ -213,6 +215,8 @@ protected void executeJobRemote(File workDir, @Nullable File jobJson) throws IOE
213215
ProcessBuilder pb = new ProcessBuilder(args);
214216
pb.directory(workDir);
215217

218+
_log.info("Executing job in '" + pb.directory().getAbsolutePath() + "': " + String.join(" ", pb.command()));
219+
216220
Process proc;
217221
try
218222
{

0 commit comments

Comments
 (0)