Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Windows Support #1035

Merged
merged 69 commits into from Oct 29, 2013
Merged

Windows Support #1035

merged 69 commits into from Oct 29, 2013

Conversation

mcolyer
Copy link
Contributor

@mcolyer mcolyer commented Oct 25, 2013

🚨 The staging area for windows support. 🚨

Issues

  • atom/apm/src/auth.coffee "security" doesn't exist on windows.
  • grunt-coffeelint/node_modules/coffeelint/package.json: "install": "[ -e lib/commandline.js ] || npm run compile",
    You can't use bash in your install script.
  • Convert backslashes to forward slashes (git icon doesn't appear in tree-view)
  • Fuzzy finder doesn't load
  • Crashes after several seconds when autocomplete, archive-view and bookmarks are loaded (but not each package independently.

Development Environment Notes

  • Install Visual C++ 2010 Express - English from Microsoft
  • Install the latest node 0.10.x from http://nodejs.org, 32bit
  • Install the latest python 2.7.x from http://python.org, 32 bit
  • Install Github for Windows
  • Add C:\Python27;C:\Program Files\nodejs;C:\Users\IEUser\Documents\GitHub\atom\node_modules\ to your PATH
  • Set ATOM_ACCESS_TOKEN to your oauth2 credentials (run security -q find-generic-password -ws 'GitHub API Token' on OSX to get your credentials).
  • Launch Github for Windows, sign in, clone Atom.
  • Use the git shell and cd into your atom checkout.
  • Run script/bootstrap

@kevinsawicki
Copy link
Contributor

@mcolyer I see the same crash as you on my vm when activating all the packages.

The weird thing though is that the packages activate just fine if I activate one package per second using setTimeout.

@kevinsawicki
Copy link
Contributor

@mcolyer This branch is 📗 now, what do you think about merging what we have here and moving the remaining todos to an issue or new pr?

@mcolyer
Copy link
Contributor Author

mcolyer commented Oct 29, 2013

I'm down with that (merging now). I've got a few ideas around what's left so I'll make the appropriate tickets also.

mcolyer added a commit that referenced this pull request Oct 29, 2013
@mcolyer mcolyer merged commit a49fd56 into master Oct 29, 2013
@mcolyer mcolyer deleted the win-bootstrap-native-modules branch October 29, 2013 16:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants