Skip to content

fix: use backslashes for Windows paths in submitter installer - #275

Merged
joel-wong-aws merged 1 commit into
aws-deadline:mainlinefrom
joel-wong-aws:correct_installer_paths
Aug 7, 2025
Merged

fix: use backslashes for Windows paths in submitter installer#275
joel-wong-aws merged 1 commit into
aws-deadline:mainlinefrom
joel-wong-aws:correct_installer_paths

Conversation

@joel-wong-aws

Copy link
Copy Markdown
Contributor

Fixes: #272

What was the problem/requirement? (What/Why)

The installer env variables use forward slashes instead of backslashes on Windows, leading to unexpected environment variable values.

What was the solution? (How)

Updated the env vars to use backslashes on Windows.

How was this change tested?

hatch run installer:build-installer --local-dev --platform windows

Ran the DeadlineCloudSubmitterForCinema4D.exe in the following scenarios:

  • User install over an existing user install: successfully submitted a job
  • User install after previous installed removed: successfully submitted a job
  • System install after previous install removed: successfully submitted a job (requires permission updates to the installation folder to allow installing the GUI modules)
  • System install over an existing system install: successfully submitted a job

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@joel-wong-aws
joel-wong-aws requested a review from a team as a code owner August 7, 2025 19:22
@joel-wong-aws
joel-wong-aws enabled auto-merge (squash) August 7, 2025 19:33
Signed-off-by: Joel Wong <127782171+joel-wong-aws@users.noreply.github.com>
@joel-wong-aws
joel-wong-aws force-pushed the correct_installer_paths branch from 830947d to ae46faa Compare August 7, 2025 19:39
@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2025

Copy link
Copy Markdown

@joel-wong-aws
joel-wong-aws merged commit 3d39370 into aws-deadline:mainline Aug 7, 2025
16 checks passed
@joel-wong-aws
joel-wong-aws deleted the correct_installer_paths branch August 8, 2025 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Installer misconfigures environment variables

3 participants