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

[Packager] Auto populate remote packager in info.plist #1307

Closed
wants to merge 1 commit into from
Closed

[Packager] Auto populate remote packager in info.plist #1307

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented May 15, 2015

Feature suggested by @frantic in #548. Adds a run script build phase to the Xcode project that automatically populates the ReactServer value in the info.plist file so that remote debugging "just works". It only populates the value if the user is building an armv* binary in debug mode so you won't accidentally populate the value when do a release build.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 15, 2015
@frantic
Copy link
Contributor

frantic commented May 15, 2015

🆒 ! Seems pretty easy. Could you please extract it into a shell script and then call it from Xcode? This way it's easier to test and review.

@brentvatne
Copy link
Collaborator

@tobyhughes - are you still interested in working on this?

@brentvatne brentvatne changed the title Auto populate remote packager in info.plist [Packager] Auto populate remote packager in info.plist Jun 1, 2015
@satya164
Copy link
Contributor

@rebeccahughes Any updates on this?

@facebook-github-bot
Copy link
Contributor

@rebeccahughes updated the pull request.

@satya164
Copy link
Contributor

Closing this. Let's reopen if you are interested to work on this again.

@satya164 satya164 closed this Dec 22, 2015
ayushjainrksh pushed a commit to MLH-Fellowship/react-native that referenced this pull request Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants