Skip to content

Commit

Permalink
Merge pull request #196 from lamby/use-usr-lib-not-var-lib
Browse files Browse the repository at this point in the history
Use /usr/lib, not /var/lib - the latter is for program state, not libraries or modules.
  • Loading branch information
dvirsky committed Oct 26, 2017
2 parents a1138b8 + ba08236 commit 0e6c1f4
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions Dockerfile
@@ -1,6 +1,6 @@
FROM redis:latest as builder

ENV LIBDIR /var/lib/redis/modules
ENV LIBDIR /usr/lib/redis/modules
ENV DEPS "python python-setuptools python-pip wget unzip build-essential"
# Set up a build environment
RUN set -ex;\
Expand All @@ -19,10 +19,10 @@ RUN set -ex;\

# Package the runner
FROM redis:latest
ENV LIBDIR /var/lib/redis/modules
ENV LIBDIR /usr/lib/redis/modules
WORKDIR /data
RUN set -ex;\
mkdir -p "$LIBDIR";
COPY --from=builder /src/redisearch.so "$LIBDIR"

CMD ["redis-server", "--loadmodule", "/var/lib/redis/modules/redisearch.so"]
CMD ["redis-server", "--loadmodule", "/usr/lib/redis/modules/redisearch.so"]

0 comments on commit 0e6c1f4

Please sign in to comment.