[PR PATCH] lighttpd: update to 1.4.73

[PR PATCH] lighttpd: update to 1.4.73

[gstrauss]

[-- Attachment #1: Type: text/plain, Size: 454 bytes --]

There is a new pull request by gstrauss against master on the void-packages repository

https://github.com/gstrauss/void-packages lighttpd-1.4.73
https://github.com/void-linux/void-packages/pull/46978

lighttpd: update to 1.4.73
lighttpd: update to 1.4.73

#### Testing the changes

I tested the changes in this PR: NO

(build-tested elsewhere: on openwrt)

A patch file from https://github.com/void-linux/void-packages/pull/46978.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-lighttpd-1.4.73-46978.patch --]
[-- Type: text/x-diff, Size: 1148 bytes --]

From a7deefd301facbd733c51b7234b7bd5f17d43e0d Mon Sep 17 00:00:00 2001
From: Glenn Strauss <gstrauss@gluelogic.com>
Date: Tue, 31 Oct 2023 01:06:46 -0400
Subject: [PATCH] lighttpd: update to 1.4.73

---
 srcpkgs/lighttpd/template | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/lighttpd/template b/srcpkgs/lighttpd/template
index 07875c1099eca..864301510a009 100644
--- a/srcpkgs/lighttpd/template
+++ b/srcpkgs/lighttpd/template
@@ -1,7 +1,7 @@
 # Template file for 'lighttpd'
 pkgname=lighttpd
-version=1.4.71
-revision=3
+version=1.4.73
+revision=1
 build_style=meson
 configure_args="-Dwith_brotli=disabled -Dwith_bzip=disabled
  -Dwith_krb5=enabled -Dwith_ldap=enabled
@@ -20,7 +20,7 @@ maintainer="Enno Boland <gottox@voidlinux.org>"
 license="BSD-3-Clause"
 homepage="https://lighttpd.net"
 distfiles="https://download.lighttpd.net/lighttpd/releases-1.4.x/lighttpd-${version}.tar.xz"
-checksum=b8b6915da20396fdc354df3324d5e440169b2e5ea7859e3a775213841325afac
+checksum=818816d0b314b0aa8728a7076513435f6d5eb227f3b61323468e1f10dbe84ca8
 
 conf_files="/etc/lighttpd/lighttpd.conf"
 system_accounts="_lighttpd"

Re: lighttpd: update to 1.4.73

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 395 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786474477

Comment:
next time just amend and force-push the previous PR instead of opening another one

> I tested the changes in this PR: NO
> (build-tested elsewhere: on openwrt)

this PR isn't very useful if it wasn't run-tested at all and wasn't built on void

Re: lighttpd: update to 1.4.73

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 436 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786474477

Comment:
next time just amend and force-push the previous PR instead of opening another one

> I tested the changes in this PR: NO
> (build-tested elsewhere: on openwrt)

this PR isn't very useful if it wasn't run-tested at all and wasn't built on void (or do you mean in xbps-src on openwrt?)

Re: lighttpd: update to 1.4.73

[gstrauss]

[-- Attachment #1: Type: text/plain, Size: 413 bytes --]

New comment by gstrauss on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786485292

Comment:
@classabbyamp I am a lighttpd developer.  I have tested this release on on multiple platforms (Linux, FreeBSD, OpenBSD, NetBSD, MacOS) and multiple architectures (under openwrt), though not specifically on void linux.  If this PR is not acceptable, then please close it.

Re: lighttpd: update to 1.4.73

[gstrauss]

[-- Attachment #1: Type: text/plain, Size: 410 bytes --]

New comment by gstrauss on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786485292

Comment:
@classabbyamp I am a lighttpd developer.  I have tested this release on multiple platforms (Linux, FreeBSD, OpenBSD, NetBSD, MacOS) and multiple architectures (under openwrt), though not specifically on void linux.  If this PR is not acceptable, then please close it.

Re: lighttpd: update to 1.4.73

[gstrauss]

[-- Attachment #1: Type: text/plain, Size: 566 bytes --]

New comment by gstrauss on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786491918

Comment:
> this PR isn't very useful if it wasn't run-tested at all and wasn't built on void (or do you mean in xbps-src on openwrt?)

The CI tests built the package on multiple architectures under void linux.  Is that not sufficient for build testing?
Regarding run-testing: lighttpd source supports `make check` to run the lighttpd test suite.  This is exercised as part of some distro CI processes.  Can void linux CI do this?

Re: lighttpd: update to 1.4.73

[gstrauss]

[-- Attachment #1: Type: text/plain, Size: 568 bytes --]

New comment by gstrauss on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786491918

Comment:
> this PR isn't very useful if it wasn't run-tested at all and wasn't built on void (or do you mean in xbps-src on openwrt?)

The CI tests built the package on multiple architectures under void linux.  Is that not sufficient for build testing?

Regarding run-testing: lighttpd source supports `make check` to run the lighttpd test suite.  This is exercised as part of some distro CI processes.  Can void linux CI do this?

Re: lighttpd: update to 1.4.73

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 223 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786492236

Comment:
ok, we'd prefer if a member of the void community submitted a PR they tested

Re: lighttpd: update to 1.4.73

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 223 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786492236

Comment:
ok, we'd prefer if a member of the void community submitted a PR they tested

Re: lighttpd: update to 1.4.73

[gstrauss]

[-- Attachment #1: Type: text/plain, Size: 755 bytes --]

New comment by gstrauss on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786492906

Comment:
@classabbyamp the void linux CI does perform lighttpd `make check`, as I can see from the build logs:
```
=> lighttpd-1.4.73_1: running do_check ...
ninja: Entering directory `build'
[0/1] Running all tests.
1/9 test_configfile  OK              0.01s
2/9 test_common      OK              0.01s
3/9 test_mod         OK              0.01s
4/9 prepare          OK              0.01s
5/9 request.t        OK              0.26s
6/9 core-condition.t OK              0.09s
7/9 mod-fastcgi.t    OK              0.10s
8/9 mod-scgi.t       OK              0.08s
9/9 cleanup          OK              0.01s
```

Re: lighttpd: update to 1.4.73

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 676 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786494550

Comment:
> The CI tests built the package on multiple architectures under void linux. Is that not sufficient for build testing?

it is, but running it on void is the more important testing

> Can void linux CI do this?

yes, it runs check on all non-cross arches

In general, we'd prefer the package get tested in a real void environment by someone who uses the package on void (in addition to the CI and `make check`), as that can shake out some kinds of bugs and incompatibilities that the other parts of testing would not catch

Re: lighttpd: update to 1.4.73

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 243 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786497068

Comment:
If someone comes along and tests it in situ that's fine too, this PR doesn't *need* to be closed

Re: lighttpd: update to 1.4.73

[gstrauss]

[-- Attachment #1: Type: text/plain, Size: 820 bytes --]

New comment by gstrauss on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786523784

Comment:
Sounds like you think it more important for joe-user to say "yep, lighttpd serves my static file index.html on void linux." than the more extensive lighttpd test suite, which performs both unit tests and integration tests (e.g. full web request to a fastcgi server, an scgi server, CGI scripts, and more).

For a different package with minimal tests, I could see the desire for in-person manual testing of a package.  For more mature packages with more extensive test suites, I find that argument to be weaker that a user manually running a few smoke tests is better than the mature test suite which runs on multiple distros and multiple platforms and multiple architectures.

Re: lighttpd: update to 1.4.73

[0x5c]

[-- Attachment #1: Type: text/plain, Size: 1086 bytes --]

New comment by 0x5c on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1786546685

Comment:
Tested here and it works

> Sounds like you think it more important for joe-user to say "yep, lighttpd serves my static file index.html on void linux." than the more extensive lighttpd test suite, which performs both unit tests and integration tests (e.g. full web request to a fastcgi server, an scgi server, CGI scripts, and more).

It's && not ||; test failure is fatal in CI and will block merging of a PR until the cause can be found and fixed, unless the test is found to be broken in the first place or needing fixtures that can't run in CI.

Manual testing is still required because of the variety of ways different packages do tests, and the extra work that would be required to verify that each one of them has enough tests and the right kind of tests that would catch runtime problems.

It is also important to check that things like the services Void ships for packages do in fact continue to work as packaged after the update.

Re: [PR PATCH] [Merged]: lighttpd: update to 1.4.73

[classabbyamp]

[-- Attachment #1: Type: text/plain, Size: 293 bytes --]

There's a merged pull request on the void-packages repository

lighttpd: update to 1.4.73
https://github.com/void-linux/void-packages/pull/46978

Description:
lighttpd: update to 1.4.73

#### Testing the changes

I tested the changes in this PR: NO

(build-tested elsewhere: on openwrt)

Re: lighttpd: update to 1.4.73

[gstrauss]

[-- Attachment #1: Type: text/plain, Size: 1115 bytes --]

New comment by gstrauss on void-packages repository

https://github.com/void-linux/void-packages/pull/46978#issuecomment-1787514727

Comment:
> [...] test failure is fatal in CI and will block merging of a PR until the cause can be found and fixed [...]

@0x5c of course.  My responses in this PR are due to @classabbyamp saying
>> this PR isn't very useful if it wasn't run-tested at all and wasn't built on void (or do you mean in xbps-src on openwrt?)

Do you agree with @classabbyamp?  Do you not want encourage submissions to maintain packages?

> It is also important to check that things like the services Void ships for packages do in fact continue to work as packaged after the update.

@0x5c Can that be added to the CI?  If a package includes a service file, there can be an automated test which starts and stops the service.

If I spin up a void VM, what is the "manual testing" that you suggest before I say "I tested the changes in this PR: yes" in the PR?  (You're trusting people not to lie here.  I am at least asking what should be "manually" tested, since I can probably automate it.)