Skip to content

Commit

Permalink
Install zsync
Browse files Browse the repository at this point in the history 
@ivan-hc After seeing your changes I think it will make sense to add
this to the docker image.
  • Loading branch information
@andy5995
andy5995 committed Jan 31, 2025
1 parent b9c74a1 commit 3f70f7b
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion Dockerfile
View file
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,8 @@ RUN \
python3-pip \
sudo \
wget \
xxd && \
xxd \
zsync && \
update-ca-certificates -f && \
rm -rf /var/lib/apt/lists

Expand Down

3 comments on commit 3f70f7b

@ivan-hc
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, so non need to add this in the buider script, right?

@andy5995
Copy link
Owner Author

@andy5995 andy5995 commented on 3f70f7b Jan 31, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ivan-hc It will be easier to leave it in the builder script for now because the new images are still building. If at some point you are using the latest image where zync is installed, apt will just report that it's already installed. Once you see that, it can be removed or just left in so you don't have to rebuild. We can just remove it later anytime.

@ivan-hc
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andy5995 you're the boss

Please sign in to comment.