<div dir="ltr">On Sun, 19 Nov 2017 03:51:19 +0100 Cyril Brulebois <<a href="mailto:kibi@debian.org">kibi@debian.org</a>> wrote:<br>> Control: tag -1 patch<br>> <br>> Hi Sebastian, hi Denis,<br>> <br>> Sebastian Andrzej Siewior <sebastian@breakpoint.cc> (2017-11-14):<br>> > On 2017-10-21 16:37:14 [+0200], Denis Briand wrote:<br>> > > Pgadmin3 is no longer supported by upstream team and I haven't<br>> > > enought skills in openssl lib to fix this bug. Feel free to raise<br>> > > the bug severity to prevent pgadmin3 to be in buster.<br>> > > <br>> > > In a marvelous world we should build pgadmin4 instead but it's<br>> > > completly not the same interface (web) and needs many work hours and<br>> > > skills in web software package building.<br>> > <br>> > What is your final pgadmin plan for Buster? According to the webpage 3<br>> > is no longer supported. Do you want 3 in or plan to package 4?<br>> <br>> I've had the pleasure to discuss this with Denis at MiniDebConf in<br>> Toulouse, and AFAIUI it seems it would be nice to keep pgadmin3 around<br>> until pgadmin4 gets packaged.<br>> <br>> Looking at the build failure, it happens in paths involving libssh2:<br>>   pgadmin/libssh2<br>>   pgadmin/include/libssh2<br>> <br>> Since this seemed like an embedded code copy, I've looked into libssh2<br>> vs. openssl 1.1, and found this:<br>>   <a href="https://github.com/libssh2/libssh2/pull/70/commits">https://github.com/libssh2/libssh2/pull/70/commits</a><br>> <br>> I've applied both commits to openssl.{c,h} in pgadmin3 (need to patch<br>> both of them manually since they are not in the same directory):<br>>   <a href="https://github.com/libssh2/libssh2/pull/70/commits/a62842a9275fe3d6ba4b67b7f01ab3f00d086f83">https://github.com/libssh2/libssh2/pull/70/commits/a62842a9275fe3d6ba4b67b7f01ab3f00d086f83</a><br>>   <a href="https://github.com/libssh2/libssh2/pull/70/commits/eb497bddd5f382ca1468d72c6cd3d804ca68afb5">https://github.com/libssh2/libssh2/pull/70/commits/eb497bddd5f382ca1468d72c6cd3d804ca68afb5</a><br>> <br>> (Add “.patch” at the end of the URL to get a raw patch.)<br>> <br>> The build can then be resumed but fails later, possibly because patches<br>> in libssh2 were incomplete or because openssl made more changes. I<br>> didn't investigate at this point since it's clear that having embedded<br>> code copies is awful already.<br>>   <a href="https://wiki.debian.org/EmbeddedCodeCopies">https://wiki.debian.org/EmbeddedCodeCopies</a><br>> <br>> I've briefly looked into <a href="http://configure.ac">configure.ac</a> to see how library detection is<br>> performed, and found out that the embedded libssh2 depends on openssl<br>> being activated through ./configure; it's the default, but can be<br>> switched off, to use libgcrypt instead. I've added two flags in<br>> debian/rules, which made it possible to build the package. I didn't<br>> update build-depends, and I didn't test a full build under cowbuilder or<br>> sbuild, but that might be something that would help you keep pgadmin3 in<br>> testing a bit longer. See attached source debdiff.<br>> <br>> Please make sure to test your package extensively, I only checked it<br>> builds with these settings; also feel free to reword the changelog entry<br>> as you wish. :)<br>> <br>> <br>> Denis, On a personal note: you did a very good job organizing this<br>> Mini-DebConf, and an even better job getting people to look at your RC<br>> bugs! ;)<br>> <br><div><br></div><div>Would be sad to see pgadmin3 go, hoping for the patch to be accepted.</div><div><br></div></div>