Merge pull request #98 from HansMuller/fix-contribute

Updated CONTRIBUTING.md

Author: Hans Muller

CONTRIBUTING.md

@@ -11,6 +11,8 @@ To get the code:
  * Fork https://github.com/domokit/sky_engine into your own GitHub account.
  * [Download depot_tools](http://www.chromium.org/developers/how-tos/install-depot-tools)
    and make sure it is in your path.
+ * If you haven't configured your machine with an SSH key that's shared by github then
+   follow the directions here: https://help.github.com/articles/generating-ssh-keys/.
  * Create a `.gclient` file in an empty directory with the following contents:
 
 ```
@@ -27,7 +29,7 @@ solutions = [
 target_os = ["android"]
 ```
 
- * `svn ls https://dart.googlecode.com/` and accept the certificate permanently.
+ * If you're on a Mac then: `svn ls https://dart.googlecode.com/` and accept the certificate permanently.
  * `gclient sync`
  * `cd src`
  * `git remote add upstream [email protected]:domokit/sky_engine.git`

build/symlink.py

@@ -2,26 +2,19 @@
 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
-
 """Make a symlink and optionally touch a file (to handle dependencies)."""
-
-
 import errno
 import optparse
 import os.path
 import shutil
 import sys
-
-
 def Main(argv):
   parser = optparse.OptionParser()
   parser.add_option('-f', '--force', action='store_true')
   parser.add_option('--touch')
-
   options, args = parser.parse_args(argv[1:])
   if len(args) < 2:
     parser.error('at least two arguments required.')
-
   target = args[-1]
   sources = args[:-1]
   for s in sources:
@@ -39,12 +32,8 @@ def Main(argv):
         os.symlink(s, t)
       else:
         raise
-
-
   if options.touch:
     with open(options.touch, 'w') as f:
       pass
-
-
 if __name__ == '__main__':
   sys.exit(Main(sys.argv))