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

Build Windows with MSVC 2019 #44

Merged
merged 4 commits into from
Aug 24, 2020
Merged

Build Windows with MSVC 2019 #44

merged 4 commits into from
Aug 24, 2020

Conversation

aminya
Copy link
Member

@aminya aminya commented Jul 11, 2020

Description of the change

This uses MSVC 2019 to build Atom on Windows. It does this on the windows-2019 image in Azure. To test the backward compatibility, the built Atom is tested on windows-2016.

Verification

The tests pass. Extra tests are added to test the backward compatibility on older Windows.

Drawbacks

none

Release notes

  • build Atom with MSVC 2019 on Windows.
  • use windows-2019 in the CI

@aminya

This comment has been minimized.

@aminya aminya mentioned this pull request Jul 11, 2020
@aminya aminya marked this pull request as draft July 12, 2020 00:32
@aminya

This comment has been minimized.

@DeeDeeG

This comment has been minimized.

apm/package.json Outdated Show resolved Hide resolved
@aminya aminya mentioned this pull request Jul 14, 2020
@DeeDeeG

This comment has been minimized.

@aminya

This comment has been minimized.

Copy link
Member Author

@aminya aminya left a comment

Choose a reason for hiding this comment

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

This needs rebasing

@aminya aminya force-pushed the master branch 2 times, most recently from eb75182 to 0c956dd Compare July 16, 2020 11:02
@DeeDeeG

This comment has been minimized.

@DeeDeeG

This comment has been minimized.

@aminya

This comment has been minimized.

@DeeDeeG

This comment has been minimized.

@DeeDeeG

This comment has been minimized.

@DeeDeeG

This comment has been minimized.

@aminya

This comment has been minimized.

@DeeDeeG

This comment has been minimized.

@aminya
Copy link
Member Author

aminya commented Aug 22, 2020

We need to update npm in a separate pull request.

@aminya aminya force-pushed the master branch 2 times, most recently from f3c93bd to e1645f5 Compare August 22, 2020 09:35
@aminya aminya force-pushed the latest_windows branch 2 times, most recently from b1b536b to 9461eb2 Compare August 23, 2020 23:10
@aminya
Copy link
Member Author

aminya commented Aug 23, 2020

Python version is causing issues in the installation of windows-build-tools.

felixrieseberg/windows-build-tools#223

@DeeDeeG
Copy link
Member

DeeDeeG commented Aug 24, 2020

What was the error with Python 3? I am trying to debug what happened so I know what to do if this comes up again. But I am not sure what happened.

Edit: Endless install log, right?
https://dev.azure.com/DeeDeeG/b/_build/results?buildId=594&view=logs&j=2985f0af-e798-5fdc-91b8-be9f0a3685c5&t=57ffbcdc-0136-5068-0857-08af536fa868

Not sure what causes that. I saw it outside of CI once. I don't think it's to do with Python versions.

@DeeDeeG
Copy link
Member

DeeDeeG commented Aug 24, 2020

We can drop the windows-build-tools install and save ~two minutes from preparation! 🎉

https://dev.azure.com/DeeDeeG/b/_build/results?buildId=593&view=logs&j=2985f0af-e798-5fdc-91b8-be9f0a3685c5&t=0a53f124-4db9-5fc3-be81-e293757effc7

Committing here, but you have my permission to just delete the commit if you don't want it.


By the way: We should be able to drop the npm update on Windows, and the Python install.

If we update the macOS image to 10.15 macOS Catalina, we can drop npm updates altogether on every OS.

@aminya
Copy link
Member Author

aminya commented Aug 24, 2020

I want to test windows-build-tools first. I know I can use Visual Studio, but the installation of Visual Studio is hard for other users. If we can't build Atom using windows-build-tools ourselves, we should drop it from the documentation.

@aminya aminya force-pushed the latest_windows branch 2 times, most recently from b1e1bff to fc5d357 Compare August 24, 2020 11:39
@aminya aminya added this to In progress in Build, bootstrap, CI via automation Aug 24, 2020
@aminya aminya changed the title try windows-2019 image Build Windows with MSVC 2019 Aug 24, 2020
@aminya aminya marked this pull request as ready for review August 24, 2020 11:44
aminya and others added 2 commits August 24, 2020 06:44
We can use the Visual Studio install included with the CI image.
Saves about two minutes of preparation time.
@aminya aminya merged commit 73017bc into master Aug 24, 2020
Build, bootstrap, CI automation moved this from In progress to Done Aug 24, 2020
@DeeDeeG
Copy link
Member

DeeDeeG commented Aug 24, 2020

For the record: windows-build-tools is a convenience script that downloads Visual Studio from Microsoft and Python from Python.org. It's not expected to be much different than directly installing them yourself. It's nice to test just to be sure, but I would be very much surprised if the Visual Studio/Python you get with windows-build-tools actually behaved differently.

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

Successfully merging this pull request may close these issues.

None yet

2 participants