public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* Official way to build BaseTools - Edk2ToolsBuild.py?
@ 2020-05-20  3:23 Rebecca Cran
  2020-05-20  3:36 ` [edk2-devel] " Sean
  0 siblings, 1 reply; 3+ messages in thread
From: Rebecca Cran @ 2020-05-20  3:23 UTC (permalink / raw)
  To: edk2-devel-groups-io

I'm wondering if using BaseTools/Edk2ToolsBuild.py will become the 
official/standard way users are expected to build BaseTools? If so, 
there are a few problems that I'd like to see fixed, which I'll see if I 
can find some time to work on.


For example: on Linux, running it without arguments gives "ValueError: 
Bad VC" - because it defaults to the VS2017 toolchain.

If you break something in BaseTools and so it fails to build, it prints 
"Exception: Failed to build." with no further details.


I'd like to improve it so it detects being run on a non-Windows platform 
and tries to use GCC, Clang etc. - and perhaps also to add a '--verbose' 
argument that can be used to give a more specific error message when it 
fails.


-- 
Rebecca Cran



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [edk2-devel] Official way to build BaseTools - Edk2ToolsBuild.py?
  2020-05-20  3:23 Official way to build BaseTools - Edk2ToolsBuild.py? Rebecca Cran
@ 2020-05-20  3:36 ` Sean
  2020-05-20  8:42   ` Philippe Mathieu-Daudé
  0 siblings, 1 reply; 3+ messages in thread
From: Sean @ 2020-05-20  3:36 UTC (permalink / raw)
  To: devel, rebecca

I am not sure what the community wants to do with it.  It was created 
for the CI build so it is tailored to the needs of the CI build but I 
have no problem with updates.

I agree with your feedback and see no reason either of those would be a 
problem for the CI use case.

Although not convenient for local builds it does currently output full 
build log to BaseTools/BaseToolsBuild/BASETOOLS_BUILD.txt.

Thanks
Sean






On 5/19/2020 8:23 PM, Rebecca Cran wrote:
> I'm wondering if using BaseTools/Edk2ToolsBuild.py will become the 
> official/standard way users are expected to build BaseTools? If so, 
> there are a few problems that I'd like to see fixed, which I'll see if I 
> can find some time to work on.
> 
> 
> For example: on Linux, running it without arguments gives "ValueError: 
> Bad VC" - because it defaults to the VS2017 toolchain.
> 
> If you break something in BaseTools and so it fails to build, it prints 
> "Exception: Failed to build." with no further details.
> 
> 
> I'd like to improve it so it detects being run on a non-Windows platform 
> and tries to use GCC, Clang etc. - and perhaps also to add a '--verbose' 
> argument that can be used to give a more specific error message when it 
> fails.
> 
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [edk2-devel] Official way to build BaseTools - Edk2ToolsBuild.py?
  2020-05-20  3:36 ` [edk2-devel] " Sean
@ 2020-05-20  8:42   ` Philippe Mathieu-Daudé
  0 siblings, 0 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-05-20  8:42 UTC (permalink / raw)
  To: devel, spbrogan, rebecca

Hi Sean,

On 5/20/20 5:36 AM, Sean wrote:
> I am not sure what the community wants to do with it.  It was created 
> for the CI build so it is tailored to the needs of the CI build but I 
> have no problem with updates.

FWIW I find it useful to reproduce CI failure locally and debug them 
(usually offline, with no access to CI for some time).

> 
> I agree with your feedback and see no reason either of those would be a 
> problem for the CI use case.
> 
> Although not convenient for local builds it does currently output full 
> build log to BaseTools/BaseToolsBuild/BASETOOLS_BUILD.txt.
> 
> Thanks
> Sean
> 
> 
> 
> 
> 
> 
> On 5/19/2020 8:23 PM, Rebecca Cran wrote:
>> I'm wondering if using BaseTools/Edk2ToolsBuild.py will become the 
>> official/standard way users are expected to build BaseTools? If so, 
>> there are a few problems that I'd like to see fixed, which I'll see if 
>> I can find some time to work on.
>>
>>
>> For example: on Linux, running it without arguments gives "ValueError: 
>> Bad VC" - because it defaults to the VS2017 toolchain.
>>
>> If you break something in BaseTools and so it fails to build, it 
>> prints "Exception: Failed to build." with no further details.
>>
>>
>> I'd like to improve it so it detects being run on a non-Windows 
>> platform and tries to use GCC, Clang etc. - and perhaps also to add a 
>> '--verbose' argument that can be used to give a more specific error 
>> message when it fails.
>>
>>
> 
> 
> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-05-20  8:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-05-20  3:23 Official way to build BaseTools - Edk2ToolsBuild.py? Rebecca Cran
2020-05-20  3:36 ` [edk2-devel] " Sean
2020-05-20  8:42   ` Philippe Mathieu-Daudé

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox