Re: [PATCH] INSTALL: clarify that libserf is auto-detected by pkg-config

Nathan Hartman <[email protected]> Tue, 4 Aug 2026 19:17:42 -0400
Newsgroups gmane.comp.version-control.subversion.devel
Message-ID <CAJT2EHqFeFqF=ganCMjriDRcRu_rZ78Wv8HgS=B8D7AtWSxKpA@mail.gmail.com>
On Tue, Aug 4, 2026 at 4:20=E2=80=AFPM Daniel Sahlberg
<[email protected]> wrote:
>
> tis 4 aug. 2026 kl. 20:45 skrev Nathan Hartman <[email protected]>=
:
>>
>> On Tue, Aug 4, 2026 at 2:04=E2=80=AFPM Nathan Hartman <hartman.nathan@gm=
ail.com> wrote:
>> >
>> > On Tue, Aug 4, 2026 at 12:37=E2=80=AFPM Pavel Lyalyakin via dev <dev@s=
ubversion.apache.org> wrote:
>> >>
>> >> Hello,
>> >
>> > (snip)
>> >>
>> >> The thing is that the INSTALL document needs a major rework (I think =
that everyone can agree with this).
>> >
>> > (snip)
>> >
>> > I agree with this!
>> >
>> > It is something I have looked into in the past, but every time I did, =
I became overwhelmed by how extensively INSTALL needed to change.
>> >
>> > One good thing is that after the CMake & vcpkg build becomes the "offi=
cial" way to build on Windows in 1.16, a huge amount of cruft can be remove=
d (all of section II-E). That will result in renumbering the later sections=
, so maybe it's a good time to just go in there with a wrecking ball.
>> >
>> > Thanks for giving INSTALL some much needed love!!
>> >
>> > By the way, I don't agree that it's not glamorous: for many users, INS=
TALL is likely one of the first pieces of documentation they encounter once=
 they decide to try Subversion. It was for me when I installed Subversion f=
or my $dayjob for the first time in 2007. INSTALL was very good, very clear=
, and it gave me confidence that I was dealing with a well made tool. There=
's something glamorous in that :-)
>> >
>> > Cheers,
>>
>>
>> I have nominated both changes (r1936876, r1936877) for backport to
>> 1.15.x in r1936879.
>>
>> Cheers,
>> Nathan
>
>
> I don=E2=80=99t have access to my computer tonight so I cannot vote but i=
f you are able, would you like to add my +1 and approve (since it is not co=
re code) ?
>
> /Daniel
>
>>

Sure, done in r1936883.

Cheers,
Nathan