This is an automated email from the ASF dual-hosted git repository.

jerryshao pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new 2ee1453fd8 [#8309] docs: replace deprecated apt-key in how-to-build.md 
(#12706)
2ee1453fd8 is described below

commit 2ee1453fd8e2c5cd823f7ddac732b0a772b04af3
Author: Abdullah <[email protected]>
AuthorDate: Fri Aug 28 15:17:36 2026 +0300

    [#8309] docs: replace deprecated apt-key in how-to-build.md (#12706)
    
    ### What changes were proposed in this pull request?
    
    The Ubuntu (WSL) Docker steps in `docs/how-to-build.md` still used
    `apt-key add`. I switched that to the signed keyring + `signed-by` repo
    line from the current Docker Ubuntu install docs, and left the
    hello-world / docker group bits as they were.
    
    ### Why are the changes needed?
    
    `apt-key` is deprecated. #8309 also mentioned a stray `curl -fsSL https`
    line; that is already gone on main.
    
    Fix: #8309
    
    ### Does this PR introduce any user-facing change?
    
    Docs only.
    
    ### How was this patch tested?
    
    Compared the snippet with
    https://docs.docker.com/engine/install/ubuntu/.
---
 docs/how-to-build.md | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/docs/how-to-build.md b/docs/how-to-build.md
index a4221cef1a..e6bad80121 100644
--- a/docs/how-to-build.md
+++ b/docs/how-to-build.md
@@ -202,8 +202,12 @@ Updating the package list ensures you have the latest 
information on the newest
 **On Ubuntu (WSL):**
 
 ```shell
-curl -fsSL https://download.docker.com/linux/ubuntu/gpg | sudo apt-key add -
-sudo add-apt-repository "deb [arch=amd64] 
https://download.docker.com/linux/ubuntu $(lsb_release -cs) stable"
+sudo apt update
+sudo apt install ca-certificates curl
+sudo install -m 0755 -d /etc/apt/keyrings
+sudo curl -fsSL https://download.docker.com/linux/ubuntu/gpg -o 
/etc/apt/keyrings/docker.asc
+sudo chmod a+r /etc/apt/keyrings/docker.asc
+echo "deb [arch=$(dpkg --print-architecture) 
signed-by=/etc/apt/keyrings/docker.asc] 
https://download.docker.com/linux/ubuntu $(. /etc/os-release && echo 
"${UBUNTU_CODENAME:-$VERSION_CODENAME}") stable" | sudo tee 
/etc/apt/sources.list.d/docker.list > /dev/null
 sudo apt update
 sudo apt install docker-ce
 sudo service docker start
@@ -211,7 +215,7 @@ sudo docker run hello-world
 sudo usermod -aG docker $USER
 ```
 
-These commands install Docker. Running `hello-world` verifies the 
installation. Adding your user to the Docker group allows you to run Docker 
commands without `sudo`.
+These commands install Docker using the signed `apt` keyring instead of the 
deprecated `apt-key` command. Running `hello-world` verifies the installation. 
Adding your user to the Docker group allows you to run Docker commands without 
`sudo`.
 
 ### Install Python 3.11
 

Reply via email to