Re: [edk2-devel] [PATCH] Simplify edksetup.sh

2019-07-15 Thread rebecca
On 2019-07-15 11:45, Laszlo Ersek wrote: > > Fair enough, as long as you don't insist on my Reviewed-by in > particular. :) I won't block the patch just because of this, but I also > won't try to decipher changes made for six different goals from each other. Not six different goals: *one* goal, o

Re: [edk2-devel] [PATCH] Simplify edksetup.sh

2019-07-15 Thread Laszlo Ersek
On 07/15/19 17:01, Rebecca Cran wrote: > On 2019-07-15 08:55, Leif Lindholm wrote: >> >> For me, the question is more with being able to trivially discern >> which patch does what. I agree they're all individually trivial, but >> as a single patch there is enough going on at once that it makes it >

Re: [edk2-devel] [PATCH] Simplify edksetup.sh

2019-07-15 Thread Laszlo Ersek
On 07/15/19 16:40, Rebecca Cran wrote: > On 2019-07-12 16:21, Laszlo Ersek wrote: >> >> As long as my opinion counts... (and I totally don't insist that it do): >> the above task list will make for a nice 6-part patch series. :) >> >> (When someone is tempted to capture a *list* of changes in a sin

Re: [edk2-devel] [PATCH] Simplify edksetup.sh

2019-07-15 Thread rebecca
On 2019-07-15 08:55, Leif Lindholm wrote: > > For me, the question is more with being able to trivially discern > which patch does what. I agree they're all individually trivial, but > as a single patch there is enough going on at once that it makes it > easier for bugs to slip through review. (And

Re: [edk2-devel] [PATCH] Simplify edksetup.sh

2019-07-15 Thread Leif Lindholm
On Mon, Jul 15, 2019 at 08:40:21AM -0600, Rebecca Cran wrote: > On 2019-07-12 16:21, Laszlo Ersek wrote: > > > > As long as my opinion counts... (and I totally don't insist that it do): > > the above task list will make for a nice 6-part patch series. :) > > > > (When someone is tempted to capture

Re: [edk2-devel] [PATCH] Simplify edksetup.sh

2019-07-15 Thread rebecca
On 2019-07-12 16:21, Laszlo Ersek wrote: > > As long as my opinion counts... (and I totally don't insist that it do): > the above task list will make for a nice 6-part patch series. :) > > (When someone is tempted to capture a *list* of changes in a single > commit message, that frequently indicate

Re: [edk2-devel] [PATCH] Simplify edksetup.sh

2019-07-12 Thread Laszlo Ersek
Hi, On 07/10/19 23:17, Rebecca Cran wrote: > o Use '$SCRIPTNAME' consistently instead of 'edksetup.sh' > > o Use the bash environment variable $PWD instead of executing the pwd > command. > > o Add quotes around variables to ensure they're evaluated correctly. > > o Simplify SetupPython3() an

Re: [edk2-devel] [PATCH] Simplify edksetup.sh

2019-07-12 Thread rebecca
On 2019-07-10 15:17, Rebecca Cran wrote: > o Use '$SCRIPTNAME' consistently instead of 'edksetup.sh' > > o Use the bash environment variable $PWD instead of executing the pwd > command. > > o Add quotes around variables to ensure they're evaluated correctly. > > o Simplify SetupPython3() and Setu

[edk2-devel] [PATCH] Simplify edksetup.sh

2019-07-10 Thread Rebecca Cran
o Use '$SCRIPTNAME' consistently instead of 'edksetup.sh' o Use the bash environment variable $PWD instead of executing the pwd command. o Add quotes around variables to ensure they're evaluated correctly. o Simplify SetupPython3() and SetupPython() functions. On Linux, "whereis" matches pyt