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

Early Visual Studio 2017 compilation supports & intructions #1447

Closed
wants to merge 3 commits into from

Conversation

trippleflux
Copy link

Tested with compiling latest erlang x64 master branch's commit on visual studio 2017 enterprise
The only last remaining problem is
src\otp\erts\etc\win32\nsis\find_redist.sh seems failed to finding
"vcredist_x64.exe"which is located in

"C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Redist\MSVC\14.10.25008"

The cl.exe for msvc 2017 enterprise is located in

" C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\amd64"

Based on old INSTAL_WIN32.md direction of recommending visual studio 12.0
or 2013, the cl.exe location for msvc 2013 x64 is :

"C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\bin\amd64"

And the vcredist_x64.exe for msvc 2013 is :

"C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\redist\1033".

When running "./otp_build installer_win32" i have got the following
error "Failed to locate vcredist_x64.exe because cl.exe was in an
unexpected location".

Any of erlang windows developer wanted to add some more info about
these problems?.

Changes the visual studio 2017 and sdk path on cygwin .bash_profile
Fixed compile error on visual studio 2015 and above caused by already defined static types.
@IngelaAndin IngelaAndin added the team:VM Assigned to OTP team VM label May 10, 2017
@dgud
Copy link
Contributor

dgud commented May 11, 2017

Thanks.

I don't want to change the instructions/recommendations of how to build on windows,
since the way it's currently described are how we do it and what we test with.

For now make a gist or add an entry on the wiki so it's not completely forgotten.

I think I will change the wx gl typedef and close this PR?

And yes we need to fix the scripts to locate the vcredist installer, but I don't have time for that now.

typedef int int32_t;
typedef __int64 int64_t;
typedef unsigned __int64 uint64_t;
#elif _MSC_VER < 1900
typedef long int int32_t;
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not only change it to:
typedef __int32 int32_t;

Copy link
Author

Choose a reason for hiding this comment

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

I was only copying the new static types from the new vs 2017 header file. My knowledge doesn't quite up to par with the your change :-(

@dgud dgud self-assigned this May 11, 2017
@trippleflux
Copy link
Author

Fair enough, i merely self compiled it because of new changes from github that i am needed and possible windows's assemblies binary fixes and optimization in above vs 2013. Possibly you could add my instruction changes into the erlang's github wikis?.

@dgud
Copy link
Contributor

dgud commented May 18, 2017

Fixed compilation and added link to this PR in wiki

@dgud dgud closed this May 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:VM Assigned to OTP team VM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants