search

wulkano / Aperture

Record the screen on macOS
MIT License
1.23k stars 111 forks source link

Rename the Swift binary #34

Closed albinekb closed 7 years ago

albinekb commented 7 years ago

This PR changes the main binary to aperture so that you know what process this is in your acitivity monitor, currently it was just "main", which is a bit confusing 😄

See here for an example (before): kapture 2017-05-30 at 2 55 54

~Also, seems like the processes aren't exited correctly? 🤔 when playing around, i had 10's of main processes in my activity monitor 😕~

see #35 👍

albinekb commented 7 years ago

I tried using the child_process argv0 option, that made it show up correctly in htop, but not in Activity Monitor.

matheuss commented 7 years ago

As far as I recall, the main is mandatory 🤔 I think it's for when you run it as a script (which I used to do when developing). Will download Xcode etc and test it 👍

albinekb commented 7 years ago

another option is to symlink main to aperture 🤔

sindresorhus commented 7 years ago

As far as I recall, the main is mandatory

I don't think it is.