Skip to content

Commit ea33903

Browse files
committed
litrpc+proto: fix ownership in docker container
This was discovered while debugging the reproducible build. We used the wrong group ID. And the environment variables aren't super portable so we use the correct command to get the numeric user and group IDs.
1 parent fc77836 commit ea33903

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

litrpc/gen_protos_docker.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ docker build -t lit-protobuf-builder \
1717
echo "Compiling and formatting *.proto files..."
1818
docker run \
1919
--rm \
20-
--user $UID:$UID \
21-
-e UID=$UID \
20+
--user $(id -u):$(id -g) \
2221
-v "$DIR/../:/build" \
2322
lit-protobuf-builder

proto/gen_protos_docker.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ docker build -q -t lit-protobuf-builder .
1111
echo "Compiling and formatting *.proto files..."
1212
docker run \
1313
--rm \
14-
--user $UID:$UID \
15-
-e UID=$UID \
14+
--user $(id -u):$(id -g) \
1615
-v "$DIR/../:/build" \
1716
lit-protobuf-builder

0 commit comments

Comments
 (0)