|
108 | 108 | # verified" in the accompanying report) — so, same reasoning as |
109 | 109 | # sqlite3, curl is built from the current upstream release |
110 | 110 | # (curl-8.21.0.tar.xz, released 2026-06-24, sha256 verified locally) |
111 | | -# rather than assumed. Its ./configure is left to auto-detect whatever |
112 | | -# TLS backend (if any) the SDK provides — NOT forced via |
113 | | -# --with-openssl — so this module still builds successfully even if no |
114 | | -# TLS library is found; the honest, flagged consequence of that |
115 | | -# fallback is that HTTPS (biblehub.com scraping specifically; the |
116 | | -# 127.0.0.1/LAN TTS+Ollama traffic is plain HTTP either way) could |
117 | | -# silently lack support in that scenario. See the report for this |
118 | | -# flagged as a real, unverified gap. |
| 111 | +# rather than assumed. UPDATE (confirmed via a real flatpak-builder CI |
| 112 | +# run): leaving ./configure to "auto-detect whatever TLS backend the |
| 113 | +# SDK provides" does NOT gracefully fall back to no-TLS — it hard-errors |
| 114 | +# ("select TLS backend(s) or disable TLS with --without-ssl") and fails |
| 115 | +# the whole build. Now forced via --with-openssl, which org.gnome.Sdk |
| 116 | +# ships (a full desktop SDK) — needed for real HTTPS support |
| 117 | +# (biblehub.com scraping specifically; the 127.0.0.1/LAN TTS+Ollama |
| 118 | +# traffic is plain HTTP either way). |
119 | 119 | # |
120 | 120 | # ── finish-args ────────────────────────────────────────────────────────── |
121 | 121 | # Derived from grepping actual runtime behavior in src/, not copied from a |
@@ -180,11 +180,17 @@ modules: |
180 | 180 | - name: curl |
181 | 181 | buildsystem: simple |
182 | 182 | build-commands: |
183 | | - # No --with-openssl / --with-gnutls forced — auto-detect whatever's |
184 | | - # available so this module cannot hard-fail the whole build over a |
185 | | - # missing TLS backend. See the manifest header comment for the |
186 | | - # honestly-flagged consequence of this fallback. |
187 | | - - ./configure --prefix=/app --disable-static --without-libpsl --disable-manual |
| 183 | + # Confirmed via a real flatpak-builder CI run: NOT forcing a TLS |
| 184 | + # backend does not gracefully fall back to no-TLS the way the |
| 185 | + # previous comment here assumed — curl's ./configure hard-errors |
| 186 | + # ("select TLS backend(s) or disable TLS with --without-ssl") when it |
| 187 | + # can't auto-detect one, failing the whole build. org.gnome.Sdk//50 |
| 188 | + # (a full desktop SDK) ships OpenSSL, so --with-openssl is the |
| 189 | + # standard, well-established choice for curl-in-Flatpak — needed for |
| 190 | + # real HTTPS support (src/native_scraper.zig's biblehub.com fetches |
| 191 | + # use it; the loopback/LAN traffic is plain HTTP and wouldn't need |
| 192 | + # this, but the scraper path does). |
| 193 | + - ./configure --prefix=/app --disable-static --without-libpsl --disable-manual --with-openssl |
188 | 194 | - make -j${FLATPAK_BUILDER_N_JOBS:-4} |
189 | 195 | - make install |
190 | 196 | sources: |
|
0 commit comments