From d0a62da988623147c45acf713d8ad0be93da97a7 Mon Sep 17 00:00:00 2001
From: Junhong Mao <junhong.mao@nokia.com>
Date: Thu, 1 Aug 2024 20:13:52 -0400
Subject: [PATCH] According to the review comments, update the rdb-cli
 installation location

---
 dockers/docker-base-bookworm/Dockerfile.j2     | 12 ------------
 dockers/docker-base-bullseye/Dockerfile.j2     | 18 ------------------
 .../broadcom/docker-syncd-brcm/Dockerfile.j2   | 18 ++++++++++++++++++
 3 files changed, 18 insertions(+), 30 deletions(-)
 mode change 100644 => 100755 dockers/docker-base-bookworm/Dockerfile.j2

diff --git a/dockers/docker-base-bookworm/Dockerfile.j2 b/dockers/docker-base-bookworm/Dockerfile.j2
old mode 100644
new mode 100755
index 2a3388d770d5..14032ed308f3
--- a/dockers/docker-base-bookworm/Dockerfile.j2
+++ b/dockers/docker-base-bookworm/Dockerfile.j2
@@ -81,18 +81,6 @@ RUN pip3 install supervisord-dependent-startup==1.4.0
 
 RUN mkdir -p /var/log/supervisor /etc/supervisor/conf.d
 
-# Install gcc, libc6-dev and python3-dev for compiling python-lzf
-RUN apt-get -y install build-essential libc6-dev python3-dev
-
-# Install python-lzf
-RUN pip3 install 'python-lzf==0.2.4'
-
-# Install rdbtools
-RUN pip3 install 'rdbtools==0.1.15'
-
-# Uninstall gcc, libc6-dev and python3-dev for compiling python-lzf
-RUN apt-get -y purge build-essential libc6-dev python3-dev
-
 # Uninstall unused dependencies
 RUN apt autoremove -y --purge
 
diff --git a/dockers/docker-base-bullseye/Dockerfile.j2 b/dockers/docker-base-bullseye/Dockerfile.j2
index 14ee284da94f..0ec093c90164 100755
--- a/dockers/docker-base-bullseye/Dockerfile.j2
+++ b/dockers/docker-base-bullseye/Dockerfile.j2
@@ -83,24 +83,6 @@ RUN pip3 install supervisord-dependent-startup==1.4.0
 
 RUN mkdir -p /var/log/supervisor /etc/supervisor/conf.d
 
-# Install gcc, libc6-dev and git for compiling librdb
-RUN apt-get -y install build-essential libc6-dev git
-
-# Obtain librdb source codes
-RUN git -C /tmp/ clone https://github.com/redis/librdb.git
-
-# Build librdb
-RUN cd /tmp/librdb/;git submodule update --init --recursive;make -j4
-
-# Install rdb-cli
-RUN install -p -D -m 0775 /tmp/librdb/bin/rdb-cli /usr/bin/rdb-cli
-
-# Clear librdb source codes
-RUN rm -rf /tmp/librdb
-
-# Uninstall gcc, libc6-dev and git for compiling librdb
-RUN apt-get -y purge build-essential libc6-dev git
-
 # Uninstall unused dependencies
 RUN apt autoremove -y --purge
 
diff --git a/platform/broadcom/docker-syncd-brcm/Dockerfile.j2 b/platform/broadcom/docker-syncd-brcm/Dockerfile.j2
index 4037ddc6af1a..a17f47719f17 100755
--- a/platform/broadcom/docker-syncd-brcm/Dockerfile.j2
+++ b/platform/broadcom/docker-syncd-brcm/Dockerfile.j2
@@ -30,6 +30,24 @@ COPY ["supervisord.conf", "/etc/supervisor/conf.d/"]
 COPY ["files/supervisor-proc-exit-listener", "/usr/bin"]
 COPY ["critical_processes", "/etc/supervisor/"]
 
+# Install make and git for compiling librdb
+RUN apt-get -y install build-essential git
+
+# Obtain librdb source codes
+RUN git clone https://github.com/redis/librdb.git
+
+# Build librdb
+RUN cd ./librdb/;git submodule update --init --recursive;make;cd ..
+
+# Install rdb-cli
+RUN install -p -D -m 0775 ./librdb/bin/rdb-cli /usr/bin/rdb-cli
+
+# Clear librdb source codes
+RUN rm -rf ./librdb
+
+# Uninstall make and git for compiling librdb
+RUN apt-get -y purge build-essential git
+
 ## Clean up
 RUN apt-get clean -y; apt-get autoclean -y; apt-get autoremove -y
 RUN rm -rf /debs