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

Daniel Sahlberg <[email protected]> Tue, 4 Aug 2026 22:19:31 +0200
Newsgroups gmane.comp.version-control.subversion.devel
Message-ID <CAMHy98NDWebU3Uw-bdvUOXF6t=rmn1hNFORZVtRTrQqQHjxuGA@mail.gmail.com>
--000000000000121e8a06583e60c0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

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@gma=
il.com>
> wrote:
> >
> > On Tue, Aug 4, 2026 at 12:37=E2=80=AFPM Pavel Lyalyakin via dev <
> [email protected]> 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
> "official" way to build on Windows in 1.16, a huge amount of cruft can be
> removed (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,
> INSTALL 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 for 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 wel=
l
> 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 if =
you are
able, would you like to add my +1 and approve (since it is not core code) ?

/Daniel


>

--000000000000121e8a06583e60c0
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"auto">tis 4 aug. 2026 kl. 20:45 skrev Nathan Hartman &lt;<a hre=
f=3D"mailto:[email protected]">[email protected]</a>&gt;:<br>=
</div><div dir=3D"auto"><div class=3D"gmail_quote gmail_quote_container"><b=
lockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-le=
ft:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Aug 4, 2026 at 2:04=
=E2=80=AFPM Nathan Hartman &lt;<a href=3D"mailto:[email protected]" =
target=3D"_blank">[email protected]</a>&gt; wrote:<br>
&gt;<br>
&gt; On Tue, Aug 4, 2026 at 12:37=E2=80=AFPM Pavel Lyalyakin via dev &lt;<a=
 href=3D"mailto:[email protected]" target=3D"_blank">dev@subversion=
.apache.org</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; Hello,<br>
&gt;<br>
&gt; (snip)<br>
&gt;&gt;<br>
&gt;&gt; The thing is that the INSTALL document needs a major rework (I thi=
nk that everyone can agree with this).<br>
&gt;<br>
&gt; (snip)<br>
&gt;<br>
&gt; I agree with this!<br>
&gt;<br>
&gt; 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.<br>
&gt;<br>
&gt; One good thing is that after the CMake &amp; vcpkg build becomes the &=
quot;official&quot; way to build on Windows in 1.16, a huge amount of cruft=
 can be removed (all of section II-E). That will result in renumbering the =
later sections, so maybe it&#39;s a good time to just go in there with a wr=
ecking ball.<br>
&gt;<br>
&gt; Thanks for giving INSTALL some much needed love!!<br>
&gt;<br>
&gt; By the way, I don&#39;t agree that it&#39;s not glamorous: for many us=
ers, INSTALL is likely one of the first pieces of documentation they encoun=
ter once they decide to try Subversion. It was for me when I installed Subv=
ersion for my $dayjob for the first time in 2007. INSTALL was very good, ve=
ry clear, and it gave me confidence that I was dealing with a well made too=
l. There&#39;s something glamorous in that :-)<br>
&gt;<br>
&gt; Cheers,<br>
<br>
<br>
I have nominated both changes (r1936876, r1936877) for backport to<br>
1.15.x in r1936879.<br>
<br>
Cheers,<br>
Nathan</blockquote><div dir=3D"auto"><br></div><div dir=3D"auto">I don=E2=
=80=99t have access to my computer tonight so I cannot vote but if you are =
able, would you like to=C2=A0add my +1 and approve (since it is not core co=
de) ?</div><div dir=3D"auto"><br></div><div dir=3D"auto">/Daniel</div><div =
dir=3D"auto"><br></div><blockquote class=3D"gmail_quote" style=3D"margin:0p=
x 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" di=
r=3D"auto"><br>
</blockquote></div></div>

--000000000000121e8a06583e60c0--