Update RPM (Fedora 38) requirements

meztez · Dec 11, 2023 · a860454 · a860454
1 parent 624e756
commit a860454
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 15 deletions.
diff --git a/README.Rmd b/README.Rmd
@@ -60,10 +60,8 @@ apt-get install -y libgrpc++-dev libprotobuf-dev protobuf-compiler-grpc \
 #### Fedora 38
 
 ```sh
-# install protoc and grpc
-dnf install -y protobuf-devel protobuf-compiler \
-               grpc-devel c-ares-devel \
-               re2-devel pkgconf
+# install grpc, protoc is automatically installed
+dnf install -y grpc-devel pkgconf
 ```
 
 #### macOS

diff --git a/README.md b/README.md
@@ -59,10 +59,8 @@ apt-get install -y libgrpc++-dev libprotobuf-dev protobuf-compiler-grpc \
 #### Fedora 38
 
 ``` sh
-# install protoc and grpc
-dnf install -y protobuf-devel protobuf-compiler \
-               grpc-devel c-ares-devel \
-               re2-devel pkgconf
+# install grpc, protoc is automatically installed
+dnf install -y grpc-devel pkgconf
 ```
 
 #### macOS

diff --git a/configure b/configure
@@ -7,7 +7,7 @@
 # Library settings
 PKG_CONFIG_NAME="protobuf grpc++"
 PKG_DEB_NAME="libgrpc++-dev libprotobuf-dev protobuf-compiler-grpc pkg-config"
-PKG_RPM_NAME="protobuf-devel protobuf-compiler grpc-devel c-ares-devel re2-devel pkgconf"
+PKG_RPM_NAME="grpc-devel pkgconf"
 PKG_BREW_NAME="grpc pkg-config"
 PKG_TEST_HEADER="<grpc/grpc.h>"
 PKG_LIBS="-lgrpc++ -lgrpc -lprotobuf"

diff --git a/tools/build/linux/Dockerfile-fedora-38 b/tools/build/linux/Dockerfile-fedora-38
@@ -7,12 +7,6 @@ RUN yum install -y \
         https://github.com/r-lib/rig/releases/download/latest/r-rig-latest-1.$(arch).rpm && \
     rig add devel
 
-# -------------------------------------------------------------------------
-# FIXME: We don't need this once grpc is in the system requirements db
-RUN dnf install -y protobuf-devel protobuf-compiler \
-                   grpc-devel c-ares-devel \
-                   re2-devel pkgconf
-
 # -------------------------------------------------------------------------
 # Only copy DESCRIPTION, so Docker can cache the dependency install
 RUN mkdir /root/bigrquerystorage
@@ -21,6 +15,10 @@ WORKDIR /root/bigrquerystorage
 # No upgrade, to always install binary packages, in case P3M is behind
 RUN R -q -e 'pak::local_install_deps(upgrade = FALSE)'
 
+# -------------------------------------------------------------------------
+# FIXME: We don't need this once grpc is in the system requirements db
+RUN dnf install -y grpc-devel pkgconf
+
 # -------------------------------------------------------------------------
 # Call R CMD build first, so we get rid of Makevars, .o, .so, etc.
 # Change file ownership, otherwise git complains