<div dir="ltr"><div><br></div><div>As a normal user who regularly builds his own postgis I have more than once been interrupted by missing of even problematic docbook dependencies. <br></div><div>It would not be missed by me.</div><div><br></div><div>Tom<br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 3, 2020 at 12:01 AM Regina Obe <<a href="mailto:lr@pcorp.us">lr@pcorp.us</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="EN-US"><div class="gmail-m_-8050092580626284340WordSection1"><p class="MsoNormal"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)">Added users to cc list to get more input.<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)">I have mixed feeling about this.  I see the value of not including it in extensions but would like to keep the make target.<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p><p class="gmail-m_-8050092580626284340MsoListParagraph"><u></u><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><span>1)<span style="font:7pt "Times New Roman"">      </span></span></span><u></u><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)">If you build from tar ball it’s already included built so you don’t need docbook to build extensions from tar ball.  But I guess if you build from git you do need dockbook. Given the craziness of our dependencies and the fact that most people don’t build their own PostGIS – the pain seems pretty minimal.<u></u><u></u></span></p><p class="gmail-m_-8050092580626284340MsoListParagraph"><u></u><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><span>2)<span style="font:7pt "Times New Roman"">      </span></span></span><u></u><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)">Building the comments has on occasion prevented us from documenting things that aren’t so.  For example if you said in the docs the function signature is<u></u><u></u></span></p><p class="gmail-m_-8050092580626284340MsoListParagraph"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p><p class="gmail-m_-8050092580626284340MsoListParagraph"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)">Foo(geometry, integer)<u></u><u></u></span></p><p class="gmail-m_-8050092580626284340MsoListParagraph"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p><p class="gmail-m_-8050092580626284340MsoListParagraph"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)">And it was defined in the code as foo(geometry, integer[])<u></u><u></u></span></p><p class="gmail-m_-8050092580626284340MsoListParagraph"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p><p class="gmail-m_-8050092580626284340MsoListParagraph"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)">Then the build would error out because the code doesn’t agree with the docs  – which I think is a nice feature.<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)">But that is the only really nice to have I can think of.  I suppose we could achieve the same by keeping the make target, but just don’t include it in extensions<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)">and have it as part of the ci steps to install the comments to confirm they install.<u></u><u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p><p class="MsoNormal"><span style="font-size:11pt;font-family:"Calibri",sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p><div style="border-color:currentcolor currentcolor currentcolor blue;border-style:none none none solid;border-width:medium medium medium 1.5pt;padding:0in 0in 0in 4pt"><div><div style="border-color:rgb(225,225,225) currentcolor currentcolor;border-style:solid none none;border-width:1pt medium medium;padding:3pt 0in 0in"><p class="MsoNormal"><b><span style="font-size:11pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11pt;font-family:"Calibri",sans-serif"> postgis-devel [mailto:<a href="mailto:postgis-devel-bounces@lists.osgeo.org" target="_blank">postgis-devel-bounces@lists.osgeo.org</a>] <b>On Behalf Of </b>Bruce Rindahl<br><b>Sent:</b> Monday, March 2, 2020 5:43 PM<br><b>To:</b> PostGIS Development Discussion <<a href="mailto:postgis-devel@lists.osgeo.org" target="_blank">postgis-devel@lists.osgeo.org</a>><br><b>Subject:</b> Re: [postgis-devel] postgis_comments.sql<u></u><u></u></span></p></div></div><p class="MsoNormal"><u></u> <u></u></p><div><p class="MsoNormal">This:<u></u><u></u></p><div><div><p class="MsoNormal"><img id="gmail-m_-8050092580626284340_x0000_i1025" src="cid:170a535d0d94cff311" alt="image.png" width="246" height="163"><u></u><u></u></p></div></div><div><p class="MsoNormal">then this:<u></u><u></u></p></div><div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal"><img id="gmail-m_-8050092580626284340_x0000_i1026" src="cid:170a535d0da692e332" alt="image.png" width="562" height="32"><u></u><u></u></p></div></div><div><p class="MsoNormal">??<u></u><u></u></p></div><div><p class="MsoNormal">vs this:<u></u><u></u></p></div><div><p class="MsoNormal"><a href="https://postgis.net/docs/DropGeometryColumn.html" target="_blank">https://postgis.net/docs/DropGeometryColumn.html</a> <u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal">I say it is a no brainer.  Especially since the function above is really not needed in PostGIS v3.0 <u></u><u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p></div><div><p class="MsoNormal"><u></u> <u></u></p></div></div><p class="MsoNormal"><u></u> <u></u></p><div><div><p class="MsoNormal">On Mon, Mar 2, 2020 at 2:34 PM Paul Ramsey <<a href="mailto:pramsey@cleverelephant.ca" target="_blank">pramsey@cleverelephant.ca</a>> wrote:<u></u><u></u></p></div><blockquote style="border-color:currentcolor currentcolor currentcolor rgb(204,204,204);border-style:none none none solid;border-width:medium medium medium 1pt;padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"><p class="MsoNormal">I have long had an animus for the comments.sql file, because it results in a couple bad couplings:<br><br>extension depends on comments depends on docbook, so you cannot build extension without having docbook/xslt tools installed<br>extension depends on commetns depends on docbook, so you cannot change documentation format<br><br>All this maybe would be worth choking down if the SQL comments provided any great value, but it’s not at all clear to me that they do. People lookup functions on <a href="http://postgis.net" target="_blank">postgis.net</a>, on the google, but not in their SQL terminal monitors or object browsers.<br><br>If we can find it without ourselves to not have SQL comments anymore, we can back out some of these weird dependencies.<br><br>Taking the temperature, who would set themselves on fire to save SQL comments?<br><br>P<br>_______________________________________________<br>postgis-devel mailing list<br><a href="mailto:postgis-devel@lists.osgeo.org" target="_blank">postgis-devel@lists.osgeo.org</a><br><a href="https://lists.osgeo.org/mailman/listinfo/postgis-devel" target="_blank">https://lists.osgeo.org/mailman/listinfo/postgis-devel</a><u></u><u></u></p></blockquote></div></div></div></div>_______________________________________________<br>
postgis-devel mailing list<br>
<a href="mailto:postgis-devel@lists.osgeo.org" target="_blank">postgis-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/postgis-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/postgis-devel</a></blockquote></div>