Skip to content

Commit 10de1f4

Browse files
authored
Merge branch 'main' into conda-upgrade
2 parents 2ec518f + dc8cf55 commit 10de1f4

7 files changed

Lines changed: 33 additions & 10 deletions

File tree

src/conda/install.sh

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,9 @@ if ! conda --version &> /dev/null ; then
152152
install_user_package cryptography
153153
# Due to https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-40897
154154
install_user_package setuptools
155+
156+
install_user_package pluggy
157+
155158
fi
156159

157160
# Display a notice on conda when not running in GitHub Codespaces

src/desktop-lite/devcontainer-feature.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"id": "desktop-lite",
3-
"version": "1.2.8",
3+
"version": "1.2.9",
44
"name": "Light-weight Desktop",
55
"documentationURL": "https://github.com/devcontainers/features/tree/main/src/desktop-lite",
66
"description": "Adds a lightweight Fluxbox based desktop to the container that can be accessed using a VNC viewer or the web. GUI-based commands executed from the built-in VS code terminal will open on the desktop automatically.",

src/desktop-lite/install.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -409,9 +409,9 @@ else
409409
fi
410410
411411
# Run whatever was passed in
412-
if [ -n "$1" ]; then
412+
if [ -n "\$1" ]; then
413413
log "Executing \"\$@\"."
414-
exec "$@"
414+
exec "\$@"
415415
else
416416
log "No command provided to execute."
417417
fi

test/conda/conda_channel_creation.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,5 +11,4 @@ check "conda-install-tensorflow" bash -c "conda create --name tensorflow-test-en
1111
check "conda-install-pytorch" bash -c "conda create --name pytorch-test-env -c conda-forge --yes pytorch"
1212

1313
# Report result
14-
reportResults
15-
14+
reportResults

test/conda/install_conda_package_after_upgrade.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ check "conda updated version" conda --version
1414
check "install pyopenssl" conda install -y -c defaults pyopenssl
1515
check "install cryptography" conda install -y -c defaults cryptography
1616
check "install scipy with bioconda" conda install -y -c bioconda scipy
17+
check "conda-forge" conda config --show channels | grep conda-forge
1718
check "if conda-notice.txt exists" cat /usr/local/etc/vscode-dev-containers/conda-notice.txt
1819

1920
# Report result
20-
reportResults
21-
21+
reportResults

test/desktop-lite/scenarios.json

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,12 @@
1212
"noVncVersion": "1.2.0"
1313
}
1414
}
15-
},
15+
},
1616
"test_vnc_resolution_as_container_env_var": {
1717
"image": "ubuntu:noble",
1818
"features": {
1919
"desktop-lite": {}
20-
}
21-
,
20+
},
2221
"containerEnv": {
2322
"VNC_RESOLUTION": "1920x1080x32"
2423
},
@@ -45,5 +44,11 @@
4544
"features": {
4645
"desktop-lite": {}
4746
}
47+
},
48+
"test_desktop_init_exec_passthrough": {
49+
"image": "ubuntu:noble",
50+
"features": {
51+
"desktop-lite": {}
52+
}
4853
}
4954
}
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
#!/bin/bash
2+
3+
set -e
4+
5+
# Optional: Import test library
6+
source dev-container-features-test-lib
7+
8+
# Verify that desktop-init.sh correctly passes through commands.
9+
# Previously, the heredoc in install.sh did not escape $1 and $@, causing them to expand
10+
# to empty strings at install time, so any command passed to desktop-init.sh was silently ignored.
11+
12+
check "command is passed through and executed" \
13+
bash -c "result=\$(/usr/local/share/desktop-init.sh echo 'passthrough-test-token' 2>/dev/null) && echo \"\$result\" | grep -q 'passthrough-test-token'"
14+
15+
# Report result
16+
reportResults

0 commit comments

Comments
 (0)