Windows viewer installer ignores `/D` switch (to override install directory)

Open

#3,098 opened on Nov 15, 2024 0 comments 0 reactions 1 assignee View on GitHub

@nat-goodspeed is already working on this.

Since Nov 19, 2024.

Assessment

This issue has not been assessed yet.

Description

bug team:viewer

Environment

Second Life Release 7.1.10.10800445603 (64bit)
Release Notes

CPU: AMD Ryzen Threadripper 3960X 24-Core Processor (3800.11 MHz)
Memory: 65413 MB
OS Version: Microsoft Windows 10 64-bit (Build 19045.5011)
Graphics Card Vendor: NVIDIA Corporation
Graphics Card: NVIDIA GeForce GTX 970/PCIe/SSE2

Windows Graphics Driver Version: 32.0.15.6603
OpenGL Version: 4.6.0 NVIDIA 566.03

Window size: 2060x1562
Font Size Adjustment: 96pt
UI Scaling: 1
Draw distance: 128m
Bandwidth: 3000kbit/s
LOD factor: 1.25
Render quality: 2
Texture memory: 4096MB
Disk cache: Max size 1638.4 MB (9.6% used)

J2C Decoder Version: KDU v7.10.4
Audio Driver Version: OpenAL, version 1.1 ALSOFT 1.23.1 / OpenAL Community / OpenAL Soft: OpenAL Soft
Dullahan: 1.14.0.202408091639
CEF: 118.4.1+g3dd6078+chromium-118.0.5993.54
Chromium: 118.0.5993.54
LibVLC Version: 3.0.21
Voice Server Version:

November 15 2024 11:42:51

Description

Automated test scripts (e.g. secondlife/vita#1) want to be able to install the subject viewer to a temporary directory, without overwriting the test user's usual viewer. The NSIS installer accepts a /D switch, allowing a command line invocation to specify the default install directory. This has worked in the past.

The current Windows installer_template.nsi logic changed so that the resulting installer's /D switch is ignored.

Reproduction steps

Download any recent viewer installer.
Open Command Prompt.
Type "full\pathname\of\installer.exe" /D=desired\install\directory
(note: no quotes around desired\install\directory)
Expected: The default install directory will be desired\install\directory
Actual: The default install directory is C:\Program Files\<channel>
e.g. C:\Program Files\SecondLifeViewer

Dominant language: C++
Stars: 299
Forks: 146
Avg merge: 1d 9h
Merged PRs (30d): 88

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

Eval bug: SIGSEGV in token-counting routes when the request arrives while the server is sleeping (stale vocab/mctx captured before the wake barrier) Open

Difficulty 2/5 1-3 hours Newbie friendliness 86/100

ggml-org/llama.cpp#29188 · Opened Sep 20, 2026
Sensor initialization takes very long when `--initial-sim-time` is set to current UNIX timestamp Open

Difficulty 2/5 1-3 hours Newbie friendliness 78/100

gazebosim/gz-sensors#662 · 1 comment · Opened Sep 20, 2026
Aligned columns on the serial monitor while keeping CSV on the SD card Open

enhancement

Difficulty 2/5 1-3 hours Newbie friendliness 76/100

NorthernWidget/Margay_Library#31 · Opened Sep 20, 2026
v1 manifest list: existing_files_count is the table-wide total-data-files Open

comp-datalake

Difficulty 2/5 1-3 hours Newbie friendliness 88/100

ClickHouse/ClickHouse#121222 · Opened Sep 20, 2026
Case collision between `Canvas` and `canvas` test directories causes platform-dependent source hashes Open

Difficulty 2/5 1-3 hours Newbie friendliness 68/100

LadybirdBrowser/ladybird#12123 · Opened Sep 20, 2026

More C++ issues

Windows viewer installer ignores `/D` switch (to override install directory)

Assessment

Description

Environment

Description

Reproduction steps

Contributor guide

First steps

More from secondlife/viewer

Similar issues

Get new issues in your inbox