Skip to content

Commit e153b0c

Browse files
committed
fix: use release archive for libxcrypt
1 parent d6e42ae commit e153b0c

File tree

4 files changed

+12
-11
lines changed

4 files changed

+12
-11
lines changed

.github/workflows/update-dependencies.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ jobs:
3232
id: generate-token
3333
if: github.ref == 'refs/heads/main' && github.repository == 'pypa/manylinux'
3434
with:
35-
app_id: ${{ secrets.MANYLINUX_BOT_APP_ID }}
36-
private_key: ${{ secrets.MANYLINUX_BOT_APP_PRIVATE_KEY }}
35+
app-id: ${{ secrets.MANYLINUX_BOT_APP_ID }}
36+
private-key: ${{ secrets.MANYLINUX_BOT_APP_PRIVATE_KEY }}
3737
- name: "Run update native dependencies"
3838
run: nox -s update_native_dependencies
3939
env:

docker/Dockerfile

+2-2
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,8 @@ RUN export LIBTOOL_ROOT=libtool-2.5.3 && \
6262

6363
COPY build_scripts/install-libxcrypt.sh /build_scripts/
6464
RUN export LIBXCRYPT_VERSION=4.4.38 && \
65-
export LIBXCRYPT_HASH=9fb45586b2e83727934fbe046a4fef4804017577f1d048f91d56d0d830a5a98e && \
66-
export LIBXCRYPT_DOWNLOAD_URL=https://github.com/besser82/libxcrypt/archive && \
65+
export LIBXCRYPT_HASH=80304b9c306ea799327f01d9a7549bdb28317789182631f1b54f4511b4206dd6 && \
66+
export LIBXCRYPT_DOWNLOAD_URL=https://github.com/besser82/libxcrypt/releases/download && \
6767
manylinux-entrypoint /build_scripts/install-libxcrypt.sh
6868

6969
FROM runtime_base AS build_base

docker/build_scripts/install-libxcrypt.sh

+7-6
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,12 @@ fi
2020
check_var "${LIBXCRYPT_VERSION}"
2121
check_var "${LIBXCRYPT_HASH}"
2222
check_var "${LIBXCRYPT_DOWNLOAD_URL}"
23-
fetch_source "v${LIBXCRYPT_VERSION}.tar.gz" "${LIBXCRYPT_DOWNLOAD_URL}"
24-
check_sha256sum "v${LIBXCRYPT_VERSION}.tar.gz" "${LIBXCRYPT_HASH}"
25-
tar xfz "v${LIBXCRYPT_VERSION}.tar.gz"
26-
pushd "libxcrypt-${LIBXCRYPT_VERSION}"
27-
./autogen.sh > /dev/null
23+
LIBXCRYPT_ROOT="libxcrypt-${LIBXCRYPT_VERSION}"
24+
25+
fetch_source "${LIBXCRYPT_ROOT}.tar.xz" "${LIBXCRYPT_DOWNLOAD_URL}/v${LIBXCRYPT_VERSION}"
26+
check_sha256sum "${LIBXCRYPT_ROOT}.tar.xz" "${LIBXCRYPT_HASH}"
27+
tar xfJ "${LIBXCRYPT_ROOT}.tar.xz"
28+
pushd "${LIBXCRYPT_ROOT}"
2829
DESTDIR=/manylinux-rootfs do_standard_install \
2930
--disable-obsolete-api \
3031
--enable-hashes=all \
@@ -41,7 +42,7 @@ DESTDIR=/manylinux-rootfs/so.1 do_standard_install \
4142
cp -P /manylinux-rootfs/so.1/usr/local/lib/libcrypt.so.1* /manylinux-rootfs/usr/local/lib/
4243
rm -rf /manylinux-rootfs/so.1
4344
popd
44-
rm -rf "v${LIBXCRYPT_VERSION}.tar.gz" "libxcrypt-${LIBXCRYPT_VERSION}"
45+
rm -rf "${LIBXCRYPT_ROOT}.tar.xz" "${LIBXCRYPT_ROOT}"
4546

4647
# Strip what we can
4748
strip_ /manylinux-rootfs

tools/update_native_dependencies.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ def _update_with_gh(tool, dry_run):
158158
url = re.match(
159159
f"^ export {tool.upper()}_DOWNLOAD_URL=(?P<url>\\S+) && \\\\$", lines[i + 2]
160160
)["url"]
161-
sha256 = _sha256(f"{url}/{latest_tag}.tar.gz")
161+
sha256 = _sha256(f"{url}/{latest_tag}/libxcrypt-{latest_version}.tar.xz")
162162
lines[i + 0] = f"RUN export {tool.upper()}_VERSION={latest_version} && \\"
163163
lines[i + 1] = f" export {tool.upper()}_HASH={sha256} && \\"
164164
message = f"Bump {tool} {current_version}{latest_version}"

0 commit comments

Comments
 (0)