Skip to content

Use powershell script to switch installed connext versions. - #812

Draft
nuclearsandwich wants to merge 15 commits into
masterfrom
nuclearsandwich/win-connext-installer
Draft

Use powershell script to switch installed connext versions.#812
nuclearsandwich wants to merge 15 commits into
masterfrom
nuclearsandwich/win-connext-installer

Conversation

@nuclearsandwich

@nuclearsandwich nuclearsandwich commented Apr 8, 2025

Copy link
Copy Markdown
Member

Needs a rebase after #811 merges.

@nuclearsandwich nuclearsandwich self-assigned this Apr 8, 2025
@nuclearsandwich

Copy link
Copy Markdown
Member Author

Part of me hopes there's something wrong here because if there isn't it means that I wrote a powershell script blind and then it worked: Build Status

@nuclearsandwich

Copy link
Copy Markdown
Member Author

Test CI. Build Status

@nuclearsandwich
nuclearsandwich force-pushed the nuclearsandwich/win-connext-installer branch from fcb5635 to 0cf16d6 Compare April 11, 2025 17:58
Comment on lines +15 to +19
cmd.exe /c "$Env:CONNEXTDDS_DIR\bin\rtipkginstall.bat" "-u" "C:\TEMP\rticonnextdds-src\openssl-3.0.12-7.3.0-host-x64Win64.rtipkg"
cmd.exe /c "$Env:CONNEXTDDS_DIR\bin\rtipkginstall.bat" "-u" "C:\TEMP\rticonnextdds-src\openssl-3.0.12-7.3.0-target-x64Win64VS2017.rtipkg"
cmd.exe /c "$Env:CONNEXTDDS_DIR\bin\rtipkginstall.bat" "-u" "C:\TEMP\rticonnextdds-src\rti_connext_dds-7.3.0-pro-target-x64Win64VS2017.rtipkg"
cmd.exe /c "$Env:CONNEXTDDS_DIR\bin\rtipkginstall.bat" "-u" "C:\TEMP\rticonnextdds-src\rti_security_plugins-7.3.0-host-x64Win64.rtipkg"
cmd.exe /c "$Env:CONNEXTDDS_DIR\bin\rtipkginstall.bat" "-u" "C:\TEMP\rticonnextdds-src\rti_security_plugins-7.3.0-target-x64Win64VS2017.rtipkg"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lobolanja I could use a second set of eyes here. I'm getting

The system cannot find the path specified.

Errors trying to run these commands but as far as I can see, they're the same invocation as was in the original Dockerfile.

@fgallegosalido

Copy link
Copy Markdown
Contributor

@cottsay I think this pull request is not necessary anymore. This was done as part of #844

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

5 participants