Merge pull request #52 from ynqa/dockerfile

Update Dockerfile

Dockerfile

@@ -1,20 +1,13 @@
-## Build stage
-FROM rust:1.79-alpine3.20 as builder
-
-RUN rustup target add x86_64-unknown-linux-musl
-
-RUN apk add musl-dev
+# Build stage
+FROM rust:1.80.0-slim-bookworm as builder
 
 WORKDIR /jnv
-
 COPY . /jnv
+RUN cargo build --release
 
-RUN cargo build --target=x86_64-unknown-linux-musl --release
-
-## Final image
-
-FROM scratch
+# Final stage
+FROM debian:bookworm-slim
 
-COPY --from=builder /jnv/target/x86_64-unknown-linux-musl/release/jnv /bin/jnv
+COPY --from=builder /jnv/target/release/jnv /bin/jnv
 
 ENTRYPOINT ["/bin/jnv"]

README.md

@@ -75,21 +75,33 @@ See [package entry on search.nixos.org](https://search.nixos.org/packages?channe
 nix-shell -p jnv
 ```
 
-### Cargo
+### Docker
+
+Build
+(In the near future, the image will be available on something of registries)
 
 ```bash
-cargo install jnv
+docker build -t jnv .
 ```
 
-## Examples
+And Run
+(The following commad is just an example. Please modify the path to the file you want to mount)
 
 ```bash
-cat data.json | jnv
+docker run -it --rm -v $(pwd)/debug.json:/jnv/debug.json jnv /jnv/debug.json
 ```
 
-Or
+### Cargo
 
 ```bash
+cargo install jnv
+```
+
+## Examples
+
+```bash
+cat data.json | jnv
+# or
 jnv data.json
 ```