svalinn / Cubit-plugin

Plugins and command extensions for Coreform Cubit
BSD 3-Clause "New" or "Revised" License
17 stars 14 forks source link

no bin folder for mac? #110

Closed bam241 closed 3 years ago

bam241 commented 3 years ago

trying to fix the missing bin folder for trelis17.1

bam241 commented 3 years ago

I tested it locally, plugin is now loading for MacOS 17.1

bam241 commented 3 years ago

@lewisgross1296 have you been able to test the plugin from there ?

lewisgross1296 commented 3 years ago

After re-installing Trelis-17.1, patching it, and moving the un-tar'd plugin to /Applications/Trelis-17.1.app/Contents/MacOS/plugins, I was able to open Trelis and view the message

-- DAGMC export command available.
-- iGeom_test command available.
-- MCNP import command available.
Finished Command: set node constraint smart
Finished Command: undo on

After this, I entered help dagmc and it output the help information

export dagmc <filename> [faceting_tolerance <faceting tolerance>] [length_tolerance <length tolerance>] [normal_tolerance <normal tolerance>] [make_watertight] [verbose] [fatal_on_curves]

gonuke commented 3 years ago

What "patch" are you applying to Trelis 17.1?

bam241 commented 3 years ago

basically, when the SDK is present, there is a bin symbolic link that point to MacOS.

the RPATH the lib uses bin and not MacOS. What I do in this is reset the RPATH to use MacOS instead of bin. it is only required for 17.1 because that the only version we support that does not ship with the SDK (so with the bin symbolic link)

lewisgross1296 commented 3 years ago

@bam sent me a patch on box that I had to unpack and move to /Applications/Trelis-17.1.app/Contents/MacOS. It was discussed in #svalinn-support on slack on Monday July 26. I think without it I couldn't log into the cnerg license server, so it probably isn't something that everyone needs to patch, just people using our license server.

bam241 commented 3 years ago

What "patch" are you applying to Trelis 17.1?

I think he is refering the lib patch required to use an updated license server, this has nothing to do with the plugin

gonuke commented 3 years ago

Thanks @bam241