From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: None (no SPF record) identity=mailfrom; client-ip=2a00:1450:400c:c09::22a; helo=mail-wm0-x22a.google.com; envelope-from=pete@akeo.ie; receiver=edk2-devel@lists.01.org Received: from mail-wm0-x22a.google.com (mail-wm0-x22a.google.com [IPv6:2a00:1450:400c:c09::22a]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 0A07F20355206 for ; Thu, 16 Nov 2017 05:17:11 -0800 (PST) Received: by mail-wm0-x22a.google.com with SMTP id b189so9849195wmd.0 for ; Thu, 16 Nov 2017 05:21:21 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=akeo-ie.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id; bh=XjQ4qd1qXnNwEypl4cU+VmIVt28VqIBSVOvmF9GCcSs=; b=rX1hIHUYO2cBOCXiBirSLCM1vyPmPueU/R6igJS/4hbrRpunQzCQj8pivlxSVF0+bB jDyxzWEJudPtJwGlcORfziDydBGax2YwCVF0prrYiZ0+T+7KnGeUvqRu0McK5HlqZA8k 4NW/yV5IHkjBhOI2c7fKI/LwL8Kze2JMgZHDARPDn2Q23lriJOBqWRyxfHhPF28tcPcW 4denKviURbMCt4fW5gbmzxaxyK1m4ocpp8a/+hTnrmSZ9YbNvZRPB/Urs9ZPpqPAI0Pg aw//XEE/Z+PYqd/N5QYwS82d8Deazh47PwiEjhiIifFc2oTsjanJX90eLEnDqyiu/z0D 8K+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=XjQ4qd1qXnNwEypl4cU+VmIVt28VqIBSVOvmF9GCcSs=; b=Kmm/R9HtzA426AceTxYBthBlsfDlEY4QdIn+th+rooHUm8648h/YrxRa1w0XY6ar/H nirEYpm1WYrPTyPYg7bF5XhsdutfC0W5iLQNXcLM56JjJva2cMGE2CUef1gJphLgiTzC +t5HJ4gTxeoxLQ7XKHDNrMZCDs5t3Zi19JaIm/HkxgrfkaHBetcDReMrUWHoORYFekuP 6woT7GZKegGIFQVril1EAdXcPOFvm9rbiRVQqXWFP5esnfLv5mlHKxOy+flmHAUXgDjq 6/a5W3MtY2AaUCzOy6tyy/0IqXa/zJnh3sONdSeyT/IqIGcbMVkanfrmHoRSlIMXSEQu +2CQ== X-Gm-Message-State: AJaThX4qA9Id3ULwNvar2qhua4gEz+lq3p8XM0HRld0a8x2EvlJpbum6 R+1afDmKkcV271m5JljulUUSb3AvOY0= X-Google-Smtp-Source: AGs4zMavsd//OT7QuUjjEiXLZ6suHr5VhiWOVOvrqKWjE24fKDDQsZCgdEJlSCI0VgZSzWmt3UcWHA== X-Received: by 10.80.165.41 with SMTP id y38mr2732767edb.72.1510838479917; Thu, 16 Nov 2017 05:21:19 -0800 (PST) Received: from localhost.localdomain ([84.203.65.190]) by smtp.gmail.com with ESMTPSA id l4sm1334097edc.20.2017.11.16.05.21.18 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 16 Nov 2017 05:21:18 -0800 (PST) From: Pete Batard To: edk2-devel@lists.01.org Cc: liming.gao@intel.com Date: Thu, 16 Nov 2017 13:21:09 +0000 Message-Id: <20171116132110.11060-1-pete@akeo.ie> X-Mailer: git-send-email 2.9.3.windows.2 Subject: [PATCH 0/1] Update for the 2017-10-18 VS2017 proposal X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 16 Nov 2017 13:17:12 -0000 The following is a set of changes that I'd like to see applied to Liming's 2017-10-18 VS2017 support proposal for IA32 and X64. (See https://lists.01.org/pipermail/edk2-devel/2017-October/016175.html) Since I am planning to submit a v2 of my VS2017 ARM/AARCH64 proposal, that relies on these alterations, I am sending this preliminary patch for review. The 3 main changes that are being introduced from the existing proposal are: 1. The use of %WindowsSdkVerBinPath% if defined, to locate the most recent version of the Windows 10 SDK. The fact that vcvars32.bat should have been called should usually ensure that this variable has been set, which in turn should ensure that we can access the required ARM64 tools and libraries. Of course, if the variable is not defined, the fallback to the hardcoded SDK top directory should still work fine for IA32 and X64. 2. Follow the way Microsoft structured the Visual Studio 2017 toolchains by introducing a specific VS2017_BIN_HOST, for host-specific elements, that is separate from the various VS2017_BIN_ targets. I believe this will make maintenance easier in the long run, as it makes the breakdown of what relates to the host and what relates to the target a lot more explicit. Note that, for clarity's sake, and unlike the convention that was used previously, I chose to add an underscore between BIN and . This is because, once we start dealing with things like VS2017_BIN_ARM and VS2017_BIN_AARCH64, we'll probably be happier with a separator for . 3. Factorize the elements that are HOST related, since they don't need to be duplicated. I'm hoping that these alterations can be agreed on. If that is the case, and considering that these changes need to be applied on top of the current VS2017 proposal, I can submit a v2 of Liming's patches, that includes these changes, for review and integration. I should also point out that, outside of these modification, the current proposal looks good to me especially as I have tested the generation of the IA32 and X64 OVMF using VS2017 (from a regular command prompt rather than a VS2017 one), as well as a handful of IA32 and X64 applications and found no issues. For the sake of validating my proposed changes, I tested both the x86 and x64 VS2017 HOSTs. Regards, /Pete Pete Batard (1): BaseTools: Use VS2017 SDK path if defined and reorganize variables BaseTools/Conf/tools_def.template | 66 ++++++++++++++++++--------------------- BaseTools/set_vsprefix_envs.bat | 15 ++++----- 2 files changed, 39 insertions(+), 42 deletions(-) -- 2.9.3.windows.2