{"thread":{"id":"54863","subject":"[BUG] git-2.30.0-rc1 - Transitive OpenLDAP requirement in libcurl","startedAt":"2020-12-21T17:22:58Z","lastAt":"2020-12-21T21:03:59Z","messageCount":5,"participants":["Randall S. Becker","Junio C Hamano","Felipe Contreras"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"412716","messageId":"002001d6d7bd$d03d7e10$70b87a30$@nexbridge.com","threadId":"54863","inReplyTo":null,"subject":"[BUG] git-2.30.0-rc1 - Transitive OpenLDAP requirement in libcurl","fromName":"Randall S. Becker","fromEmail":"rsbecker@nexbridge.com","sentAt":"2020-12-21T17:22:04Z","receivedAt":"2020-12-21T17:22:58Z","isPatch":false,"sender":{"key":"randall.becker@nexbridge.ca","avatar":"https://avatars.githubusercontent.com/u/28956764?v=4"},"body":"The NonStop platform does not have an implementation of OpenLDAP, but git is\nattempting to use it. We are getting unresolved references from libcurl,\nwhich makes references to openldap regardless of whether it exists. We do\nnot expect git to use ldap_setup_connection, for example. We need a way of\ndisabling the use of this package. This appears to be is a net new for issue\nfor 2.23.0, which we just did not get far enough on rc0 to encounter. \n\nNeed assistance to resolve this.\n\nThanks,\nRandall\n\nSample (there are more) references:\n\neld command line:\n /usr/bin/eld -o git-imap-send -set systype oss -set highpin on -set\n highrequestor on -set inspect on /usr/lib/ccplmain.o imap-send.o http.o\n common-main.o -L/usr/local/lib -lcurl -lssl -lcrypto -lssl -lcrypto -lz\n -lssl -lcrypto -lcrypto libgit.a xdiff/lib.a -lz -lintl -liconv\n -L/usr/local/lib -lfloss -lutil -lrld -L /G/system/sys04 -lcre -lcrtl\n -lossk -lossf -lsec -li18n -licnv -losse -linet -lossh -lossc\n -allow_duplicate_procs\n\n**** ERROR **** [1210]:\n /usr/local/lib/libcurl.a(libcurl_la-openldap.o): In function\n `ldap_setup_connection':\n libcurl_la-openldap.o(.text._153926392+0x72): unresolved reference to\n ldap_url_parse.\n\n\n-- Brief whoami:\n NonStop developer since approximately 211288444200000000\n UNIX developer since approximately 421664400\n-- In my real life, I talk too much.\n\n\n"},{"id":"412749","messageId":"xmqqeejjm2ek.fsf@gitster.c.googlers.com","threadId":"54863","inReplyTo":"002001d6d7bd$d03d7e10$70b87a30$@nexbridge.com","subject":"Re: [BUG] git-2.30.0-rc1 - Transitive OpenLDAP requirement in libcurl","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-12-21T18:37:07Z","receivedAt":"2020-12-21T18:38:08Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Randall S. Becker\" writes:\n\n> The NonStop platform does not have an implementation of OpenLDAP, but git is\n> attempting to use it. We are getting unresolved references from libcurl,\n> which makes references to openldap regardless of whether it exists. We do\n> not expect git to use ldap_setup_connection, for example. We need a way of\n> disabling the use of this package. This appears to be is a net new for issue\n> for 2.23.0, which we just did not get far enough on rc0 to encounter. \n\nI assume \"23\" on the last line is typo for \"30\".\n\nBetween v2.29.0..v2.30.0-rc1, there is nothing other than 3b990aa6\n(push: parse and set flag for \"--force-if-includes\", 2020-10-03),\nwhich seem to have nothing to do with auth, that touches http*.c or\nremote-curl.c\n\nIs it possible that you updated your build procedure and/or options\nfor libcurl since last time you built a working Git successfully?\nIf you build v2.29.0 with the same toolchain and library that failed\nwith -rc1, does the result work OK?\n\nDoes NonStop software collection have programs other than Git that\nlinks with libcurl, and if so do they work (in other words, is the\nlatest source of Git the only thing that suffers from this issue)?\n\nSorry, I have *no* further suggestions even if you answered any of\nthe above questions to me, but I hope others (inclusing you and your\nfolks) may have better insights after answering the above questions\n;-)\n\n> Need assistance to resolve this.\n>\n> Thanks,\n> Randall\n>\n> Sample (there are more) references:\n>\n> eld command line:\n> /usr/bin/eld -o git-imap-send -set systype oss -set highpin on -set\n> highrequestor on -set inspect on /usr/lib/ccplmain.o imap-send.o http.o\n> common-main.o -L/usr/local/lib -lcurl -lssl -lcrypto -lssl -lcrypto -lz\n> -lssl -lcrypto -lcrypto libgit.a xdiff/lib.a -lz -lintl -liconv\n> -L/usr/local/lib -lfloss -lutil -lrld -L /G/system/sys04 -lcre -lcrtl\n> -lossk -lossf -lsec -li18n -licnv -losse -linet -lossh -lossc\n> -allow_duplicate_procs\n>\n> **** ERROR **** [1210]:\n> /usr/local/lib/libcurl.a(libcurl_la-openldap.o): In function\n> `ldap_setup_connection':\n> libcurl_la-openldap.o(.text._153926392+0x72): unresolved reference to\n> ldap_url_parse.\n>\n>\n> -- Brief whoami:\n> NonStop developer since approximately 211288444200000000\n> UNIX developer since approximately 421664400\n> -- In my real life, I talk too much.\n"},{"id":"412765","messageId":"003e01d6d7ce$ccc648f0$6652dad0$@nexbridge.com","threadId":"54863","inReplyTo":"xmqqeejjm2ek.fsf@gitster.c.googlers.com","subject":"RE: [BUG] git-2.30.0-rc1 - Transitive OpenLDAP requirement in libcurl","fromName":"Randall S. Becker","fromEmail":"rsbecker@nexbridge.com","sentAt":"2020-12-21T19:23:39Z","receivedAt":"2020-12-21T19:24:35Z","isPatch":false,"sender":{"key":"randall.becker@nexbridge.ca","avatar":"https://avatars.githubusercontent.com/u/28956764?v=4"},"body":"On December 21, 2020 1:37 PM, Junio C Hamano wrote:\n> \"Randall S. Becker\" writes:\n> \n> > The NonStop platform does not have an implementation of OpenLDAP, but\n> > git is attempting to use it. We are getting unresolved references from\n> > libcurl, which makes references to openldap regardless of whether it\n> > exists. We do not expect git to use ldap_setup_connection, for\n> > example. We need a way of disabling the use of this package. This\n> > appears to be is a net new for issue for 2.23.0, which we just did not\nget far\n> enough on rc0 to encounter.\n> \n> I assume \"23\" on the last line is typo for \"30\".\n\nYes, I goofed up here. 2.30.\n\n> Between v2.29.0..v2.30.0-rc1, there is nothing other than 3b990aa6\n> (push: parse and set flag for \"--force-if-includes\", 2020-10-03), which\nseem\n> to have nothing to do with auth, that touches http*.c or remote-curl.c\n> \n> Is it possible that you updated your build procedure and/or options for\nlibcurl\n> since last time you built a working Git successfully?\n\nThe only change on the system anywhere near the toolchain was removing\nMKDIR_WO_TRAILING_SLASH and NO_SETITIMER.\n\n> If you build v2.29.0 with the same toolchain and library that failed with\n-rc1,\n> does the result work OK?\n> \n> Does NonStop software collection have programs other than Git that links\n> with libcurl, and if so do they work (in other words, is the latest source\nof Git\n> the only thing that suffers from this issue)?\n\ngit is the only product my group builds that uses libcurl. There is a less\nfunctional\nversion of libcurl that comes with the operating system that we ignore. We\nare not\nlinking with that version. The version of curl we have, we built but not\nsince 2.29.2.\nSo I'm confused.\n\n> \n> Sorry, I have *no* further suggestions even if you answered any of the\nabove\n> questions to me, but I hope others (inclusing you and your\n> folks) may have better insights after answering the above questions\n> ;-)\n> \n> > Need assistance to resolve this.\n> >\n> > Thanks,\n> > Randall\n> >\n> > Sample (there are more) references:\n> >\n> > eld command line:\n> > /usr/bin/eld -o git-imap-send -set systype oss -set highpin on -set\n> > highrequestor on -set inspect on /usr/lib/ccplmain.o imap-send.o\nhttp.o\n> > common-main.o -L/usr/local/lib -lcurl -lssl -lcrypto -lssl -lcrypto\n-lz\n> > -lssl -lcrypto -lcrypto libgit.a xdiff/lib.a -lz -lintl -liconv\n> > -L/usr/local/lib -lfloss -lutil -lrld -L /G/system/sys04 -lcre -lcrtl\n> > -lossk -lossf -lsec -li18n -licnv -losse -linet -lossh -lossc\n> > -allow_duplicate_procs\n> >\n> > **** ERROR **** [1210]:\n> > /usr/local/lib/libcurl.a(libcurl_la-openldap.o): In function\n> > `ldap_setup_connection':\n> > libcurl_la-openldap.o(.text._153926392+0x72): unresolved reference to\n> > ldap_url_parse.\n> >\n> >\n> > -- Brief whoami:\n> > NonStop developer since approximately 211288444200000000 UNIX\n> > developer since approximately 421664400\n> > -- In my real life, I talk too much.\n\n"},{"id":"412772","messageId":"5fe108752910_f93820887@natae.notmuch","threadId":"54863","inReplyTo":"003e01d6d7ce$ccc648f0$6652dad0$@nexbridge.com","subject":"RE: [BUG] git-2.30.0-rc1 - Transitive OpenLDAP requirement in libcurl","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2020-12-21T20:41:25Z","receivedAt":"2020-12-21T20:42:08Z","isPatch":false,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Randall S. Becker wrote:\n> git is the only product my group builds that uses libcurl. There is a\n> less functional version of libcurl that comes with the operating\n> system that we ignore. We are not linking with that version. The\n> version of curl we have, we built but not since 2.29.2.\n> So I'm confused.\n\nCould be: b990f02fd8 (config.mak.uname: remove unused NEEDS_SSL_WITH_CURL flag, 2020-11-11)\n\nIt seems it was attempting to only remove NEEDS_SSL_WITH_CURL, which\nindeed wasn't used anymore, but it also removed:\n\n #NO_CURL = YesPlease\n\nI've no idea about these systems, but a quick search makes me wonder if\nperhaps this was not intended.\n\n-- \nFelipe Contreras\n"},{"id":"412773","messageId":"004601d6d7dc$a99af4d0$fcd0de70$@nexbridge.com","threadId":"54863","inReplyTo":"5fe108752910_f93820887@natae.notmuch","subject":"RE: [BUG] git-2.30.0-rc1 - Transitive OpenLDAP requirement in libcurl","fromName":"Randall S. Becker","fromEmail":"rsbecker@nexbridge.com","sentAt":"2020-12-21T21:02:53Z","receivedAt":"2020-12-21T21:03:59Z","isPatch":false,"sender":{"key":"randall.becker@nexbridge.ca","avatar":"https://avatars.githubusercontent.com/u/28956764?v=4"},"body":"On December 21, 2020 3:41 PM, Felipe Contreras wrote:\n> To: Randall S. Becker ; 'Junio C Hamano'\n> \n> Cc: git@vger.kernel.org\n> Subject: RE: [BUG] git-2.30.0-rc1 - Transitive OpenLDAP requirement in\n> libcurl\n> \n> Randall S. Becker wrote:\n> > git is the only product my group builds that uses libcurl. There is a\n> > less functional version of libcurl that comes with the operating\n> > system that we ignore. We are not linking with that version. The\n> > version of curl we have, we built but not since 2.29.2.\n> > So I'm confused.\n> \n> Could be: b990f02fd8 (config.mak.uname: remove unused\n> NEEDS_SSL_WITH_CURL flag, 2020-11-11)\n> \n> It seems it was attempting to only remove NEEDS_SSL_WITH_CURL, which\n> indeed wasn't used anymore, but it also removed:\n\nI wondered about this one. I have rolled this back, with no positive effect. The 2.30.0-rc1 build works fine on the x86 platform (currently under test) but not on the Itanium (older) so I'm not suspecting this issue.\n\n> #NO_CURL = YesPlease\n\nWe have been building with Curl for years, so probably not this.\n\n> I've no idea about these systems, but a quick search makes me wonder if\n> perhaps this was not intended.\n\nIs there some detection going on about OpenLDAP?\n\n"}]}