Skip to content

Partial Mac OS Support - #98

Merged
Josh Wittner (jwittner) merged 8 commits into
microsoft:developfrom
Ziugy:develop
Apr 16, 2018
Merged

Partial Mac OS Support#98
Josh Wittner (jwittner) merged 8 commits into
microsoft:developfrom
Ziugy:develop

Conversation

@Ziugy

Copy link
Copy Markdown
Member

Addresses issues with running on Mac OS - #41

  • Get-UnitySetupInstance
  • Get-UnityProjectInstance
  • Start-UnityEditor

Still to do:

  • Installing Unity version onto the Mac machine. (Find-UnitySetupInstaller)

@jwittner

Copy link
Copy Markdown
Member

Robert Onulak (@Ziugy) Bah, looks like my changes added conflicts. You'll have to merge develop into your branch or rebase.

@Ziugy

Copy link
Copy Markdown
Member Author

Josh Wittner (@jwittner) Resolved!

@jwittner Josh Wittner (jwittner) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's drop the 'setup' component like we discussed and clean up our platform testing.

… unsupported. Drops 'Setup' in favor of just stating 'Windows' on Windows platform. Updates README.md and -Examples to match with Setup changes.
@Ziugy

Copy link
Copy Markdown
Member Author

Dropped Setup in favor of the OS running the script. Cleaned up platform through Get-OperatingSystem and the [OperatingSystem] enumeration. Added in exception handling in functions where functionality isn't implemented yet.

@jwittner
Josh Wittner (jwittner) merged commit d3dae74 into microsoft:develop Apr 16, 2018
@jwittner

Copy link
Copy Markdown
Member

Tested on Windows and Linux and got expected results. Great work! Can't wait for full support. ;)

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