WIP: 915 Windows deployment script #948

Draft
a24timsv wants to merge 1 commit from 915-Windows-deployment-script into team_1_week_6
Collaborator

Added the Windows deployment script and changed the Unix script to match the coding conventions.
Also changed the names.

Relevant issue: #915

Added the Windows deployment script and changed the Unix script to match the coding conventions. Also changed the names. Relevant issue: #915
Added the Windows deployment script and changed the Unix script to match the coding conventions.
Also changed the names.
Collaborator

Review

  • Everything seems good

Question before merge ready:
Should we try it out on a windows machine in order to actually know that it works?

# Review - Everything seems good Question before merge ready: Should we try it out on a windows machine in order to actually know that it works?
a22erigr removed review request for b24idalu 2026-05-13 13:17:43 +00:00
Author
Collaborator

Yes, try it and post result from server logs.

Yes, try it and post result from server logs.
Collaborator

image

Tried to execute the script on my Windows PC and it could not find the path.

![image](/attachments/8f6b2fb2-6e4c-4198-8415-7f66697c3262) Tried to execute the script on my Windows PC and it could not find the path.
a24julot changed target branch from Temp-Main-Week-5 to team_1_week_6 2026-05-13 13:55:20 +00:00
Collaborator

Apparently Windows require special permissions of some sort to SSH with the private.key, we will talk to Henrik about the key because we suspect that it is a private key, but we should probably have a public key.

Apparently Windows require special permissions of some sort to SSH with the private.key, we will talk to Henrik about the key because we suspect that it is a private key, but we should probably have a public key.
b24johka changed title from 915 Windows deployment script to WIP: 915 Windows deployment script 2026-05-15 10:11:43 +00:00
Collaborator

A discussion has been taking place about if it would be a simple fix to use the docker WSL (Windows Subsystem for Linux), we already have it since docker is running WSL. It could work out fine in order for Windows users to be able to deploy.
I will create a separate issue on this.

A discussion has been taking place about if it would be a simple fix to use the docker WSL (Windows Subsystem for Linux), we already have it since docker is running WSL. It could work out fine in order for Windows users to be able to deploy. I will create a separate issue on this.
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin 915-Windows-deployment-script:915-Windows-deployment-script
git switch 915-Windows-deployment-script

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch team_1_week_6
git merge --no-ff 915-Windows-deployment-script
git switch 915-Windows-deployment-script
git rebase team_1_week_6
git switch team_1_week_6
git merge --ff-only 915-Windows-deployment-script
git switch 915-Windows-deployment-script
git rebase team_1_week_6
git switch team_1_week_6
git merge --no-ff 915-Windows-deployment-script
git switch team_1_week_6
git merge --squash 915-Windows-deployment-script
git switch team_1_week_6
git merge --ff-only 915-Windows-deployment-script
git switch team_1_week_6
git merge 915-Windows-deployment-script
git push origin team_1_week_6
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!948
No description provided.