Skip to content

Commit

Permalink
Merge pull request #429 from ios-control/gabebear-patch-2
Browse files Browse the repository at this point in the history
Remove node/npm from README installation steps (in favor of brew)

Verified that this fixes the issue we saw in  #415 ... but still not sure why npm is failing.
  • Loading branch information
gabebear committed Dec 10, 2019
2 parents 414e60f + 688f9ed commit b7b17da
Showing 1 changed file with 5 additions and 22 deletions.
27 changes: 5 additions & 22 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,42 +21,25 @@ The 1.x branch has been archived (renamed for now), all development is to be on

## Installation

ios-deploy installation is made simple using the node.js package manager. If you use [Homebrew](https://brew.sh/), install [node.js](https://nodejs.org):

```
brew install node
```

Now install ios-deploy with the [node.js](https://nodejs.org) package manager:

If you have previously installed ios-deploy via `npm`, uninstall it by running:
```
npm install -g ios-deploy
sudo npm uninstall -g ios-deploy
```

To build from source:
Install ios-deploy via [Homebrew](https://brew.sh/) by running:

```
xcodebuild
brew install ios-deploy
```

This will build `ios-deploy` into the `build/Release` folder.

## Testing

Run:

```
npm install && npm test
python -m py_compile src/scripts/*.py && xcodebuild -target ios-deploy-lib && xcodebuild test -scheme ios-deploy-tests
```

### OS X 10.11 El Capitan or greater

If you are *not* using a node version manager like [nvm](https://github.com/creationix/nvm) or [n](https://github.com/tj/n), you may have to do either of these three things below when under El Capitan:

1. Add the `--unsafe-perm=true` flag when installing ios-deploy
2. Add the `--allow-root` flag when installing ios-deploy
3. Ensure the `nobody` user has write access to `/usr/local/lib/node_modules/ios-deploy/ios-deploy`

## Usage

Usage: ios-deploy [OPTION]...
Expand Down

0 comments on commit b7b17da

Please sign in to comment.