From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 68F8FC432C3 for ; Wed, 27 Nov 2019 21:42:28 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 47E6620870 for ; Wed, 27 Nov 2019 21:42:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729144AbfK0Vm1 (ORCPT ); Wed, 27 Nov 2019 16:42:27 -0500 Received: from smtp.hosts.co.uk ([85.233.160.19]:33055 "EHLO smtp.hosts.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728113AbfK0Vm0 (ORCPT ); Wed, 27 Nov 2019 16:42:26 -0500 Received: from [79.66.3.179] (helo=[192.168.1.22]) by smtp.hosts.co.uk with esmtpa (Exim) (envelope-from ) id 1ia54p-0000Vh-Ah; Wed, 27 Nov 2019 21:42:23 +0000 Subject: Re: [PATCH 1/1] contrib/buildsystems: fix Visual Studio Debug configuration To: Alexandr Miloslavskiy , Alexandr Miloslavskiy via GitGitGadget , git@vger.kernel.org Cc: Junio C Hamano References: <525669b3b38ed57d6d4f188dfe0bb8fe10b63749.1569227313.git.gitgitgadget@gmail.com> <8e7ff0b2-9f62-8ec6-5316-eb9cee25024e@syntevo.com> From: Philip Oakley Message-ID: Date: Wed, 27 Nov 2019 21:42:23 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.9.1 MIME-Version: 1.0 In-Reply-To: <8e7ff0b2-9f62-8ec6-5316-eb9cee25024e@syntevo.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-GB Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Hi Alexandr, On 27/11/2019 18:59, Alexandr Miloslavskiy wrote: > On 27.11.2019 18:56, Philip Oakley wrote: >> I just bumped against a potential issue like this. I was test >> compiling [1a,b] the `vs/master` branch from Git-For-Windows and got >> the LINK error that the 'libcurl-d.lib' was not found (4 places). >> >> Error    LNK1104    cannot open file 'libcurl-d.lib' git-imap-send >> C:\git-sdk-64\usr\src\git\git-imap-send\ LINK    1 >> >> Having just located this email, I changed the build type to 'Release' >> and the errors disappeared. >> >> Do we also need to identify where the libcurl-d.lib will be found? >> i.e. is it something that needs including via the sdk pacman list (I >> think I'm up to date but maybe not..) >> >> A quick web search didn't show any hits for `libcurl-d.lib` (with the >> dash `-`), though did find a few for `libcurld.lib`. > > If you clone `git-for-windows` and build in VS using `git.sln`, it > will automatically clone `git-for-windows\compat\vcbuild\vcpkg` and > build various things there, resulting in > > `git-for-windows\compat\vcbuild\vcpkg\buildtrees\curl\x64-windows-dbg\lib\libcurl-d.dll` > > > `git-for-windows\compat\vcbuild\vcpkg\installed\x64-windows\debug\bin\libcurl-d.dll` > > > `git-for-windows\compat\vcbuild\vcpkg\packages\curl_x64-windows\debug\bin\libcurl-d.dll` > > > Which will be picked up by solution to build git. > > I have built Debug many times now and didn't have any issues. If you > do, I would suggest to clone a new copy and build it. I already have the clone of vcpkg from 30/03/2019 with those three files already present. Have they been updated since then? The error report is specifically that the .lib file is missing (which I can't find..) However the plain `libcurl.lib` is present with matching dll's. It just feels a little unusual. On a side note there's a few (not many) Stackoverflow questions about building libcurl-d.lib, but they feel almost tangential https://stackoverflow.com/questions/51418238/libcurl-a-debug-lib-libcurl-a-lib-are-generated-instead-of-libcurld-lib-libcur https://stackoverflow.com/questions/37126943/libcurld-dll-is-missing-from-your-computer-adding-libcurl-to-visual-studio-pr quick look.. not sure if the assertion about choosing "ONE" (of debug/release) in https://github.com/git-for-windows/git/blob/master/compat/vcbuild/README#L26-L30 is part of the issue. That README doesn't really cover the update method for the vcpkg repository. It presumes you start from a fresh clone, which can be slow. I fetched/pulled the vcpkg repoo but no sign of an update. Philip