These are chat archives for MightyPork/rpw

22nd
Jun 2016
Michael Beltran
@mikkoman3_twitter
Jun 22 2016 02:01
Hi. Is it possible to set photoshop as the default editor on Mac? It seems to want some sort of executable but it treats the mac "app" as folders and doesn't allow them to be selected.
Michael Beltran
@mikkoman3_twitter
Jun 22 2016 02:11
I just now searched previous answers to this but the readme and the menu options don't talk about how to set up for Mac.
Ondřej Hruška
@MightyPork
Jun 22 2016 06:04
@mikkoman3_twitter look at closed github issues, there's this exact thing solved (more than once).
Michael Beltran
@mikkoman3_twitter
Jun 22 2016 07:36
Not quiet. I followed the instructions there and they didn't work for me
It seems spaces in the folder or app name throws rpw off
Ondřej Hruška
@MightyPork
Jun 22 2016 07:37
Not my fault you have spaces in a file name. It worked for others though
Michael Beltran
@mikkoman3_twitter
Jun 22 2016 07:38
That's how photoshop unpacks things
I was able to get around it by getting rid of the spaces. It would be nice if there was support for Mac formating
Otherwise it doesn't really support mac imo
But I got it to work so all is good
Ondřej Hruška
@MightyPork
Jun 22 2016 07:42
I don't use Mac, so can't try this, all Mac support is mostly accidental.
You can send a pull request if you find how to work around the spaces - but they shouldn't be in a file name to begin with, that's just poor choice (on adobe's part, here)
Michael Beltran
@mikkoman3_twitter
Jun 22 2016 07:44
I tried the various ways to type the spaces that mac accepts ( /Applications/Adobe\ Photoshop) ("Applications/Adobe Photoshop") etc and it works in the mac terminal but not in rpw. So far only getting rid of the spaces worked.
Michael Beltran
@mikkoman3_twitter
Jun 22 2016 07:50
I went ahead and suggested this on #86, I think it will solve that guys problem. Thanks for the help, good night.
Ondřej Hruška
@MightyPork
Jun 22 2016 07:50
yea the bug is right here
https://github.com/MightyPork/rpw/blob/master/src/net/mightypork/rpw/utils/files/DesktopApi.java#L224
it should treat "\ " nicely. Currently just splits by spaces
Michael Beltran
@mikkoman3_twitter
Jun 22 2016 07:56
I see