Skip to content

Commit 597e0a2

Browse files
authored
Merge pull request #722 from dbt-msft/fix/devcontainer-odbc-env-standard
Relocate SQLSERVER_TEST_DRIVER to setup_env.sh to comply with no spaces.
2 parents d4ecf7e + cb4c59c commit 597e0a2

2 files changed

Lines changed: 6 additions & 1 deletion

File tree

.devcontainer/devcontainer.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
}
2121
},
2222
"containerEnv": {
23-
"SQLSERVER_TEST_DRIVER": "ODBC Driver 18 for SQL Server",
2423
"SQLSERVER_TEST_HOST": "127.0.0.1",
2524
"SQLSERVER_TEST_USER": "SA",
2625
"SQLSERVER_TEST_PASS": "L0calTesting!",

.devcontainer/setup_env.sh

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
set -euo pipefail
22

3+
# Persist for all shell sessions (Zed uses devcontainer exec, no VS Code server for remoteEnv)
4+
if ! grep -qF 'SQLSERVER_TEST_DRIVER' ~/.bashrc 2>/dev/null; then
5+
echo 'export SQLSERVER_TEST_DRIVER="ODBC Driver 18 for SQL Server"' >> ~/.bashrc
6+
fi
7+
export SQLSERVER_TEST_DRIVER="ODBC Driver 18 for SQL Server"
8+
39
cp test.env.sample test.env
410

511
sudo apt-get update

0 commit comments

Comments
 (0)