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

INT-379: Fix config so windows actually builds. #112

Merged
merged 1 commit into from
Aug 29, 2015

Conversation

imlucas
Copy link
Contributor

@imlucas imlucas commented Aug 25, 2015

Just some metadata munging needed. After running npm run release, you'll get the following files:

scout-win32-installer

cc @kangas

@kangas
Copy link
Contributor

kangas commented Aug 25, 2015

Hmm, this looks pretty much what I had on tree/fix-windows. But that wasn't sufficient for me last week, even with electron-0.30.4.

Will test ASAP.

@kangas
Copy link
Contributor

kangas commented Aug 29, 2015

Tested on Evergreen: https://evergreen.mongodb.com/task/scout_windows_64_compile_60a545e8c09a2ffcc5b7f8fc5c86467db558bf2f_15_08_29_16_12_49

It worked, but the final s3.put stage has the wrong path. I'll fix that separately.

kangas added a commit that referenced this pull request Aug 29, 2015
INT-379: Fix config so windows actually builds.
@kangas kangas merged commit f66ee26 into master Aug 29, 2015
@rueckstiess rueckstiess deleted the INT-379_windows-installer branch September 1, 2015 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants