[edk2-devel] File in the build question. I'm working on a smart git grep

[Andrew Fish via groups.io]

> On Jul 20, 2020, at 6:18 PM, Jiang, Guomin <guomin.jiang at intel.com> wrote:
> 
> I would like that add BaseTols/Scripts/ to the PATH.
> 
> Usually, I use ```set PATH=%PATH%;%WORKSPACE%\BaseTools\Scripts``` to add new directory to the environment variable.
> 
> But I suggest that we should judge if the last character in PATH variable is ';' character and then add new directory.
> 
> Let me know what I can help you.

I’m thinking I could file a bugzilla to add BaseTools/Scripts/ to the path for edksetup.sh/edksetup.bat and then I can just contribute my Python git extension to BaseTools/Scripts

As Laszlo pointed out there is generic value having  BaseTools/Scripts/ in the path so I filed: https://bugzilla.tianocore.org/show_bug.cgi?id=2859

Thanks,

Andrew Fish

> 
>> -----Original Message-----
>> From: devel at edk2.groups.io <mailto:devel at edk2.groups.io> <devel at edk2.groups.io <mailto:devel at edk2.groups.io>> On Behalf Of Laszlo
>> Ersek
>> Sent: Tuesday, July 21, 2020 3:55 AM
>> To: Andrew Fish <afish at apple.com <mailto:afish at apple.com>>; edk2-devel-groups-io
>> <devel at edk2.groups.io <mailto:devel at edk2.groups.io>>
>> Subject: Re: [edk2-devel] File in the build question. I'm working on a smart git
>> grep
>> 
>> On 07/18/20 22:23, Andrew Fish wrote:
>>> Wanted to see if people are OK with adding BaseTools/Scripts/ to the path
>> when you run `. edksetuip.sh` or edksetup.bat?
>>> 
>>> If we do that and I add BaseTools/Scripts/git-pgrep then it will show up as
>> an extended git command.
>>> 
>>> I was thinking over time we could add git-edk2* git commands to help
>> automate the process or maintainer workflow.
>>> 
>>> If people are OK with adding the path I can make a patch set to contribute
>> git-pgrep, but I need some help with how to add the new path to
>> edksetup.bat.
>> 
>> To be honest, I think edksetup.sh should add BaseTools/Scripts/ to the PATH
>> environment variable anyway. I find it a bit awkward to run
>> 
>>  python BaseTools/Scripts/GetMaintainer.py -l ...
>> 
>> rather than just
>> 
>>  GetMaintainer.py -l ...
>> 
>> Thanks
>> Laszlo
>> 
>> 
>> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#62935): https://edk2.groups.io/g/devel/message/62935
Mute This Topic: https://groups.io/mt/75533926/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20200720/6964c024/attachment.htm>