[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [PATCH 1/2] Allow user to specify desired build architecture


  • To: Nicholas Tsirakis <niko.tsirakis@xxxxxxxxx>, "win-pv-devel@xxxxxxxxxxxxxxxxxxxx" <win-pv-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Owen Smith <owen.smith@xxxxxxxxxx>
  • Date: Mon, 9 Nov 2020 08:07:52 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=XXNtx4HZIH1s/GlQQJyv45y1wFzMdYuhdlSQbNvHw2Q=; b=BizfD2DI8+ufOvelwk+ufMxiUEfTduq6Lc6m70FUbI6Z46lfkxoPWL/IZNP+SH0t40E2HD4ID/UDDPFd9W4+MZwDj3uUrDrgJzmnxIbYNCg5H+YrIxVFNHzRvKXB1T5t3ant+HI1BWMHTT/DodelqIrU4sm7mJ+feidXsLfUXejCTLU2/nsU4G1oBbUpCkIMbl7MYMb1refyQseO0muVHsO5pbDbrImqJumPyN9OTulQ1ye/BHaJlbgjOF4auAwTquZ8Nno38cNnLwrymlUOmYbxncdT6T22YFgiXnpdNgn6+OVsHla24xVIrAcH74UtnJ7zBaqo2ppIDVQiXDstxg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Tg0D3Eqku2A2vh+iApATRBtE5sVrEmJalTbbOvNCU9f5+74sL9e5wbc/d7z0Z2ZbFCY61jNTuO+BPpgoyT8S8ZS71scJgwBksUnxC7EtcLIVtgfcMfGL8txvaX9ghQ46IKXtX4kf+Dnk45mwzJRL7RJhkOMk+wUNZ+Vp9tJOuv9XEGkaBWS5OUflxJKtsU+WHSu5xze1YgTlN+slJFkPcTUCgNQtjjOIMxMwj4jxh2qR3o88rvzwc7vGds2PgSDp5wfcYHRCS6lbpp73OuMIrwwSZkHUiyb86wqXrWeB+TgLgJSf3zD4n8+24R7QbnuZMArIyikEuIKghkPh9E7KJw==
  • Authentication-results: esa1.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: "paul@xxxxxxx" <paul@xxxxxxx>, Ben Chalmers <ben.chalmers@xxxxxxxxxx>, Nicholas Tsirakis <tsirakisn@xxxxxxxxxxxx>
  • Delivery-date: Mon, 09 Nov 2020 08:08:03 +0000
  • Ironport-sdr: 3i5+QI52im9pG3m0N7VRDxdfK1TFvO0lqxmPdmcRqOxxVDQ2Q42OAVVr7lVJdKNVhBc+UONqQ1 ZncVukkDMA3iNMs4UK38niRI7bgGejJ3HV/NF5p5r5LrKdw48W78CQ4pC/4XK9zqiy+8brA3Ls dlpZ4lj2AzhJgv92QOil1ZUobBr6Q0LQo7Z6lXxRhkTn3xxGGy99ejKuWFw9mzmUbXqZ4cJGST erJpo+RI0HN5XLtvnDOwEMkAjaTZJ4zFP+6Ku6DBuFzSkiwoTMWsCEOIgLz+LgKSU63RGaS51n +Tg=
  • List-id: Developer list for the Windows PV Drivers subproject <win-pv-devel.lists.xenproject.org>
  • Thread-index: AQHWtH5BJf25pae+0UWo81C4mchDeam/dNoQ
  • Thread-topic: [PATCH 1/2] Allow user to specify desired build architecture

I would suggest using 
[string]::IsNullOrEmpty($Arch)
Instead of 
$Arch -eq ""

Owen

> -----Original Message-----
> From: Nicholas Tsirakis <niko.tsirakis@xxxxxxxxx>
> Sent: 06 November 2020 20:49
> To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: paul@xxxxxxx; Ben Chalmers <ben.chalmers@xxxxxxxxxx>; Owen Smith
> <owen.smith@xxxxxxxxxx>; Nicholas Tsirakis <tsirakisn@xxxxxxxxxxxx>
> Subject: [PATCH 1/2] Allow user to specify desired build architecture
> 
> [CAUTION - EXTERNAL EMAIL] DO NOT reply, click links, or open attachments
> unless you have verified the sender and know the content is safe.
> 
> Often times we only need to build a driver for a single targeted architecture.
> Continue to build both by default, but allow the user to specify one if 
> desired.
> 
> Signed-off-by: Nicholas Tsirakis <tsirakisn@xxxxxxxxxxxx>
> ---
>  build.ps1 | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/build.ps1 b/build.ps1
> index c66ac05..06626fd 100644
> --- a/build.ps1
> +++ b/build.ps1
> @@ -5,6 +5,7 @@
>  param(
>       [Parameter(Mandatory = $true)]
>       [string]$Type,
> +     [string]$Arch,
>       [switch]$Sdv
>  )
> 
> @@ -82,8 +83,13 @@ Set-Item -Path Env:MAJOR_VERSION -Value '9'
>  Set-Item -Path Env:MINOR_VERSION -Value '1'
>  Set-Item -Path Env:MICRO_VERSION -Value '0'
> 
> -Build "x86" $Type
> -Build "x64" $Type
> +if ($Arch -eq "" -or $Arch -eq "x86" -or $Arch -eq "Win32") {
> +     Build "x86" $Type
> +}
> +
> +if ($Arch -eq "" -or $Arch -eq "x64") {
> +     Build "x64" $Type
> +}
> 
>  if ($Sdv) {
>       SdvBuild
> --
> 2.25.4
> 




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.