Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ansible request to fix MSVS_2013 broken installation #2178

Closed
1 of 3 tasks
Willsparker opened this issue May 10, 2021 · 6 comments · Fixed by #2740
Closed
1 of 3 tasks

Ansible request to fix MSVS_2013 broken installation #2178

Willsparker opened this issue May 10, 2021 · 6 comments · Fixed by #2740

Comments

@Willsparker
Copy link
Contributor

Ref:

Details:
Registering the MSVS_2013 shared libraries (this task), intermittently fails with :

10:59:23 TASK [MSVS_2013 : Register Visual Studio Community 2013 DIA SDK shared libraries] ***
10:59:26 failed: [127.0.0.1] (item=C:\Program Files (x86)\Microsoft Visual Studio 12.0\DIA SDK\bin\msdia120.dll) => {"ansible_loop_var": "item", "changed": true, "cmd": "regsvr32 /s \"C:\\Program Files (x86)\\Microsoft Visual Studio 12.0\\DIA SDK\\bin\\msdia120.dll\"", "delta": "0:00:00.093433", "end": "2021-05-05 09:57:05.619367", "item": "C:\\Program Files (x86)\\Microsoft Visual Studio 12.0\\DIA SDK\\bin\\msdia120.dll", "msg": "non-zero return code", "rc": 3, "start": "2021-05-05 09:57:05.525933", "stderr": "", "stderr_lines": [], "stdout": "", "stdout_lines": []}

(return code 3).

There's no stderr because regsvr32 is being run with /s, however that may help with debugging. See: https://devblogs.microsoft.com/oldnewthing/20180921-00/?p=99795 which may give some insight.

According to above, RC=3 means the loadlibrary call is failing (or that it's a red herring). Either way, removing /s will give some insight, so that's where I'll start.

@Willsparker Willsparker self-assigned this May 10, 2021
@Willsparker Willsparker changed the title Ansible request for MSVS_2013 Ansible request for MSVS_2013 register libraries bug May 10, 2021
@Willsparker
Copy link
Contributor Author

VPC Test, with removal of the /s tag: https://ci.adoptopenjdk.net/job/VagrantPlaybookCheck/1162/#
(the branch I'm doing the work in can be see here: https://github.com/Willsparker/openjdk-infrastructure/tree/2178)

@Willsparker
Copy link
Contributor Author

Adding /s did not work, unfortunately, as it caused that task to hang - I'll have to try testing this locally..

@Willsparker
Copy link
Contributor Author

It appears our MSVS_2013 installation is broken (I would assume from the same changes that Microsoft did that caused the MSVS_2010 installation to break). The playbook task will report the installation task as correct, but will then fail when registering the libraries ; When I tried MSVS_2013 installation on a local VM, I had an 'Unable to Locate Package Source' issue for VS_PreClean_vs.exe, whatever that means. Anyway, this issue has instead become about getting MSVS_2013 installed correctly

@Willsparker Willsparker changed the title Ansible request for MSVS_2013 register libraries bug Ansible request to fix MSVS_2013 broken installation May 12, 2021
@sxa
Copy link
Member

sxa commented May 13, 2021

Related: #1270
Also, this is potentially an important consideration for @andrew-m-leonard as part of adoptium/temurin-build#2522

@sxa
Copy link
Member

sxa commented Sep 23, 2021

If we could have a local copy of the installer available, as mentioned in #1270 (comment), that could provide a solution for this

@steelhead31
Copy link
Contributor

I have a fix for the VS2013 installation, by creating a local installation media for the Community Edition ( via MSDN subscription ), VS2013 can be installed. The new media has been placed in the vendor files, and Im working on the playbook changes to copy this media, and install from that, rather than via the web downloader.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

5 participants