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 <<a hre= f=3D"mailto:[email protected]">[email protected]</a>>:<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 <<a href=3D"mailto:[email protected]" = target=3D"_blank">[email protected]</a>> wrote:<br> ><br> > On Tue, Aug 4, 2026 at 12:37=E2=80=AFPM Pavel Lyalyakin via dev <<a= href=3D"mailto:[email protected]" target=3D"_blank">dev@subversion= .apache.org</a>> wrote:<br> >><br> >> Hello,<br> ><br> > (snip)<br> >><br> >> The thing is that the INSTALL document needs a major rework (I thi= nk that everyone can agree with this).<br> ><br> > (snip)<br> ><br> > I agree with this!<br> ><br> > 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> ><br> > One good thing is that after the CMake & vcpkg build becomes the &= quot;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 wr= ecking ball.<br> ><br> > Thanks for giving INSTALL some much needed love!!<br> ><br> > By the way, I don't agree that it'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's something glamorous in that :-)<br> ><br> > 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--