Skip to content

Commit a0ca3da

Browse files
committed
remove all remnants of commented out curl
for #96 if anyone still needs this, likely best to inherit from the image and adding in this step. CI is already slow with the libraries we have and trying to make room for aws's crypto lib without going insane #141. Signed-off-by: clux <[email protected]>
1 parent 69bcbd3 commit a0ca3da

File tree

7 files changed

+4
-58
lines changed

7 files changed

+4
-58
lines changed

Dockerfile.arm64

+1-13
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@ RUN chmod a+X /root
5454
# Convenience list of versions and variables for compilation later on
5555
# This helps continuing manually if anything breaks.
5656
ENV SSL_VER="1.1.1w" \
57-
CURL_VER="8.12.1" \
5857
ZLIB_VER="1.3.1" \
5958
PQ_VER="11.12" \
6059
SQLITE_VER="3490100" \
@@ -95,7 +94,7 @@ RUN curl -sSL https://zlib.net/zlib-$ZLIB_VER.tar.gz | tar xz && \
9594
make -j$(nproc) && make install && \
9695
cd .. && rm -rf zlib-$ZLIB_VER
9796

98-
# Build openssl (used in curl and pq)
97+
# Build openssl (used in pq)
9998
# Would like to use zlib here, but can't seem to get it to work properly
10099
# TODO: fix so that it works
101100
RUN curl -sSL https://www.openssl.org/source/openssl-$SSL_VER.tar.gz | tar xz && \
@@ -105,17 +104,6 @@ RUN curl -sSL https://www.openssl.org/source/openssl-$SSL_VER.tar.gz | tar xz &&
105104
make -j$(nproc) && make all install_sw && \
106105
cd .. && rm -rf openssl-$SSL_VER
107106

108-
# Build curl (needs with-zlib and all this stuff to allow https)
109-
# curl_LDFLAGS needed on stretch to avoid fPIC errors - though not sure from what
110-
# RUN curl -sSL https://curl.se/download/curl-$CURL_VER.tar.gz | tar xz && \
111-
# cd curl-$CURL_VER && \
112-
# CC="musl-gcc -fPIC -pie" LDFLAGS="-L$PREFIX/lib" CFLAGS="-I$PREFIX/include" ./configure \
113-
# --enable-shared=no --with-zlib --enable-static=ssl --enable-optimize --prefix=$PREFIX \
114-
# --with-ca-path=/etc/ssl/certs/ --with-ca-bundle=/etc/ssl/certs/ca-certificates.crt --without-ca-fallback \
115-
# --with-openssl --without-libpsl && \
116-
# make -j$(nproc) curl_LDFLAGS="-all-static" && make install && \
117-
# cd .. && rm -rf curl-$CURL_VER
118-
119107
# Build libpq
120108
RUN curl -sSL https://ftp.postgresql.org/pub/source/v$PQ_VER/postgresql-$PQ_VER.tar.gz | tar xz && \
121109
cd postgresql-$PQ_VER && \

Dockerfile.x86_64

+1-13
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ RUN chmod a+X /root
5353
# Convenience list of versions and variables for compilation later on
5454
# This helps continuing manually if anything breaks.
5555
ENV SSL_VER="1.1.1w" \
56-
CURL_VER="8.12.1" \
5756
ZLIB_VER="1.3.1" \
5857
PQ_VER="11.12" \
5958
SQLITE_VER="3490100" \
@@ -94,7 +93,7 @@ RUN curl -sSL https://zlib.net/zlib-$ZLIB_VER.tar.gz | tar xz && \
9493
make -j$(nproc) && make install && \
9594
cd .. && rm -rf zlib-$ZLIB_VER
9695

97-
# Build openssl (used in curl and pq)
96+
# Build openssl (used in pq)
9897
# Would like to use zlib here, but can't seem to get it to work properly
9998
# TODO: fix so that it works
10099
RUN curl -sSL https://www.openssl.org/source/openssl-$SSL_VER.tar.gz | tar xz && \
@@ -104,17 +103,6 @@ RUN curl -sSL https://www.openssl.org/source/openssl-$SSL_VER.tar.gz | tar xz &&
104103
make -j$(nproc) && make all install_sw && \
105104
cd .. && rm -rf openssl-$SSL_VER
106105

