I have removed the script with the motivation that it doesn't work as-is and the awk sources (since it was only a dependency for the script). I've also nominated for backport.
Cheers, Daniel Den tors 16 juli 2026 kl 16:25 skrev Daniel Sahlberg < [email protected]>: > Good catch, and unfortunately something I'm responsible for. > > It has been a long term project to review and try to update > build-svn-deps-win.pl to a working state. I have something half finished > but I never got around to complete it. That project started long before we > had CMake but I agree it superseeds this script. > > Awk was required for something, can't remember what right now, but there > was something that made the sourceforge version unsuitable. Still, it seems > like a bad idea to distribute the source with our tarballs. > > As for removing build-svn-deps-win.pl completely, I'm fine with that. It > clearly isn't working in the current state and if someone is using it, they > already have their own patched version. > > /Daniel > > Den tors 16 juli 2026 kl 15:53 skrev Ivan Zhakov <[email protected]>: > >> I have just noticed that source tree and tarball contains awk source >> code. It was added in r1914373 [1] >> [[[ >> In tools/dev: Copy awk from >> https://svn.apache.org/repos/asf/httpd/httpd/win32-msi/trunk/awk >> >> build-svn-deps-win.pl require AWK but a binary version is not so easy to >> find >> anymore. Thus include the sources here so we can build our own. >> ]]] >> >> However: >> >> 1. Apache HTTP Server no longer requires AWK for CMake-based build since >> r1919413. The change was backported to Apache HTTP Server 2.4.x in r1926319 >> [3]. >> > >> 2. tools/dev/awk is included in tarball, but NOTICE doesn't contains >> Copyright information for awk source code. >> >> 3. The build-svn-deps-win.pl script uses NMake-based build for Apache >> HTTP server, which is outdated. >> >> 4. I would say that most parts of the build-svn-deps-win.pl script are >> superseded by using CMake together with vcpkg (also described in INSTALL). >> >> 5. The build-svn-deps-win.pl script doesn't validate checksums when >> fetching remote data. >> >> Given all the above I suggest we remove tools/dev/awk and tools/dev/ >> build-svn-deps-win.pl from source code and final Subversion 1.15.0 >> tarball. >> >> Thoughts? >> >> [1]: https://svn.apache.org/viewvc/?revision=1914373&view=revision >> [2]: https://svn.apache.org/viewvc/?revision=r1919413&view=revision >> [3]: https://svn.apache.org/viewvc/?revision=r1926319&view=revision >> >> -- >> Ivan Zhakov >> >
