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

Exporter Packaging and Installer [AARD-1742] #1073

Merged
merged 26 commits into from
Aug 21, 2024

Conversation

BrandonPacewic
Copy link
Member

@BrandonPacewic BrandonPacewic commented Jul 31, 2024

Now that we only have the exporter that needs to be installed locally. Previously we were installing the extension into the user directory for addins, we are now using the ApplicationPlugins folder which is standard Autodesk wide. It also has the advantage of not being able to be changed by the user. This, however, also means that the exporter will no longer have permissions within it's installed directory so we need to redirect things into the Application Support path. Hence the changes within the exporter that would appear to not be directly related to packaging.

Essentially this installer works exactly how installers work from the Fusion Addin Store work.

  • For MacOS, new .pkg installer.
  • For Windows, new .exe installer.

When running the exporter from the installer logs can be found in:

  • Mac: ~/.config/Autodesk/Synthesis/Logs
  • Win: %appdata%/Autodesk/Synthesis/Logs

Otherwise the log directory will be the same:

  • exporter/SynthesisFusionExporter/logs/

This was done as we don't always have write permissions in the ApplicationPlugins directory.

Some other things:

  • Install script will now mark the exporter with a version so you can tell them apart.
  • Install script has an additional step to uninstall previous versions of the exporter.

Warning

Switching between running the version of the exporter within ApplicationPlugins and another manually added path will cause problems.

Restart Fusion in-between runs of the exporter within different directories to avoid caching issues.

JIRA Issue

@BrandonPacewic BrandonPacewic self-assigned this Jul 31, 2024
@BrandonPacewic BrandonPacewic changed the title Exporter Packaging and Installer [[AARD-1742](https://jira.autodesk.com/browse/AARD-1742)] Exporter Packaging and Installer [AARD-1742] Jul 31, 2024
@BrandonPacewic BrandonPacewic changed the title Exporter Packaging and Installer [AARD-1742] Exporter Packaging and Installer [AARD-1742] Jul 31, 2024
@BrandonPacewic BrandonPacewic changed the title Exporter Packaging and Installer [AARD-1742] Exporter Packaging and Installer [AARD-1742] Aug 6, 2024
* dev: (125 commits)
  Forgot to add the button change :(
  merge fixes
  Import formatting fixes
  Ensure browser cache key
  Added `BaseException` catch for google `VersionError`
  Version detection for protobuf fix
  Removed commented code
  Increased number of cascades
  Rename jolt submodule
  Added Hunters jolt submodule
  Close config panel when you choose to spawn a new robot or field
  Delete, create, and modify input schemes in a way that is easy and integrated with the config panel
  Fixed scrollbar
  Input config rework, cleanup, and comments
  Formatting
  Fixed
  Shorten formatter line lengths
  Versions work differently?
  More verbose isort messages
  Completely functional panel
  ...
@BrandonPacewic BrandonPacewic marked this pull request as ready for review August 19, 2024 21:16
@BrandonPacewic BrandonPacewic requested review from HunterBarclay and a team as code owners August 19, 2024 21:16
@BrandonPacewic BrandonPacewic requested review from PepperLola, azaleacolburn, a-crowell and LucaHaverty and removed request for a team August 19, 2024 21:16
Copy link
Contributor

@azaleacolburn azaleacolburn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works perfectly for me!

Copy link
Contributor

@PepperLola PepperLola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works well 👎

Copy link
Member Author

@BrandonPacewic BrandonPacewic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Should have someone test on Windows before we merge.

@HunterBarclay HunterBarclay merged commit 1473f05 into dev Aug 21, 2024
13 checks passed
@HunterBarclay HunterBarclay deleted the branp/1742/exporter-packaging branch August 21, 2024 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants