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

fix(xcode): Plist vars are always expanded #1403

Merged
merged 2 commits into from
Nov 24, 2022

Conversation

krystofwoldrich
Copy link
Member

@krystofwoldrich krystofwoldrich commented Nov 23, 2022

In #1311 expanding Plist var when reading from the path was removed. I believe this was a mistake as this change is causing issues in react native.

Fixes getsentry/sentry-react-native#2581
Fixes #1395

src/utils/xcode.rs Outdated Show resolved Hide resolved
src/utils/xcode.rs Outdated Show resolved Hide resolved
@kamilogorek
Copy link
Contributor

Nice find, thanks!

Co-authored-by: Kamil Ogórek <kamil.ogorek@gmail.com>
@krystofwoldrich
Copy link
Member Author

@kamilogorek Thanks for the suggestions and review.

@krystofwoldrich krystofwoldrich merged commit 634b11c into master Nov 24, 2022
@krystofwoldrich krystofwoldrich deleted the xcode-always-expand-xcode-vars branch November 24, 2022 08:45
@krystofwoldrich krystofwoldrich restored the xcode-always-expand-xcode-vars branch November 24, 2022 08:47
@kamilogorek kamilogorek deleted the xcode-always-expand-xcode-vars branch November 24, 2022 08:48
@kamilogorek kamilogorek restored the xcode-always-expand-xcode-vars branch November 24, 2022 08:48
@kamilogorek
Copy link
Contributor

Ugh, you didn't squash the PR :P

@krystofwoldrich
Copy link
Member Author

Yeah, I've noticed too, but late. 😕

@kamilogorek kamilogorek deleted the xcode-always-expand-xcode-vars branch November 24, 2022 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants