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

WATCHMAN_VERSION not used #43

Open
n4n0lix opened this issue May 13, 2019 · 3 comments
Open

WATCHMAN_VERSION not used #43

n4n0lix opened this issue May 13, 2019 · 3 comments
Labels
enhancement New feature or request

Comments

@n4n0lix
Copy link

n4n0lix commented May 13, 2019

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.

@michaelgmcd
Copy link

michaelgmcd commented May 13, 2019

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

@gengjiawen
Copy link
Member

@michaelgmcd Feel free to send a PR.

@gengjiawen gengjiawen added the enhancement New feature or request label May 16, 2019
@michaelgmcd
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants