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

Not updating project.pbxproj? #232

Open
GrabbenD opened this issue Oct 6, 2020 · 5 comments
Open

Not updating project.pbxproj? #232

GrabbenD opened this issue Oct 6, 2020 · 5 comments

Comments

@GrabbenD
Copy link

GrabbenD commented Oct 6, 2020

The CURRENT_PROJECT_VERSION entries in ios/{project_name}.xcodeproj/project.pbxproj aren't being updated when the package is executed. Is there anything that we can do about that?

@GrabbenD
Copy link
Author

GrabbenD commented Oct 6, 2020

Found the issue. It's due to target.name === appPkg.name condition. The solution is to update "name": in the main package.json to match the prefix of the project folder in ios/ directory. E.g "name": "MyProject" if
you've got ios/MyProject.xcodeproj. It's not ideal since in our case it's not following the lowercase convention but it's okay.

if (target.name === appPkg.name) {

@stovmascript
Copy link
Owner

I guess we could change this condition to always test both properties after converting them .toLowerCase(). Do you want to make a PR?

@geraintwhite
Copy link
Contributor

This check also means it won't work for multiple targets. It updates both Info.plist files but only one of the targets in the project.pbxproj.

@geraintwhite
Copy link
Contributor

Using --legacy and setting CURRENT_PROJECT_VERSION in the project rather than the targets seems to do the trick.

@dmlitvinov
Copy link

Using --legacy and setting CURRENT_PROJECT_VERSION in the project rather than the targets seems to do the trick.

Incredible, but it works as it should!
Thank you

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

No branches or pull requests

4 participants