Skip to content

Commit

Permalink
release: Remove the getgroups workaround allowing users belonging to …
Browse files Browse the repository at this point in the history
…more than 32 groups

Following the merge of #5381 this workaround is no longer needed
  • Loading branch information
dra27 authored and kit-ty-kate committed Sep 18, 2024
1 parent 21bb251 commit ac8a72d
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 1 addition & 0 deletions master_changes.md
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,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]
Expand Down
1 change: 0 additions & 1 deletion release/Dockerfile.in
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down

0 comments on commit ac8a72d

Please sign in to comment.