Merge pull request #6200 from dra27/simpler-release

Remove the getgroups workaround allowing users belonging to more than 32 groups
ocaml · Sep 18, 2024 · b9e808b · b9e808b
2 parents 6f0cb44 + ac8a72d
commit b9e808b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/master_changes.md b/master_changes.md
@@ -128,6 +128,7 @@ users)
   * Exclude the .git directory from the release archive when using GNU tar [#6066 @kit-ty-kate]
   * Ensure non-existing %.cache target fail with a fatal error [#6066 @kit-ty-kate]
   * Remove opam 2.1 support from the release script [#6084 #6175 @kit-ty-kate]
+  * Remove the getgroups workaround allowing users belonging to more than 32 groups [#6200 @dra27 @kit-ty-kate]
 
 ## Install script
   * Provide a shell/install.ps1 PowerShell script to install opam on Windows [#5906 @kit-ty-kate @dra27]

diff --git a/release/Dockerfile.in b/release/Dockerfile.in
@@ -14,7 +14,6 @@ RUN tar xzf %OCAMLV%.tar.gz
 WORKDIR ocaml-%OCAMLV%
 RUN sed -i 's/gnueabi/*eabi/' configure
 RUN sed -i 's/musl/musl*/' configure
-RUN sed -i -e 's/NGROUPS_MAX/65536/' otherlibs/unix/getgroups.c
 RUN ./configure %CONF% -prefix /usr/local
 RUN make "-j$(nproc)" && make install && rm -rf /root/ocaml-%OCAMLV% /root/ocaml-%OCAMLV%.tar.gz