Re: [edk2-devel] [PATCH] BaseTools/Scripts: Add scripts to set PACKAGES_PATH environment

2020-04-06 Thread Sean via groups.io
Bob and Ray, My point isn't that there is anything wrong with script.  This script is designed to work with edk2-platforms repo or other repos of similar layout.    This is a "platform" thing that is needed on some platforms (not edk2 core platforms).  To me this means it does't belong in edk2

Re: [edk2-devel] [PATCH] BaseTools/Scripts: Add scripts to set PACKAGES_PATH environment

2020-04-06 Thread Bob Feng
; Luo, Heng Cc: Feng, Bob C Subject: RE: [edk2-devel] [PATCH] BaseTools/Scripts: Add scripts to set PACKAGES_PATH environment Sean, PyTools is good. This patch is to support those platforms that haven’t adopted PyTools for their platform build. Thanks, Ray From: devel@edk2.groups.io

Re: [edk2-devel] [PATCH] BaseTools/Scripts: Add scripts to set PACKAGES_PATH environment

2020-04-06 Thread Ni, Ray
] [PATCH] BaseTools/Scripts: Add scripts to set PACKAGES_PATH environment I am not a fan of this. The community and design meetings have had a few discussions about tools (PyTools has been presented twice) and a few brief discussions about common patterns to build platforms but I don't think

Re: [edk2-devel] [PATCH] BaseTools/Scripts: Add scripts to set PACKAGES_PATH environment

2020-04-02 Thread Sean via groups.io
I am not a fan of this.  The community and design meetings have had a few discussions about tools (PyTools has been presented twice) and a few brief discussions about common patterns to build platforms but I don't think there is real alignment.  Each "platform" has its own way of doing things.

[edk2-devel] [PATCH] BaseTools/Scripts: Add scripts to set PACKAGES_PATH environment

2020-04-02 Thread Heng Luo
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2656 1. Add GetPackagesPath.py, it will be used to get package pathes from special directories. A sub directory is a qualified package path when an EDKII Package can be found under it. 2. Add SetPackagesPath.bat and SetPackagesPath.sh, these