Skip to content

Fix: Add embedded Info.plist for location permissions (Fix #50, #51) #53

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

Merged
merged 1 commit into from
Apr 11, 2025

Conversation

yclw
Copy link
Contributor

@yclw yclw commented Apr 11, 2025

- Add required Info.plist with location permissions
- Configure linker sectcreate in Package.swift
- Remove temporary script workaround
- Closes fulldecent#50, fulldecent#51
- Contributes to fulldecent#30 (versioning support)
- Supersedes fulldecent#52
@yclw
Copy link
Contributor Author

yclw commented Apr 11, 2025

Troubleshooting Discovery
So here's what I found: You can actually strip down the Info.plist to just the CFBundleIdentifier like this:

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
	<key>CFBundleIdentifier</key>
	<string>com.fulldecent.CoreLocationCLI</string>
</dict>
</plist>

...and macOS will still show the location permission prompt! 🎉 (Probably because of how macOS handles permissions under the hood.)

BUT - I decided to keep all the other keys in the final version just to be safe.

@fulldecent fulldecent merged commit 1c4d038 into fulldecent:main Apr 11, 2025
@fulldecent
Copy link
Owner

Thank you, looks good

@yclw yclw deleted the fix-50-51-permanent-solution branch April 12, 2025 04:22
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.

Can't grant location permissions on MacOs 14 (Sonoma)
2 participants