added ICON

README.md

@@ -52,7 +52,7 @@ Windows note: Some people are reporting problems with windows 11.
 
 # Old/broken/spurious binaries
 
-* **Windows* *: [samplebrain_0.18_win.zip](https://static.thentrythis.org/samplebrain/samplebrain_0.18_win.zip)
+* **Windows**: [samplebrain_0.18_win.zip](https://static.thentrythis.org/samplebrain/samplebrain_0.18_win.zip)
 * **Mac (intel)**: [samplebrain_0.18_macintel.zip](https://static.thentrythis.org/samplebrain/samplebrain_0.18_macintel.zip)
 * **Mac (m1)**: [samplebrain_0.18_m1_v2.dmg](https://static.thentrythis.org/samplebrain/samplebrain_0.18_m1_v2.dmg) 
 
@@ -72,7 +72,7 @@ Install libraries for the sample engine (use brew on mac, MinGW on win):
 
     $ sudo apt install libsndfile1-dev portaudio19-dev liblo-dev libfftw3-dev
 
-Install dependancies for the interface:
+Install dependencies for the interface:
 
     $ sudo apt install build-essential qtcreator qt5-default
 
@@ -88,9 +88,9 @@ Build & run it:
 ## Mac
 Install libraries for sample engine:
 
-    $ brew install fftw portaudio liblo
+    $ brew install fftw portaudio liblo libsndfile
 
-Install dependancies for the interface:
+Install dependencies for the interface:
 
     $ brew install qt
     $ brew link qt
@@ -108,22 +108,24 @@ Build & run it:
 
 To make a mac app bundle:
 
-Run `macdeployqt` to copy all dependencies inside the app.
+Run `macdeployqt` which copies all dependencies inside the app.
-
-Create Mac app bundle:
 
     $ cd build
     $ macdeployqt
 
-Copy desktop/samplebrain.icns (the icon) to the Resources directory in the bundle.
+If the icon is not visible, you might need to copy desktop/samplebrain.icns (the icon) to the Resources directory in the app bundle.
 
     $ cp ../desktop/samplebrain.icns samplebrain.app/Contents/Resources
 
-Edit Info.plist to add samplebrain.icns to CFBundleIconFile. Key `CFBundleIconFile` should match:
+Then edit Info.plist to add samplebrain.icns to CFBundleIconFile. Key `CFBundleIconFile` should match:
 
 	<key>CFBundleIconFile</key>
 	<string>samplebrain.icns</string>
 
+You might also need to resign the app bundle after making any changes
+
+    $ codesign --force --deep --sign - samplebrain.app
+
 ## What's here
 
 1. brain:

samplebrain.pro

@@ -49,7 +49,8 @@ LIBS += -L.. -L/usr/local/lib -L/opt/homebrew/lib -lportaudio -lfftw3 -lsndfile
 QMAKE_CXXFLAGS += -O3 -Wall -Wno-unused -std=c++11
 
 # assets
-RESOURCES     = app/samplebrain.qrc
+RESOURCES = app/samplebrain.qrc
+ICON      = desktop/samplebrain.icns
 
 PREFIX = $$(PREFIX)
 isEmpty(PREFIX) {