add docs for showing Emacs in spotlight #517

whompyjaw · 2022-10-09T02:46:56Z

Personally, I love to launch apps through Spotlight. Symlinking (ln -s) the Emacs.app to /Applications did not make the link/app to show in Spotlight (even after reboot). So, I found an example on SO that worked using Finder's Create Alias here. I figured others would like to do this :)

I looked over the Development Guidelines and I didn't see any specific rules on docs, but please let me know if you prefer any other phrasing or ordering :)

d12frosted

I like the idea, but I would add this to caveats section of the formula itself. Even more, I would provide a Apple Script one-liner that will create alias for you. If you need help with that, let me know.

The problem with this proposed documentation is that you need to know installation location, which differs from platform to platform (Apple Silicon and Intel) and from Emacs version.

d12frosted · 2022-10-17T13:09:55Z

README.org

@@ -336,6 +339,20 @@ Knows issues:
 | =--with-retro-sink-bw-icon=                   | Unknown                    | [[/icons/preview/retro-sink-bw_128.png]]                   | [[https://www.teuton.org/~ejm/emacsicon/][→ Link]]  |
 | =--with-retro-sink-icon=                      | [[https://www.teuton.org/~ejm/][→ Erik Mugele]]              | [[/icons/preview/retro-sink_128.png]]                      | [[https://www.teuton.org/~ejm/emacsicon/][→ Link]]  |
 | =--with-spacemacs-icon=                       | [[https://github.com/nashamri][→ Nasser Alshammari]]        | [[/icons/preview/spacemacs_128.png]]                       | [[https://github.com/nashamri/spacemacs-logo][→ Link]]  |
+** Spotlight Alias
+*** Apple Silicon


Aliases work the same regardless of the platform. So I would remove this distinction.

d12frosted · 2022-10-17T13:16:33Z

README.org

@@ -336,6 +339,20 @@ Knows issues:
 | =--with-retro-sink-bw-icon=                   | Unknown                    | [[/icons/preview/retro-sink-bw_128.png]]                   | [[https://www.teuton.org/~ejm/emacsicon/][→ Link]]  |
 | =--with-retro-sink-icon=                      | [[https://www.teuton.org/~ejm/][→ Erik Mugele]]              | [[/icons/preview/retro-sink_128.png]]                      | [[https://www.teuton.org/~ejm/emacsicon/][→ Link]]  |
 | =--with-spacemacs-icon=                       | [[https://github.com/nashamri][→ Nasser Alshammari]]        | [[/icons/preview/spacemacs_128.png]]                       | [[https://github.com/nashamri/spacemacs-logo][→ Link]]  |
+** Spotlight Alias
+*** Apple Silicon
+Symlinking (via ~ln -s~) ~Emacs.app~ won't be visible in Spotlight (It did not work for me even after reboot). However creating an Alias through Finder worked.


I would rephrase it to be less personalized. E.g. Spotlight doesn't pick up symlinks, so you have to create an alias, do blah-blah.

d12frosted · 2022-10-17T13:32:22Z

Just to explain a little bit more the idea I proposed.

Today we have the following caveats section:

==> Caveats
Emacs.app was installed to:
  /opt/homebrew/opt/emacs-plus@29

To link the application to default Homebrew App location:
  ln -s /opt/homebrew/opt/emacs-plus@29/Emacs.app /Applications

It is defined in each formula like this:

homebrew-emacs-plus/Formula/emacs-plus@29.rb

Lines 246 to 258 in 85c84fd

    
             def caveats 
        
               <<~EOS 
        
                 Emacs.app was installed to: 
        
                   #{prefix} 
        
                 To link the application to default Homebrew App location: 
        
                   ln -s #{prefix}/Emacs.app /Applications 
        
                 Your PATH value was injected into Emacs.app/Contents/Info.plist 
        
                 Report any issues to http://github.com/d12frosted/homebrew-emacs-plus 
        
               EOS 
        
             end

I propose to either completely remove ln -s (cause it doesn't work with Spotlight) or to add an extra note to use the following script for Spotlight:

osascript -e 'tell application "Finder" to make alias file to posix file "/opt/homebrew/opt/emacs-plus@29/Emacs.app" at POSIX file "/Applications"'

The good thing is that caveats can be constructed using some path variables:

osascript -e 'tell application "Finder" to make alias file to posix file "#{prefix}/Emacs.app" at POSIX file "/Applications"'

This section appears after you install Emacs+ and when you run brew info emacs-plus.

WDYT?

Related: d12frosted/homebrew-emacs-plus#517

aguynamedben · 2023-02-03T06:39:57Z

+1 to your solution. I ran into this snag and the osascript worked great for me! I usually use "Emacs for OS X", but tried your distro for native-comp + Emacs 29 and it's working awesome!

The symlink solution also doesn't give me the icon, which was confusing, and opens a terminal before it opens Emacs. The alias is great and I think putting it in brew info is fair if it can't be done automatically.

Thank you for working on this fantastic version of Emacs ❤️

LukeDRussell

Works for me!

whompyjaw · 2023-05-27T03:47:49Z

Just to explain a little bit more the idea I proposed.

Today we have the following caveats section:
==> Caveats
Emacs.app was installed to:
  /opt/homebrew/opt/emacs-plus@29

To link the application to default Homebrew App location:
  ln -s /opt/homebrew/opt/emacs-plus@29/Emacs.app /Applications                                                                                                                                            
It is defined in each formula like this:

homebrew-emacs-plus/Formula/emacs-plus@29.rb

Lines 246 to 258 in 85c84fd

def caveats

<<~EOS

Emacs.app was installed to:

#{prefix}

To link the application to default Homebrew App location:

ln -s #{prefix}/Emacs.app /Applications

Your PATH value was injected into Emacs.app/Contents/Info.plist

Report any issues to http://github.com/d12frosted/homebrew-emacs-plus

EOS

end

I propose to either completely remove ln -s (cause it doesn't work with Spotlight) or to add an extra note to use the following script for Spotlight:
osascript -e 'tell application "Finder" to make alias file to posix file "/opt/homebrew/opt/emacs-plus@29/Emacs.app" at POSIX file "/Applications"'
The good thing is that caveats can be constructed using some path variables:
osascript -e 'tell application "Finder" to make alias file to posix file "#{prefix}/Emacs.app" at POSIX file "/Applications"'
This section appears after you install Emacs+ and when you run brew info emacs-plus.

WDYT?

This looks great! The script worked for me as well (showed up in spotlight). I made changes per your suggestions, but I don't see any reason to include my doc request if the caveat will cover that? I also don't see why you can't just call the osascript automatically in the brew file? I am not very familiar with brew's install process, but I would think you can call this osascript.

Added osascript command

whompyjaw · 2023-05-27T04:03:33Z

Sorry that I was MIA for so long. I was working so much that I didn't really dev much in personal time (sadly work doesn't use macOS). But, hopefully this is fixed? I still see that it can't be rebased due to conflicts. I thought I resolved them, so not sure. I don't work too much with merging forks into a master branch.

d12frosted · 2023-12-21T22:24:55Z

Fixed by #642

add docs for showing Emacs in spotlight

4e31291

d12frosted reviewed Oct 17, 2022

View reviewed changes

d12frosted mentioned this pull request Nov 11, 2022

Emacs doesn't appear in Applications after installation #488

Closed

eddiegroves added a commit to eddiegroves/dotfiles that referenced this pull request Jan 19, 2023

Add note about Emacs in Spotlight

62cf086

Related: d12frosted/homebrew-emacs-plus#517

d12frosted mentioned this pull request Feb 19, 2023

Migration to brew cask #41

Closed

d12frosted mentioned this pull request Mar 6, 2023

Support for Tree-Sitter #527

Closed

LukeDRussell reviewed May 27, 2023

View reviewed changes

whompyjaw and others added 3 commits May 26, 2023 20:53

Added fixes per owner's request

d7c45bb

Added osascript command

Merge branch 'master' into add-spotlight-to-docs

f0f58b5

Added clearer instruction

fbbec3c

d12frosted mentioned this pull request Sep 26, 2023

Application is not linked out of box #618

Closed

aguynamedben mentioned this pull request Dec 14, 2023

Linker library load error on master branch (emacs-plus@28 --with-xwidgets --with-native-comp) #309

Closed

3 tasks

danking mentioned this pull request Dec 21, 2023

Fix linking how-to in formula caveats #642

Merged

d12frosted closed this Dec 21, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

add docs for showing Emacs in spotlight #517

add docs for showing Emacs in spotlight #517

whompyjaw commented Oct 9, 2022

d12frosted left a comment

d12frosted Oct 17, 2022

d12frosted Oct 17, 2022

d12frosted commented Oct 17, 2022

aguynamedben commented Feb 3, 2023

LukeDRussell left a comment

whompyjaw commented May 27, 2023

whompyjaw commented May 27, 2023

d12frosted commented Dec 21, 2023

add docs for showing Emacs in spotlight #517

add docs for showing Emacs in spotlight #517

Conversation

whompyjaw commented Oct 9, 2022

d12frosted left a comment

Choose a reason for hiding this comment

d12frosted Oct 17, 2022

Choose a reason for hiding this comment

d12frosted Oct 17, 2022

Choose a reason for hiding this comment

d12frosted commented Oct 17, 2022

aguynamedben commented Feb 3, 2023

LukeDRussell left a comment

Choose a reason for hiding this comment

whompyjaw commented May 27, 2023

whompyjaw commented May 27, 2023

d12frosted commented Dec 21, 2023