From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-io0-x231.google.com (mail-io0-x231.google.com [IPv6:2607:f8b0:4001:c06::231]) (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 7BC3721A07AB5 for ; Thu, 22 Jun 2017 08:55:46 -0700 (PDT) Received: by mail-io0-x231.google.com with SMTP id c201so25571396ioe.1 for ; Thu, 22 Jun 2017 08:57:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=KG4b4/EuIfJffq00vrKZ3+FGFVXPkAY4qDiJTKOapxc=; b=WgrW0eDiSj/v7vubslSzaxNCYcCCMZX56Vikb2jyeQYw+a/tq68TynYaA6vDNYXWda Zc3XR+LhC9zxjngETlrEkiCDy37WbeuvVg8VaCQU/xJOPvO0eqGV4X3aWxbrWeHPPGi5 puqkbAdO7OQDZCX8u+OVnOxhrsVTUCA/0lUOo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=KG4b4/EuIfJffq00vrKZ3+FGFVXPkAY4qDiJTKOapxc=; b=lFqzOqK0ixgzkCl5kFZCgY29vdYnHuaIexH04mwy+1QSZHSk+2eY9erJXZdB7fVyxa lblEBrcA967iaeu1RTtKFnr+KtAYO/qE5NzPwmB3eo3X2TnT85tphkJek3TomDsWUwNq 5ZZK+3YBKBblfAtR/7Q8tK2NkmFBx53CwvPU0NYlmaLBW/QVCik2v5HskZiquYq/t7yM aBYfZ6K6HEyETdS6lF3WIw53P7qlwk+sUud8GiQ9cw3EJb6onVn9oB8f3HEhPmCKGb1R mEd1ZcDEX10kA5wLFadRXDdUWDVTePIaXkrFCkPGYtJUFC8bBRIkAnudO8KMVYvNqemr 6QNA== X-Gm-Message-State: AKS2vOylmm/DDCSRfYcE9G8IKH74KcsW+BFJLsSOx4T5+wGWrs7hfVS0 zpoQBtQHBWbbQ9b+lUyHBDA6ws0fLHrI X-Received: by 10.107.63.139 with SMTP id m133mr3102321ioa.87.1498147030850; Thu, 22 Jun 2017 08:57:10 -0700 (PDT) MIME-Version: 1.0 Received: by 10.107.164.76 with HTTP; Thu, 22 Jun 2017 08:57:10 -0700 (PDT) In-Reply-To: <20170622124918.GJ26676@bivouac.eciton.net> References: <20170503225539.GQ1657@bivouac.eciton.net> <20170621174456.GH26676@bivouac.eciton.net> <20170622124918.GJ26676@bivouac.eciton.net> From: Ard Biesheuvel Date: Thu, 22 Jun 2017 15:57:10 +0000 Message-ID: To: Leif Lindholm Cc: "edk2-devel@lists.01.org" , Andrew Fish , "Kinney, Michael D" , "Richardson, Brian" , Graeme Gregory , Linaro UEFI , Ryan Harkin , "haojian.zhuang@linaro.org" , Chenhui Sun , Heyi Guo , "Duran, Leo" , Marcin Wojtas , Evan Lloyd , Alan Ott Subject: Re: [RFC] migration of OpenPlatformPkg to tianocore 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, 22 Jun 2017 15:55:46 -0000 Content-Type: text/plain; charset="UTF-8" On 22 June 2017 at 12:49, Leif Lindholm wrote: > On Thu, Jun 22, 2017 at 11:46:16AM +0000, Ard Biesheuvel wrote: >> On 22 June 2017 at 11:39, Ard Biesheuvel wrote: >> > On 21 June 2017 at 17:44, Leif Lindholm wrote: >> >> Hi all, >> >> >> >> I have now pushed updated versions of devel-OpenPlatformPkg branches >> >> to edk2-platforms and edk2-non-osi. I now also have a Readme.md at the >> >> top-level of edk2-platforms describing >> >> >> >> All the silly hacks are gone, and all platforms now build. >> >> >> >> I would really appreciate if people could have a look (and a play). >> >> >> > >> > Cello does not build atm in the new arrangement: >> > >> > build.py... >> > : error 000E: File/directory not found in workspace >> > OpenPlatformPkg/Drivers/Xhci/RenesasFirmwarePD720202/RenesasFirmwarePD720202.inf >> > is not found in packages path: >> > /home/ard/build/edk2/../edk2-platforms >> > /home/ard/build/edk2/../edk2-non-osi >> >> I also think the new PACKAGES_PATH breaks the inclusion of Dsdt.hex: >> >> /home/ard/build/edk2-platforms/Silicon/AMD/Styx/AcpiTables/Dsdt.c:22:20: >> fatal error: Dsdt.hex: No such file or directory >> #include >> ^ >> compilation terminated. >> >> which resides at >> >> Build/Cello/DEBUG_GCC5/AARCH64/edk2-platforms/Silicon/AMD/Styx/AcpiTables/AcpiAml/OUTPUT/Dsdt.hex >> >> but the include is defined as >> >> GCC:*_*_AARCH64_PLATFORM_FLAGS = >> -I$(BIN_DIR)/Silicon/AMD/Styx/AcpiTables/AcpiAml/OUTPUT > > You're not reading the documentation ;) (Have a glance at Readme.md.) > > Yes, this is the behaviour I had before this latest reshuffle. > Mike's proposed (and functional) workaround is to not have "edk2" as > your WORKSPACE, but a common superdirectory of edk2, edk2-platforms > and edk2-non-osi. > > Hopefully at some point we can get rid of this restriction, but for > now it's sort of necessary. > Thanks for clearing that up. And as it turns out, WORKSPACE is not required to be a parent directory to the core EDK2 checkout or any of the alternative sources listed in PACKAGES_PATH, which is actually quite convenient.