docker-android icon indicating copy to clipboard operation
docker-android copied to clipboard

WATCHMAN_VERSION not used

Open n4n0lix opened this issue 5 years ago • 3 comments

In the Dockerfile there is a line ARG WATCHMAN_VERSION=4.9.0 which is never used.

In the same context, when for example watchman watch-del-all is executed in the container it results in watchman: command not found. But I can still build react-native apps using this docker image.

n4n0lix avatar May 13 '19 08:05 n4n0lix

We extended this image to add watchman and a few other features: ruby, fastlane, sudo, and procps (needed for this error). If one of the maintainers would like these added to the core image, I can look into making a PR.

Here's the watchman part:

FROM reactnativecommunity/react-native-android

ENV WATCHMAN_VERSION=4.9.0

# Install Watchman
RUN apt-get update \
  && apt-get install -y python python-dev pkg-config libssl-dev autoconf automake libtool \
  && cd /tmp \
  && git clone https://github.com/facebook/watchman.git \
  && cd watchman \
  && git checkout v${WATCHMAN_VERSION} \
  && ./autogen.sh \
  && ./configure \
  && make \
  && make install \
  && cd $HOME \
  && rm -rf /tmp/watchman

michaelgmcd avatar May 13 '19 17:05 michaelgmcd

@michaelgmcd Feel free to send a PR.

gengjiawen avatar May 16 '19 11:05 gengjiawen

Sounds good. I'll get to this when time permits!

michaelgmcd avatar May 16 '19 17:05 michaelgmcd