That is what I use.
Additionally in my .bashrc I have:
EMACS="/Applications/Emacs.app/Contents/MacOS/Emacs"
EMACSPOS="-g 140x66+655+23 --fullheight"
e() { ( $EMACS $EMACSPOS "$@" & ) }
I also have it in the dock, but sometimes environment variables are not set up properly.
For this case, I create a plist file ~/.MacOSX/environment.plist and can add stuff to it like this:
add_plist PATH "$PATH"
where:
add_plist () {
/usr/libexec/PlistBuddy -c 'Delete :'"$1" ~/.MacOSX/environment.plist >/dev/null
/usr/libexec/PlistBuddy -c 'Add :'"$1"' string "'"$2"'"' ~/.MacOSX/environment.plist >/dev/null
}