AcademySoftwareFoundation / MaterialX

MaterialX is an open standard for the exchange of rich material and look-development content across applications and renderers.
http://www.materialx.org/
Apache License 2.0
1.86k stars 352 forks source link

Instructions for running MaterialXGraphEditor on Mac #1948

Open hybridherbst opened 3 months ago

hybridherbst commented 3 months ago

I followed the instructions to download the pre-built MaterialX release with graph editor and viewer.

However, the instructions page does only contain info on how to build it, not how to run it. Usual start attempts with python3 bin/MaterialXGraphEditor or ./MaterialXGraphEditor fail.

How do I run the pre-built version on Mac? :)

ld-kerley commented 3 months ago

I just downloaded the pre-built macOS zip - and it looks like the permissions are not set correctly on the executables.

if you open a terminal and change directory in to the root of where you unzip the archive and run

> chmod 775 bin/MaterialX*

Then I was able to run both tools.

I did also hit code security issue, and had to bypass the macOS security to actually launch the apps. I believe these binaries are not properly code signed.

jstone-lucasfilm commented 3 months ago

We're very open to ideas on improving the automated GitHub Actions step that generates our pre-built binaries for all platforms, and here's where it's currently defined:

https://github.com/AcademySoftwareFoundation/MaterialX/blob/main/.github/workflows/main.yml#L275

jstone-lucasfilm commented 3 months ago

Here's some earlier discussion of this issue and a potential path forward, if anyone has the bandwidth to take this on:

https://github.com/AcademySoftwareFoundation/MaterialX/issues/847

hybridherbst commented 3 months ago

Thank you @ld-kerley – setting the permissions on the files to 755 fixed the issue. I think one part that could be improved is the documentation – even with that being a non-ideal state, some information on the installer being borked and the required extra steps would be good :)