Re: State of tools/dev/awk and tools/build/build-svn-deps-win.pl

Daniel Sahlberg <[email protected]>
Newsgroups gmane.comp.version-control.subversion.devel
Message-ID <CAMHy98MjQkLLMpRUW4G+kBcTbG-DANwxxSzLiri1Q+WjtN5i9A@mail.gmail.com>
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
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.