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

fixed failure of removing framework sometimes #19

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

chenxiaoyao
Copy link

Sometimes, path/name of framework may not contain double quotation in project.pbxproj, when remove these framework, it will fail

KristianDD pushed a commit to NativeScript/node-xcode that referenced this pull request Jan 30, 2019
from cordova-common, with some changes

Testing of this package on old Node.js versions is included,
since some non-Cordova users may still expect this to work.

nvm install --latest-npm is used for Node.js 0.8, as long as needed;
thanks for guidance:
* https://stackoverflow.com/questions/9755841/how-can-i-change-the-version-of-npm-using-nvm/47519162#47519162
* https://github.com/browserify/browserify/blob/master/.travis.yml

Co-authored-by: Christopher J. Brody <chris.brody@gmail.com>
Co-authored-by: Darryl Pogue <darryl@dpogue.ca>
Co-authored-by: Audrey So <audreyso@apache.org>
Co-authored-by: filmaj <maj.fil@gmail.com>
Co-authored-by: Steve Gill <stevengill97@gmail.com>
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.

1 participant