107-
# Build curl (needs with-zlib and all this stuff to allow https)
108-
# curl_LDFLAGS needed on stretch to avoid fPIC errors - though not sure from what
109-
# RUN curl -sSL https://curl.se/download/curl-$CURL_VER.tar.gz | tar xz && \
110-
# cd curl-$CURL_VER && \
111-
# CC="musl-gcc -fPIC -pie" LDFLAGS="-L$PREFIX/lib" CFLAGS="-I$PREFIX/include" ./configure \
112-
# --enable-shared=no --with-zlib --enable-static=ssl --enable-optimize --prefix=$PREFIX \
113-
# --with-ca-path=/etc/ssl/certs/ --with-ca-bundle=/etc/ssl/certs/ca-certificates.crt --without-ca-fallback \
114-
# --with-openssl --without-libpsl && \
115-
# make -j$(nproc) curl_LDFLAGS="-all-static" && make install && \
116-
# cd .. && rm -rf curl-$CURL_VER
117-
118106
# Build libpq
119107
RUN curl -sSL https://ftp.postgresql.org/pub/source/v$PQ_VER/postgresql-$PQ_VER.tar.gz | tar xz && \
120108
cd postgresql-$PQ_VER && \

README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -54,10 +54,11 @@ The following system libraries are compiled against `musl-gcc`:
5454
- [x] pq ([pq-sys crate](https://github.com/sgrif/pq-sys) used by [diesel](https://github.com/diesel-rs/diesel))
5555
- [x] sqlite3 ([libsqlite3-sys crate](https://github.com/jgallagher/rusqlite/tree/master/libsqlite3-sys) used by [diesel](https://github.com/diesel-rs/diesel))
5656
- [x] zlib (used by pq and openssl)
57-
- [~] curl ([curl crate](https://github.com/carllerche/curl-rust)) - dropped [#96](https://github.com/clux/muslrust/issues/96)
5857

5958
We **[try](https://github.com/clux/muslrust/blob/main/update_libs.py)** to keep these up to date.
6059

60+
NB: C library for `curl` has been removed in newer tags from 2025. See [#96](https://github.com/clux/muslrust/issues/96).
61+
6162
## Developing
6263

6364
Clone, tweak, build, and run tests:
@@ -79,7 +80,6 @@ Before we push a new version of muslrust we [test](https://github.com/clux/muslr
7980
- [x] [openssl](https://crates.io/crates/openssl)
8081
- [x] [flate2](https://crates.io/crates/flate2)
8182
- [x] [rand](https://crates.io/crates/rand)
82-
- [~] [curl](https://crates.io/crates/curl) - dropped [#96](https://github.com/clux/muslrust/issues/96)
8383

8484
## Caching
8585

justfile

-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ _t_macos_aarch64 crate:
4949
test: (_ti "plain") (_ti "ssl") (_ti "rustls") (_ti "pq") (_ti "serde") (_ti "zlib") (_ti "hyper") (_ti "dieselpg") (_ti "dieselsqlite")
5050
# Test all crates against built container in ci (inheriting set PLATFORM/TARGET_DIR/AR vars)
5151
test-ci: (_t "plain") (_t "ssl") (_t "rustls") (_t "pq") (_t "serde") (_t "zlib") (_t "hyper") (_t "dieselpg") (_t "dieselsqlite")
52-
# NB: taken out curl for #96 and build issuse
5352

5453
# Cleanup everything
5554
clean: clean-docker clean-tests

test/curlcrate/Cargo.toml

-7
This file was deleted.

test/curlcrate/src/main.rs

-21
This file was deleted.

update_libs.py

-1
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,6 @@ def rustup_version():
8282

8383
if __name__ == '__main__':
8484
PACKAGES = {
85-
'CURL': pkgver('curl'),
8685
#'PQ': pkgver('postgresql-old-upgrade'), # see https://github.com/clux/muslrust/issues/81
8786
'SQLITE': convert_sqlite_version(pkgver('sqlite')),
8887
'SSL': convert_openssl_version(pkgver('openssl-1.1')),

0 commit comments

Comments
 (0)