Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lighttpd: update to lighttpd 1.4.77 #1210

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gstrauss
Copy link
Contributor

lighttpd: update to lighttpd 1.4.77

dmacks
dmacks previously approved these changes Jan 11, 2025
Copy link
Member

@dmacks dmacks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I see a few support features are not enabled (brotli, xxhash, some others) that are lightweight dependencies. Worth trying to enable them?

@gstrauss
Copy link
Contributor Author

gstrauss commented Jan 11, 2025

xxhash is provided as part of lighttpd (and I generally keep it up to date), though there is an option to build lighttpd against a system-provided xxhash shared library. lighttpd 1.4.77 ships with xxHash v0.8.3. fink-distribution currently has xxHash v0.8.0, though that won't make much difference for lighttpd use of XXH32().

If fink is used on low-resource systems, it might be better to leave brotli out. However, on more modern hardware, then yes, lighttpd mod_deflate support for brotli is recommended.

lighttpd can be built with many features enabled or disabled, so I look to the fink maintainers for guidance and assistance configuring the fink-specific syntax in the *.info files.

BTW, lighttpd mod_webdav implements UUIDv4, so does not rely on libuuid anymore, and web/lighttpd.info can use --with-webdav-locks instead of --without-webdav-locks, though it does not really matter since mod_webdav locks are enabled if --with-webdav-props is enabled and the dependencies (libxml2 and sqlite3) are present.

Edit: also web/lighttpd.patch could remove mimetype.assign from the lighttpd.conf and use the mimetypes list of common web types which is built into modern lighttpd.
https://git.lighttpd.net/lighttpd/lighttpd1.4/src/branch/master/src/configfile.c#L949

@dmacks
Copy link
Member

dmacks commented Jan 11, 2025

Definitely no objection to keeping small support libraries onboard. The reason I thought about those two during ./configure because they are self-contained (would not have a large dependency tree brought along. I pushed fink's xxhash to 1.4.77, thanks for the heads-up:) For brotli, it uses cmake, so it's a lightweight library itself but maybe not a simple build...meh.

Feel free to make any packaging changes you think are best.

@gstrauss
Copy link
Contributor Author

Feel free to make any packaging changes you think are best.

I added a commit to simplify the lighttpd.conf.
Let's see what your peers think.
Feel free to edit the PR and remote that additional commit.

@dmacks
Copy link
Member

dmacks commented Jan 19, 2025

Simpler==good! The .patch MD5 needs to be updated in the .info.

I don't use lua in fink, but looking at the ./configure output, there's an upstream bug:

configure: Couldn\'t find lua54

Shell vs autotools quoting.

@gstrauss
Copy link
Contributor Author

Simpler==good! The .patch MD5 needs to be updated in the .info.

lighttpd.info MD5 updated for lighttpd.patch

configure: Couldn\'t find lua54

lighttpd configure tests for multiple versions of lua and multiple spellings.
Some "error" trace from configure is expected. This part of configure hasn't changed in recent versions of lighttpd, so this presumably worked with lighttpd-1.4.76 in fink. Would you share the lines in the ./configure output above and below that line. lighttpd ./configure --with-lua will error out and will not build lighttpd if a version of lua to use was not found, and --with-lua was specified.

lighttpd ./configure contains this:
for luaname in lua54 lua5.4 lua-5.4 lua53 lua5.3 lua-5.3 lua52 lua5.2 lua-5.2 lua51 lua5.1 lua-5.1 lua; do ...
so lighttpd ./configure will check all those paths before finally trying lua and presumably finding lua.pc.

...I'll replace Couldn\'t with Could not in lighttpd ./configure for a future release.

@gstrauss
Copy link
Contributor Author

rebased PR onto tip of fink master branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants