diff options
| author | Denys Fedoryshchenko <denys.f@collabora.com> | 2026-08-04 17:50:49 +0300 |
|---|---|---|
| committer | Denys Fedoryshchenko <denys.f@collabora.com> | 2026-08-12 10:42:35 +0300 |
| commit | 4e36e08e90dbd3c27f566ca396f4284b77f1bf40 (patch) | |
| tree | bc9b7dd464baae4efa554d86c71c39ed1ae4c68c /.github/workflows | |
| parent | 9554ce05e13fdb8c55a54e259b4154c9aeaca58d (diff) | |
| download | accel-ppp-4e36e08e90dbd3c27f566ca396f4284b77f1bf40.tar.gz accel-ppp-4e36e08e90dbd3c27f566ca396f4284b77f1bf40.zip | |
crypto: drop the dangling crypto.h symlink and its last references
c912d090 ("crypto: Removed internal tomcat crypto.") deleted the
crypto/ tree but left accel-pppd/include/crypto.h behind, a tracked
symlink to ../../crypto/crypto.h which has pointed at nothing since.
backup_file.c still includes it, so it fails to compile with
fatal error: crypto.h: No such file or directory
That goes unnoticed because accel-pppd/CMakeLists.txt has
ADD_SUBDIRECTORY(backup) commented out, i.e. backup_file.c is not part
of any build; the breakage only shows up for whoever re-enables it.
Include <openssl/md5.h> instead, which is what the file actually needs
(MD5_CTX and friends) and what c912d090 did for every file it touched.
sstp.c and radius/packet.c defer to crypto.h for the rationale behind
their OPENSSL_API_COMPAT define. Spell it out locally instead, and point
at the project wide ADD_DEFINITIONS() in the top level CMakeLists.txt
added by c1689506 ("openssl: suppress deprecated API warnings"), noting
why the local define is kept despite being redundant with it: it has to
be visible before the first OpenSSL header.
Then remove the symlink, which nothing references anymore.
Diffstat (limited to '.github/workflows')
0 files changed, 0 insertions, 0 deletions
