Skip to content

Commit efab700

Browse files
committed
copy in devcontainer-features.builtin.env
1 parent e9b1947 commit efab700

2 files changed

Lines changed: 3 additions & 4 deletions

File tree

src/spec-configuration/containerFeaturesConfiguration.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -237,7 +237,9 @@ FROM $_DEV_CONTAINERS_BASE_IMAGE AS dev_containers_target_stage
237237
USER root
238238
239239
${(() => {
240-
return useBuildKitBuildContexts ? '' : `
240+
return useBuildKitBuildContexts ? `
241+
COPY --from=dev_containers_feature_content_source {contentSourceRootPath}/devcontainer-features.builtin.env /tmp/build-features/
242+
` : `
241243
COPY --from=dev_containers_feature_content_normalize /tmp/build-features /tmp/build-features
242244
`;
243245
})()}
@@ -319,7 +321,6 @@ function escapeQuotesForShell(input: string) {
319321

320322
export function getFeatureLayers(featuresConfig: FeaturesConfig, containerUser: string, remoteUser: string, useBuildKitBuildContexts = false, contentSourceRootPath = '/tmp/build-features/') {
321323
let result = `RUN \\
322-
mkdir -p /tmp/build-features && \\
323324
echo "_CONTAINER_USER_HOME=$(getent passwd ${containerUser} | cut -d: -f6)" >> /tmp/build-features/devcontainer-features.builtin.env && \\
324325
echo "_REMOTE_USER_HOME=$(getent passwd ${remoteUser} | cut -d: -f6)" >> /tmp/build-features/devcontainer-features.builtin.env
325326

src/test/container-features/generateFeaturesConfig.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,6 @@ describe('validate generateFeaturesConfig()', function () {
7272
// getFeatureLayers
7373
const actualLayers = getFeatureLayers(featuresConfig, 'testContainerUser', 'testRemoteUser');
7474
const expectedLayers = `RUN \\
75-
mkdir -p /tmp/build-features && \\
7675
echo "_CONTAINER_USER_HOME=$(getent passwd testContainerUser | cut -d: -f6)" >> /tmp/build-features/devcontainer-features.builtin.env && \\
7776
echo "_REMOTE_USER_HOME=$(getent passwd testRemoteUser | cut -d: -f6)" >> /tmp/build-features/devcontainer-features.builtin.env
7877
@@ -131,7 +130,6 @@ RUN cd /tmp/build-features/second_2 \\
131130
// getFeatureLayers
132131
const actualLayers = getFeatureLayers(featuresConfig, 'testContainerUser', 'testRemoteUser');
133132
const expectedLayers = `RUN \\
134-
mkdir -p /tmp/build-features && \\
135133
echo "_CONTAINER_USER_HOME=$(getent passwd testContainerUser | cut -d: -f6)" >> /tmp/build-features/devcontainer-features.builtin.env && \\
136134
echo "_REMOTE_USER_HOME=$(getent passwd testRemoteUser | cut -d: -f6)" >> /tmp/build-features/devcontainer-features.builtin.env
137135

0 commit comments

Comments
 (0)