Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed workflows and missing header in twistSplineNode #24

Merged
merged 14 commits into from
Nov 29, 2023

Conversation

mdilena
Copy link
Contributor

@mdilena mdilena commented Nov 28, 2023

Types of Changes

  • Bugfix (change that fixes an issue)
  • New Feature (change that adds functionality)
  • Documentation Update (if none of the other choices apply)

Proposed Changes

  • Added a missing include that was preventing from compiling the plug-in
  • Reworked github workflows as anything but Linux builds where hanging or failing. Turning on windows-latest and macos-latest resolved the hanging, while on macos the 7z command to extract the devkit wasn't working anymore, so the dmg is now getting mounted and the build points to the contents of that volume directly

EDIT:
Forgot to mention, also split the macos build into x86_64 and arm_64 to compile against new Apple chips in Maya 2024.

@tbttfox
Copy link
Member

tbttfox commented Nov 29, 2023

Awesome! Thanks for this. Especially for the mac stuff. I have zero clue when it comes to that.

I'm going to merge, but I'll hold off on bumping the version because I think there's a bit more work to do for the module setup... and I also have no idea how to differentiate between x86 and arm using a .mod file 🤷

@tbttfox tbttfox merged commit becad5a into blurstudio:master Nov 29, 2023
12 checks passed
@mdilena
Copy link
Contributor Author

mdilena commented Nov 30, 2023

Completely forgot about the mod file, tried to look up some info but can't really find if there's a way to specify the architecture there, although it doesn't seem so. Will do some more experiment with the actions as I'm hoping it can be compiled with compatibility to both in the same bundle file!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants