Pritam Baral
7627e0b0f0
Minor fix + Remove last references to /tmp
2014-05-19 23:57:48 +05:30
Corey Johnson
dbe15f7dda
Merge pull request #2142 from aureliojargas/patch-1
...
Remove unused 's' option from getopts
2014-05-10 15:53:21 -07:00
Aurelio Jargas
2b73dff0f4
Use $(...) for command substitution, not backticks
...
Currently both \`...\` and `$(...)` are used for command substitution. It's better to chose just one and stick to it, to avoid confusion.
Some arguments favoring `$(...)`:
* **Easier to read.** The backtick is easily confused with apostrophes and quotes.
* **Easier to nest.** Nesting backticks commands is an escaping nightmare. Using parenthesis is natural: `$(foo $(bar))`
2014-05-10 09:15:53 -03:00
Aurelio Jargas
adf0ff0a67
Remove unused 's' option from getopts
...
This option is not handled by the next `case` command.
2014-05-10 08:41:30 -03:00
Thomas Buckley-Houston
1ea54f8c92
🐧 If atom.sh has non-zero exit show errors captured by nohup. Refers to #1814
2014-05-06 17:21:01 +01:00
Cheng Zhao
ae1d3b1e80
🐧 Search atom from installed position.
2014-03-25 15:40:43 +08:00
Cheng Zhao
5151faea3d
Use unix style install.
2014-03-25 15:40:43 +08:00
Cheng Zhao
19ab197bc1
🐧 Make atom.sh work.
2014-03-18 16:27:43 +08:00
Corey Johnson
96e96c3c7f
Merge pull request #1757 from atom/cj-shhhhh
...
Don't show error messages for version and help cli options
2014-03-13 17:42:19 -07:00
probablycorey
15c51b4417
Redirect help and version stderr to /dev/null
...
Closes #1580 based on @zcbenz's suggestion.
2014-03-13 15:00:24 -07:00
Kevin Sawicki
31ecd39f08
Don't echo app location
2014-03-13 12:41:08 -07:00
probablycorey
1cbaf8d1f5
Added comments because shell scripts can be confusing
2014-03-13 11:14:26 -07:00
probablycorey
2fd71f2fa8
Use mdfind to locate Atom.app if it isn't in /Applications
2014-03-13 10:57:55 -07:00
probablycorey
54907455fe
Add ~/Desktop
2014-03-12 17:44:00 -07:00
probablycorey
ba8ab21a0d
Search for the Atom app in the usual locations when using the atom cli
...
Sorta closes #1595
2014-03-12 17:10:36 -07:00
Kevin Sawicki
bd3dc453ef
Wrap command line arguments in quotes
...
Closes #1639
2014-03-11 13:35:19 -07:00
Kevin Sawicki
e8c399428c
Set ATOM_PATH env var when launching specs
2013-10-14 10:33:37 -07:00
Matt Colyer
64e8c978e4
Return proper return code from atom.sh
...
Also improve the test failure message returned by apm.
2013-10-07 14:28:13 -07:00
probablycorey
b92b6e689d
Rearrange command line option string
...
I apologize, but I could not resist rearranging the command line
options to spell "WTF VHS"
2013-10-03 10:27:26 -07:00
Matt Colyer
fad7c9586c
Update older (and incorrect) flag in atom.sh
2013-10-02 13:10:10 -07:00
Matt Colyer
62b7599bea
Add spec-directory command line option
2013-10-02 12:17:42 -07:00
Kevin Sawicki
c27103db8a
Expect output when -t/--test flag is specified
2013-08-28 12:54:16 -07:00
probablycorey
204de3ac24
Support long version for foreground command
2013-08-21 16:19:37 -07:00
probablycorey
986a9ce0a0
Add cli option to keep the browser process running in the foreground
2013-08-21 16:19:37 -07:00
Matt Colyer
82882624ce
Handle atom standard output from atom.sh
2013-08-09 10:23:05 -07:00
Kevin Sawicki
718175eb34
Make atom.sh executable
2013-05-15 10:29:27 -07:00
probablycorey
6ee24cc50e
💄
2013-04-29 12:10:40 -07:00
probablycorey
599328cb9c
Better error message when Atom.app is not found
2013-04-17 16:26:13 -07:00
probablycorey
9b8abe3552
If Atom.app doesn't exist, wait 5 seconds and try again.
...
Closes #351
When Sparkle updates Atom, it deletes the app and replaces it. This
causes the `atom` cli to fail when called within this brief time
period. This gives `atom` a five second grace period if it is not
found.
2013-04-17 16:25:42 -07:00
Corey Johnson & Kevin Sawicki
b874f95eb1
Launch Atom with full path to .app
2013-02-25 16:11:13 -08:00
Kevin Sawicki
d35c871e17
Install atom executable during window.startup()
...
Previously this was done during `rake install`.
Also default to `~/github/atom` as the default resource path
when no `--resource-path` argument is specified. This argument
will now be required when running in dev mode if the repository
is not at the default location.
Closes #300
2013-02-22 21:19:34 -08:00