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

Commit a70d64b

Browse files
committed
fix: use the xxd tool from the image, instead of downloading vim
1 parent a92fce1 commit a70d64b

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

snapshot/android/snapshot-generator.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,8 @@ SnapshotGenerator.prototype.getSnapshotToolCommand = function (snapshotToolPath,
248248
}
249249

250250
SnapshotGenerator.prototype.getXxdCommand = function (srcOutputDir) {
251-
return `xxd -i ${SNAPSHOT_BLOB_NAME}.blob > ${join(srcOutputDir, `${SNAPSHOT_BLOB_NAME}.c`)}`;
251+
// https://github.com/NativeScript/docker-images/tree/master/v8-snapshot/bin
252+
return `/bin/xxd -i ${SNAPSHOT_BLOB_NAME}.blob > ${join(srcOutputDir, `${SNAPSHOT_BLOB_NAME}.c`)}`;
252253
}
253254

254255
SnapshotGenerator.prototype.getPathInDocker = function (mappedLocalDir, mappedDockerDir, targetPath) {
@@ -309,8 +310,7 @@ SnapshotGenerator.prototype.buildCSource = function (androidArch, blobInputDir,
309310
const blobsInputInDocker = `/blobs/${androidArch}`
310311
const srcOutputDirInDocker = `/dist/src/${androidArch}`;
311312
const buildCSourceCommand = this.getXxdCommand(srcOutputDirInDocker);
312-
// add vim in order to get xxd
313-
command = `docker run -v "${blobInputDir}:${blobsInputInDocker}" -v "${srcOutputDir}:${srcOutputDirInDocker}" ${SNAPSHOTS_DOCKER_IMAGE} /bin/sh -c "cd ${blobsInputInDocker} && apk add vim && ${buildCSourceCommand}"`;
313+
command = `docker run --rm -v "${blobInputDir}:${blobsInputInDocker}" -v "${srcOutputDir}:${srcOutputDirInDocker}" ${SNAPSHOTS_DOCKER_IMAGE} /bin/sh -c "cd ${blobsInputInDocker} && ${buildCSourceCommand}"`;
314314
}
315315
else {
316316
command = this.getXxdCommand(srcOutputDir);
@@ -347,7 +347,7 @@ SnapshotGenerator.prototype.runMksnapshotTool = function (tool, mksnapshotParams
347347
const inputFilePathInDocker = this.getPathInDocker(inputFileDir, appDirInDocker, inputFile);
348348
const outputPathInDocker = this.getPathInDocker(blobOutputDir, blobOutputDirInDocker, blobOutputDir);
349349
const toolCommandInDocker = this.getSnapshotToolCommand(toolPathInDocker, inputFilePathInDocker, outputPathInDocker, toolParams);
350-
command = `docker run -v "${inputFileDir}:${appDirInDocker}" -v "${blobOutputDir}:${blobOutputDirInDocker}" ${SNAPSHOTS_DOCKER_IMAGE} /bin/sh -c "${toolCommandInDocker}"`;
350+
command = `docker run --rm -v "${inputFileDir}:${appDirInDocker}" -v "${blobOutputDir}:${blobOutputDirInDocker}" ${SNAPSHOTS_DOCKER_IMAGE} /bin/sh -c "${toolCommandInDocker}"`;
351351
} else {
352352
command = this.getSnapshotToolCommand(toolPath, inputFile, blobOutputDir, toolParams);
353353
}

0 commit comments

Comments
 (0)