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

[3.x] Add GDNative Framework support for macOS, add Unix symlink API. #46860

Merged

Conversation

bruvzg
Copy link
Member

@bruvzg bruvzg commented Mar 10, 2021

Adds ability to select, load and export (both .dmg and .zip) macOS ".frameworks" as GDNative libraries.
Adds minimal symlink API to DirAccess (frameworks usually contain multiple relative symlinks).

Framework export won't work from Windows (If there are symlinks in the framework). NTFS have support for symlinks, but I do not see any way to get relative symlink target path, all functions seems to return full path. And creating symlinks seems to require privilege elevation. Not sure what to do with it (currently it will print a warning).

Same as #46866 for 3.2.

@bruvzg bruvzg added this to the 3.2 milestone Mar 10, 2021
@akien-mga akien-mga requested a review from a team March 10, 2021 11:12
@bruvzg bruvzg marked this pull request as ready for review March 10, 2021 12:09
@bruvzg bruvzg requested review from a team as code owners March 10, 2021 12:09
Base automatically changed from 3.2 to 3.x March 16, 2021 11:11
@aaronfranke aaronfranke modified the milestones: 3.2, 3.3 Mar 16, 2021
@bruvzg bruvzg changed the title [3.2] Add GDNative Framework support for macOS, add Unix symlink API. [3.3] Add GDNative Framework support for macOS, add Unix symlink API. Mar 16, 2021
@akien-mga akien-mga modified the milestones: 3.3, 3.4 Mar 26, 2021
@akien-mga akien-mga changed the title [3.3] Add GDNative Framework support for macOS, add Unix symlink API. [3.x] Add GDNative Framework support for macOS, add Unix symlink API. Mar 26, 2021
@bruvzg bruvzg force-pushed the symlinks_and_macos_gdn_framework_export branch 2 times, most recently from 9b847c7 to a68e8f8 Compare May 6, 2021 20:31
@bruvzg bruvzg force-pushed the symlinks_and_macos_gdn_framework_export branch from a68e8f8 to 15e62c3 Compare May 22, 2021 14:36
@bruvzg bruvzg marked this pull request as draft May 22, 2021 14:53
@bruvzg bruvzg force-pushed the symlinks_and_macos_gdn_framework_export branch 2 times, most recently from ba5a03e to 688158e Compare May 24, 2021 07:08
@bruvzg bruvzg marked this pull request as ready for review May 24, 2021 07:59
@bruvzg bruvzg force-pushed the symlinks_and_macos_gdn_framework_export branch from 688158e to 53f05c9 Compare May 24, 2021 09:50
@akien-mga akien-mga merged commit b94b09c into godotengine:3.x May 24, 2021
@akien-mga
Copy link
Member

Thanks!

@bruvzg bruvzg deleted the symlinks_and_macos_gdn_framework_export branch November 2, 2021 21:57
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.

3 participants