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

Docker container fails due to npm error #831

Closed
10 tasks
michaeltintiuc opened this issue Jan 31, 2020 · 0 comments · Fixed by #832
Closed
10 tasks

Docker container fails due to npm error #831

michaeltintiuc opened this issue Jan 31, 2020 · 0 comments · Fixed by #832

Comments

@michaeltintiuc
Copy link
Contributor

The Issue

Steps to reproduce

  1. Clone lbry-android repo
  2. Follow DOCKER-DEV.md instructions
  3. Run lbry-android docker-build
  4. npm install -g ... errors out

Expected behaviour

Container should be created successfully

Actual behaviour

npm throws error:

npm ERR! Linux 5.3.0-7625-generic
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "install" "-g" "yarn" "react-native-cli"
npm ERR! node v8.10.0
npm ERR! npm  v3.5.2
npm ERR! code EMISSINGARG
...
The command '/bin/sh -c dpkg --add-architecture i386 &&     apt-get -y update &&     apt-get install -y       curl ca-certificates software-properties-common gpg-agent wget       python3.7 python3.7-dev python3-pip python2.7 python2.7-dev python3.7-venv       python-pip zlib1g-dev m4 zlib1g:i386 libc6-dev-i386 gawk nodejs npm unzip openjdk-8-jdk       autoconf autogen automake libtool libffi-dev build-essential       ccache git libncurses5:i386 libstdc++6:i386       libgtk2.0-0:i386 libpangox-1.0-0:i386 libpangoxft-1.0-0:i386 libidn11:i386 &&     npm install -g yarn react-native-cli &&     pip2 install --upgrade cython setuptools &&     pip2 install git+https://github.com/lbryio/buildozer.git@master &&     ln -s /src/scripts/build-docker.sh /usr/local/bin/build &&     adduser lbry-android --gecos GECOS --shell /bin/bash --disabled-password --home /home/lbry-android &&     mkdir /home/lbry-android/.npm-packages &&     echo "prefix=/home/lbry-android/.npm-packages" > /home/lbry-android/.npmrc &&     chown -R lbry-android:lbry-android /home/lbry-android &&     mkdir /src &&     chown lbry-android:lbry-android /src &&     mkdir /dist &&     chown lbry-android:lbry-android /dist' returned a non-zero code: 1

System Configuration

  • Device: Docker
  • Android version: N/A

Anything Else

Seems like the older 3.5.2 npm version to causing this as per: npm/cli#681
Updating to latest npm fixes it.

Screenshots

Internal Use

Acceptance Criteria

Definition of Done

  • Tested against acceptance criteria
  • Tested against the assumptions of the user story
  • The project builds without errors
  • Unit tests are written and passing
  • Tests on devices/browsers listed in the issue have passed
  • QA performed & issues resolved
  • Refactoring completed
  • Any configuration or build changes documented
  • Documentation updated
  • Peer Code Review performed
@michaeltintiuc michaeltintiuc mentioned this issue Jan 31, 2020
9 tasks
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 a pull request may close this issue.

1 participant