subject:"\[edk2\-devel\] \[PATCH 3\/3\] OvmfwPkg\: Don't exclude XCODE Modules"

Re: [edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

2020-05-26 Thread Andrew Fish via groups.io




> On May 26, 2020, at 4:45 AM, Laszlo Ersek  wrote:
> 
> On 05/26/20 06:10, Andrew Fish wrote:
>> 
>> 
>>> On May 25, 2020, at 12:31 PM, Laszlo Ersek  wrote:
>>> 
>>> Hi Andrew,
>>> 
>>> On 05/24/20 23:20, Andrew Fish via groups.io  wrote:
 With this BZ getting fixed we no longer need to special case XCODE.
 
 Cc: Ard Biesheuvel 
 Cc: Jiewen Yao 
 Cc: Jordan Justen 
 Cc: Philippe Mathieu-Daudé 
 Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=557
 Signed-off-by: Andrew Fish 
 
 Signed-off-by: Andrew Fish 
 ---
 OvmfPkg/OvmfPkgIa32.dsc| 3 +--
 OvmfPkg/OvmfPkgIa32.fdf| 2 --
 OvmfPkg/OvmfPkgIa32X64.dsc | 4 ++--
 OvmfPkg/OvmfPkgIa32X64.fdf | 2 --
 OvmfPkg/OvmfPkgX64.dsc | 3 +--
 OvmfPkg/OvmfPkgX64.fdf | 2 --
 OvmfPkg/OvmfXen.dsc| 3 +--
 OvmfPkg/OvmfXen.fdf| 2 --
 8 files changed, 5 insertions(+), 16 deletions(-)
>>> 
>> 
>> Laszlo,
>> 
>> Thanks for the feedback. 
>> 
>> Can I ask that you go to https://www.tianocore.org, click on How to 
>> Contribute and point me at the chain of links I did not follow, if I missed 
>> it tit is likely due to too many links and too much information being vended.
> 
> You didn't miss anything, starting from that page, I think. I don't
> remember ever clicking "How to Contribute" on that page. :)
> 
> In fact if I grep the current edk2-wiki project source, at commit
> de8fae02bbcc ("Add acknowledgements page", 2020-05-21), I find no
> references to "SetupGit.py".
> 
>> When people are starting out we should vend them the instructions that work 
>> and let them opt in to learning more.
> 
> "Working instructions" is a moving target. When I wrote the unkempt
> guide, it was serious work, I had to set aside resources. When we
> changed the workflow to replace "git-push" (by maintainers) with github
> PRs (to trigger CI), documenting that was again serious work (for Mike
> -- Mike updated both the official workflow article and the unkempt
> guide, as I couldn't volunteer for the latter).
> 
> The more documentation we add, the larger the burden to update them
> grows. It's an on-going commitment. Given the constant scarcity of
> developer (and reviewer) cycles, we can only choose between:
> 
> - "code, plus more or less up-to-date docs", and
> - "code, plus no docs".
> 
> Leif wrote SetupGit.py in the first place to save people the effort of
> going through some of the unkempt guide steps.
> 
> If we decide that no SetupGit.py (or similar utilities) should be
> written without documenting them in the wiki, that won't force
> contributors to document their workflow-related contributions; it will
> cause them to not writing the utilities in the first place.
> 
> Open source development communities teach contributors the workflow by
> doing. For example, I have contributed to two open source projects that
> are *exclusively* managed on github.com, using "github native" pull
> requests and such. (Namely, openssl, and "p11-glue/p11-kit".) I'm the
> kind of guy that very carefully reads the documentation first, and
> starts pushing the buttons only second, and I *still* got wrong my
> initial contributions to both projects.
> 
> With OpenSSL, I missed details of how review worked and details about
> the CLA. Maintainers taught me those bits on the PR, while they were
> reviewing my code.
> 
> With p11-kit, I had missed that I was expected to write a unit test at
> once, for the new code. Maintainers pointed that out in my PR.
> 
> I posit that virtually no up-to-date technical documentation exists
> unless an organization treats that documentation as a *product* (with
> its own resource allocations, technical writers, subject matter expert
> reviewers, project managers, and so on).
> 
>> 
>>> (1) Please run "BaseTools/Scripts/SetupGit.py" in your edk2 clone,
>>> because right now, the patch is formatted/posted with too many CR
>>> characters.
>>> 
>> 
>> I filed https://bugzilla.tianocore.org/show_bug.cgi?id=2767 since I passed 
>> PatchCheck.py but did not run  BaseTools/Scripts/SetupGit.py
> 
> Thanks.
> 
> For the record: I didn't reject your contribution (I'm very happy you
> posted a patch series); instead, I asked for an update.
> 

Lazlo,

I understand that is just a tooling issue on my side. Sorry for my delay in 
fixing up the patches but I have some higher priority work PRs I need to get 
resolved and that is delaying me fixing up the patch set. 

At my work we have a tradition of trying to update our documentation when we 
onboard new people as that new set of eyes helps point out the small things 
that could have saved people a lot of time.  So I'm not so much complaining, 
but just trying to help. 

> In particular, the number of empty lines inserted into the DSC files is
> not consistent across the OVMF DSC files, and that fact has nothing to
> do with git configuration -- it would need fixing identically even if
> the series had been sub

Re: [edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

2020-05-26 Thread Laszlo Ersek

On 05/26/20 06:10, Andrew Fish wrote:
> 
> 
>> On May 25, 2020, at 12:31 PM, Laszlo Ersek  wrote:
>>
>> Hi Andrew,
>>
>> On 05/24/20 23:20, Andrew Fish via groups.io  wrote:
>>> With this BZ getting fixed we no longer need to special case XCODE.
>>>
>>> Cc: Ard Biesheuvel 
>>> Cc: Jiewen Yao 
>>> Cc: Jordan Justen 
>>> Cc: Philippe Mathieu-Daudé 
>>> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=557
>>> Signed-off-by: Andrew Fish 
>>>
>>> Signed-off-by: Andrew Fish 
>>> ---
>>> OvmfPkg/OvmfPkgIa32.dsc| 3 +--
>>> OvmfPkg/OvmfPkgIa32.fdf| 2 --
>>> OvmfPkg/OvmfPkgIa32X64.dsc | 4 ++--
>>> OvmfPkg/OvmfPkgIa32X64.fdf | 2 --
>>> OvmfPkg/OvmfPkgX64.dsc | 3 +--
>>> OvmfPkg/OvmfPkgX64.fdf | 2 --
>>> OvmfPkg/OvmfXen.dsc| 3 +--
>>> OvmfPkg/OvmfXen.fdf| 2 --
>>> 8 files changed, 5 insertions(+), 16 deletions(-)
>>
> 
> Laszlo,
> 
> Thanks for the feedback. 
> 
> Can I ask that you go to https://www.tianocore.org, click on How to 
> Contribute and point me at the chain of links I did not follow, if I missed 
> it tit is likely due to too many links and too much information being vended.

You didn't miss anything, starting from that page, I think. I don't
remember ever clicking "How to Contribute" on that page. :)

In fact if I grep the current edk2-wiki project source, at commit
de8fae02bbcc ("Add acknowledgements page", 2020-05-21), I find no
references to "SetupGit.py".

> When people are starting out we should vend them the instructions that work 
> and let them opt in to learning more.

"Working instructions" is a moving target. When I wrote the unkempt
guide, it was serious work, I had to set aside resources. When we
changed the workflow to replace "git-push" (by maintainers) with github
PRs (to trigger CI), documenting that was again serious work (for Mike
-- Mike updated both the official workflow article and the unkempt
guide, as I couldn't volunteer for the latter).

The more documentation we add, the larger the burden to update them
grows. It's an on-going commitment. Given the constant scarcity of
developer (and reviewer) cycles, we can only choose between:

- "code, plus more or less up-to-date docs", and
- "code, plus no docs".

Leif wrote SetupGit.py in the first place to save people the effort of
going through some of the unkempt guide steps.

If we decide that no SetupGit.py (or similar utilities) should be
written without documenting them in the wiki, that won't force
contributors to document their workflow-related contributions; it will
cause them to not writing the utilities in the first place.

Open source development communities teach contributors the workflow by
doing. For example, I have contributed to two open source projects that
are *exclusively* managed on github.com, using "github native" pull
requests and such. (Namely, openssl, and "p11-glue/p11-kit".) I'm the
kind of guy that very carefully reads the documentation first, and
starts pushing the buttons only second, and I *still* got wrong my
initial contributions to both projects.

With OpenSSL, I missed details of how review worked and details about
the CLA. Maintainers taught me those bits on the PR, while they were
reviewing my code.

With p11-kit, I had missed that I was expected to write a unit test at
once, for the new code. Maintainers pointed that out in my PR.

I posit that virtually no up-to-date technical documentation exists
unless an organization treats that documentation as a *product* (with
its own resource allocations, technical writers, subject matter expert
reviewers, project managers, and so on).

> 
>> (1) Please run "BaseTools/Scripts/SetupGit.py" in your edk2 clone,
>> because right now, the patch is formatted/posted with too many CR
>> characters.
>>
> 
> I filed https://bugzilla.tianocore.org/show_bug.cgi?id=2767 since I passed 
> PatchCheck.py but did not run  BaseTools/Scripts/SetupGit.py

Thanks.

For the record: I didn't reject your contribution (I'm very happy you
posted a patch series); instead, I asked for an update.

In particular, the number of empty lines inserted into the DSC files is
not consistent across the OVMF DSC files, and that fact has nothing to
do with git configuration -- it would need fixing identically even if
the series had been submitted via a github.com PR.

Thanks
Laszlo

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

View/Reply Online (#60270): https://edk2.groups.io/g/devel/message/60270
Mute This Topic: https://groups.io/mt/74445737/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

2020-05-25 Thread Andrew Fish via groups.io



> On May 25, 2020, at 12:31 PM, Laszlo Ersek  wrote:
> 
> Hi Andrew,
> 
> On 05/24/20 23:20, Andrew Fish via groups.io  wrote:
>> With this BZ getting fixed we no longer need to special case XCODE.
>> 
>> Cc: Ard Biesheuvel 
>> Cc: Jiewen Yao 
>> Cc: Jordan Justen 
>> Cc: Philippe Mathieu-Daudé 
>> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=557
>> Signed-off-by: Andrew Fish 
>> 
>> Signed-off-by: Andrew Fish 
>> ---
>> OvmfPkg/OvmfPkgIa32.dsc| 3 +--
>> OvmfPkg/OvmfPkgIa32.fdf| 2 --
>> OvmfPkg/OvmfPkgIa32X64.dsc | 4 ++--
>> OvmfPkg/OvmfPkgIa32X64.fdf | 2 --
>> OvmfPkg/OvmfPkgX64.dsc | 3 +--
>> OvmfPkg/OvmfPkgX64.fdf | 2 --
>> OvmfPkg/OvmfXen.dsc| 3 +--
>> OvmfPkg/OvmfXen.fdf| 2 --
>> 8 files changed, 5 insertions(+), 16 deletions(-)
> 

Laszlo,

Thanks for the feedback. 

Can I ask that you go to https://www.tianocore.org, click on How to Contribute 
and point me at the chain of links I did not follow, if I missed it tit is 
likely due to too many links and too much information being vended. When people 
are starting out we should vend them the instructions that work and let them 
opt in to learning more. 

> (1) Please run "BaseTools/Scripts/SetupGit.py" in your edk2 clone,
> because right now, the patch is formatted/posted with too many CR
> characters.
> 

I filed https://bugzilla.tianocore.org/show_bug.cgi?id=2767 since I passed 
PatchCheck.py but did not run  BaseTools/Scripts/SetupGit.py

Thanks,

Andrew Fish

> (2) I'm not on the CC list -- is that an oversight, or should I have
> received the message directly due to some other means? (I've only found
> this message in my list folder).
> 
> (3) As the diffstat above indicates it too, the
> "OvmfPkg/OvmfPkgIa32X64.dsc" update is inconsistent with the other DSC
> file updates, namely in whitespace.
> 
> In the other DSC files, only one newline is inserted (just before
> Shell.inf), but in "OvmfPkgIa32X64.dsc", two newlines are inserted. One
> before Shell.inf, and another before "TftpDynamicCommand.inf".
> 
> (4) There's a typo in the subject line of this patch; please replace
> "OvmfwPkg" with "OvmfPkg".
> 
> Thank you!
> Laszlo
> 
>> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
>> index d0df9cbbfb2b..8166d1588ae5 100644
>> --- a/OvmfPkg/OvmfPkgIa32.dsc
>> +++ b/OvmfPkg/OvmfPkgIa32.dsc
>> @@ -878,7 +878,6 @@ [Components]
>>   OvmfPkg/Csm/Csm16/Csm16.inf
>> 
>> !endif
>> 
>> 
>> 
>> -!if $(TOOL_CHAIN_TAG) != "XCODE5"
>> 
>>   ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf {
>> 
>> 
>> 
>>   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
>> 
>> @@ -887,7 +886,7 @@ [Components]
>> 
>> 
>>   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
>> 
>>   }
>> 
>> -!endif
>> 
>> +
>> 
>>   ShellPkg/Application/Shell/Shell.inf {
>> 
>> 
>> 
>>   
>> ShellCommandLib|ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf
>> 
>> diff --git a/OvmfPkg/OvmfPkgIa32.fdf b/OvmfPkg/OvmfPkgIa32.fdf
>> index e2b759aa8d05..cb1a8b1eba60 100644
>> --- a/OvmfPkg/OvmfPkgIa32.fdf
>> +++ b/OvmfPkg/OvmfPkgIa32.fdf
>> @@ -288,10 +288,8 @@ [FV.DXEFV]
>> INF  FatPkg/EnhancedFatDxe/Fat.inf
>> 
>> INF  MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf
>> 
>> 
>> 
>> -!if $(TOOL_CHAIN_TAG) != "XCODE5"
>> 
>> INF  ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf
>> 
>> INF  
>> OvmfPkg/LinuxInitrdDynamicShellCommand/LinuxInitrdDynamicShellCommand.inf
>> 
>> -!endif
>> 
>> INF  ShellPkg/Application/Shell/Shell.inf
>> 
>> 
>> 
>> INF MdeModulePkg/Logo/LogoDxe.inf
>> 
>> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
>> index b3ae62fee92b..e60d25db575e 100644
>> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
>> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
>> @@ -892,7 +892,7 @@ [Components.X64]
>>   OvmfPkg/Csm/Csm16/Csm16.inf
>> 
>> !endif
>> 
>> 
>> 
>> -!if $(TOOL_CHAIN_TAG) != "XCODE5"
>> 
>> +
>> 
>>   ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf {
>> 
>> 
>> 
>>   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
>> 
>> @@ -901,7 +901,7 @@ [Components.X64]
>> 
>> 
>>   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
>> 
>>   }
>> 
>> -!endif
>> 
>> +
>> 
>>   ShellPkg/Application/Shell/Shell.inf {
>> 
>> 
>> 
>>   
>> ShellCommandLib|ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf
>> 
>> diff --git a/OvmfPkg/OvmfPkgIa32X64.fdf b/OvmfPkg/OvmfPkgIa32X64.fdf
>> index bfca1eff9e83..4cd4dd652a9f 100644
>> --- a/OvmfPkg/OvmfPkgIa32X64.fdf
>> +++ b/OvmfPkg/OvmfPkgIa32X64.fdf
>> @@ -289,10 +289,8 @@ [FV.DXEFV]
>> INF  FatPkg/EnhancedFatDxe/Fat.inf
>> 
>> INF  MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf
>> 
>> 
>> 
>> -!if $(TOOL_CHAIN_TAG) != "XCODE5"
>> 
>> INF  ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf
>> 
>> INF  
>> OvmfPkg/LinuxInitrdDynamicShellCommand/LinuxInitrdDynamicShellCommand.inf
>> 
>> -!endif
>> 
>> INF  ShellPkg/Applica

Re: [edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

2020-05-25 Thread Laszlo Ersek

Hi Andrew,

On 05/24/20 23:20, Andrew Fish via groups.io wrote:
> With this BZ getting fixed we no longer need to special case XCODE.
> 
> Cc: Ard Biesheuvel 
> Cc: Jiewen Yao 
> Cc: Jordan Justen 
> Cc: Philippe Mathieu-Daudé 
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=557
> Signed-off-by: Andrew Fish 
> 
> Signed-off-by: Andrew Fish 
> ---
>  OvmfPkg/OvmfPkgIa32.dsc| 3 +--
>  OvmfPkg/OvmfPkgIa32.fdf| 2 --
>  OvmfPkg/OvmfPkgIa32X64.dsc | 4 ++--
>  OvmfPkg/OvmfPkgIa32X64.fdf | 2 --
>  OvmfPkg/OvmfPkgX64.dsc | 3 +--
>  OvmfPkg/OvmfPkgX64.fdf | 2 --
>  OvmfPkg/OvmfXen.dsc| 3 +--
>  OvmfPkg/OvmfXen.fdf| 2 --
>  8 files changed, 5 insertions(+), 16 deletions(-)

(1) Please run "BaseTools/Scripts/SetupGit.py" in your edk2 clone,
because right now, the patch is formatted/posted with too many CR
characters.

(2) I'm not on the CC list -- is that an oversight, or should I have
received the message directly due to some other means? (I've only found
this message in my list folder).

(3) As the diffstat above indicates it too, the
"OvmfPkg/OvmfPkgIa32X64.dsc" update is inconsistent with the other DSC
file updates, namely in whitespace.

In the other DSC files, only one newline is inserted (just before
Shell.inf), but in "OvmfPkgIa32X64.dsc", two newlines are inserted. One
before Shell.inf, and another before "TftpDynamicCommand.inf".

(4) There's a typo in the subject line of this patch; please replace
"OvmfwPkg" with "OvmfPkg".

Thank you!
Laszlo

> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
> index d0df9cbbfb2b..8166d1588ae5 100644
> --- a/OvmfPkg/OvmfPkgIa32.dsc
> +++ b/OvmfPkg/OvmfPkgIa32.dsc
> @@ -878,7 +878,6 @@ [Components]
>OvmfPkg/Csm/Csm16/Csm16.inf
> 
>  !endif
> 
>  
> 
> -!if $(TOOL_CHAIN_TAG) != "XCODE5"
> 
>ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf {
> 
>  
> 
>gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
> 
> @@ -887,7 +886,7 @@ [Components]
>  
> 
>gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
> 
>}
> 
> -!endif
> 
> +
> 
>ShellPkg/Application/Shell/Shell.inf {
> 
>  
> 
>
> ShellCommandLib|ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf
> 
> diff --git a/OvmfPkg/OvmfPkgIa32.fdf b/OvmfPkg/OvmfPkgIa32.fdf
> index e2b759aa8d05..cb1a8b1eba60 100644
> --- a/OvmfPkg/OvmfPkgIa32.fdf
> +++ b/OvmfPkg/OvmfPkgIa32.fdf
> @@ -288,10 +288,8 @@ [FV.DXEFV]
>  INF  FatPkg/EnhancedFatDxe/Fat.inf
> 
>  INF  MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf
> 
>  
> 
> -!if $(TOOL_CHAIN_TAG) != "XCODE5"
> 
>  INF  ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf
> 
>  INF  
> OvmfPkg/LinuxInitrdDynamicShellCommand/LinuxInitrdDynamicShellCommand.inf
> 
> -!endif
> 
>  INF  ShellPkg/Application/Shell/Shell.inf
> 
>  
> 
>  INF MdeModulePkg/Logo/LogoDxe.inf
> 
> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
> index b3ae62fee92b..e60d25db575e 100644
> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
> @@ -892,7 +892,7 @@ [Components.X64]
>OvmfPkg/Csm/Csm16/Csm16.inf
> 
>  !endif
> 
>  
> 
> -!if $(TOOL_CHAIN_TAG) != "XCODE5"
> 
> +
> 
>ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf {
> 
>  
> 
>gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
> 
> @@ -901,7 +901,7 @@ [Components.X64]
>  
> 
>gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
> 
>}
> 
> -!endif
> 
> +
> 
>ShellPkg/Application/Shell/Shell.inf {
> 
>  
> 
>
> ShellCommandLib|ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf
> 
> diff --git a/OvmfPkg/OvmfPkgIa32X64.fdf b/OvmfPkg/OvmfPkgIa32X64.fdf
> index bfca1eff9e83..4cd4dd652a9f 100644
> --- a/OvmfPkg/OvmfPkgIa32X64.fdf
> +++ b/OvmfPkg/OvmfPkgIa32X64.fdf
> @@ -289,10 +289,8 @@ [FV.DXEFV]
>  INF  FatPkg/EnhancedFatDxe/Fat.inf
> 
>  INF  MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf
> 
>  
> 
> -!if $(TOOL_CHAIN_TAG) != "XCODE5"
> 
>  INF  ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf
> 
>  INF  
> OvmfPkg/LinuxInitrdDynamicShellCommand/LinuxInitrdDynamicShellCommand.inf
> 
> -!endif
> 
>  INF  ShellPkg/Application/Shell/Shell.inf
> 
>  
> 
>  INF MdeModulePkg/Logo/LogoDxe.inf
> 
> diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
> index f7fe75ebf531..3b46bce5a6e0 100644
> --- a/OvmfPkg/OvmfPkgX64.dsc
> +++ b/OvmfPkg/OvmfPkgX64.dsc
> @@ -888,7 +888,6 @@ [Components]
>OvmfPkg/Csm/Csm16/Csm16.inf
> 
>  !endif
> 
>  
> 
> -!if $(TOOL_CHAIN_TAG) != "XCODE5"
> 
>ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf {
> 
>  
> 
>gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
> 
> @@ -897,7 +896,7 @@ [Components]
>  
> 
>gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE
> 
>}
> 
> -!endif
> 
> +
> 
>ShellPkg/Application/Shell/Shell.inf {
> 
>  
> 
>
> ShellCommandLib|ShellP

[edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

2020-05-24 Thread Andrew Fish via groups.io

With this BZ getting fixed we no longer need to special case XCODE.

Cc: Ard Biesheuvel 
Cc: Jiewen Yao 
Cc: Jordan Justen 
Cc: Philippe Mathieu-Daudé 
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=557
Signed-off-by: Andrew Fish 

Signed-off-by: Andrew Fish 
---
 OvmfPkg/OvmfPkgIa32.dsc| 3 +--
 OvmfPkg/OvmfPkgIa32.fdf| 2 --
 OvmfPkg/OvmfPkgIa32X64.dsc | 4 ++--
 OvmfPkg/OvmfPkgIa32X64.fdf | 2 --
 OvmfPkg/OvmfPkgX64.dsc | 3 +--
 OvmfPkg/OvmfPkgX64.fdf | 2 --
 OvmfPkg/OvmfXen.dsc| 3 +--
 OvmfPkg/OvmfXen.fdf| 2 --
 8 files changed, 5 insertions(+), 16 deletions(-)

diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
index d0df9cbbfb2b..8166d1588ae5 100644
--- a/OvmfPkg/OvmfPkgIa32.dsc
+++ b/OvmfPkg/OvmfPkgIa32.dsc
@@ -878,7 +878,6 @@ [Components]
   OvmfPkg/Csm/Csm16/Csm16.inf

 !endif

 

-!if $(TOOL_CHAIN_TAG) != "XCODE5"

   ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf {

 

   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE

@@ -887,7 +886,7 @@ [Components]
 

   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE

   }

-!endif

+

   ShellPkg/Application/Shell/Shell.inf {

 

   
ShellCommandLib|ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf

diff --git a/OvmfPkg/OvmfPkgIa32.fdf b/OvmfPkg/OvmfPkgIa32.fdf
index e2b759aa8d05..cb1a8b1eba60 100644
--- a/OvmfPkg/OvmfPkgIa32.fdf
+++ b/OvmfPkg/OvmfPkgIa32.fdf
@@ -288,10 +288,8 @@ [FV.DXEFV]
 INF  FatPkg/EnhancedFatDxe/Fat.inf

 INF  MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf

 

-!if $(TOOL_CHAIN_TAG) != "XCODE5"

 INF  ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf

 INF  OvmfPkg/LinuxInitrdDynamicShellCommand/LinuxInitrdDynamicShellCommand.inf

-!endif

 INF  ShellPkg/Application/Shell/Shell.inf

 

 INF MdeModulePkg/Logo/LogoDxe.inf

diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
index b3ae62fee92b..e60d25db575e 100644
--- a/OvmfPkg/OvmfPkgIa32X64.dsc
+++ b/OvmfPkg/OvmfPkgIa32X64.dsc
@@ -892,7 +892,7 @@ [Components.X64]
   OvmfPkg/Csm/Csm16/Csm16.inf

 !endif

 

-!if $(TOOL_CHAIN_TAG) != "XCODE5"

+

   ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf {

 

   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE

@@ -901,7 +901,7 @@ [Components.X64]
 

   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE

   }

-!endif

+

   ShellPkg/Application/Shell/Shell.inf {

 

   
ShellCommandLib|ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf

diff --git a/OvmfPkg/OvmfPkgIa32X64.fdf b/OvmfPkg/OvmfPkgIa32X64.fdf
index bfca1eff9e83..4cd4dd652a9f 100644
--- a/OvmfPkg/OvmfPkgIa32X64.fdf
+++ b/OvmfPkg/OvmfPkgIa32X64.fdf
@@ -289,10 +289,8 @@ [FV.DXEFV]
 INF  FatPkg/EnhancedFatDxe/Fat.inf

 INF  MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf

 

-!if $(TOOL_CHAIN_TAG) != "XCODE5"

 INF  ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf

 INF  OvmfPkg/LinuxInitrdDynamicShellCommand/LinuxInitrdDynamicShellCommand.inf

-!endif

 INF  ShellPkg/Application/Shell/Shell.inf

 

 INF MdeModulePkg/Logo/LogoDxe.inf

diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
index f7fe75ebf531..3b46bce5a6e0 100644
--- a/OvmfPkg/OvmfPkgX64.dsc
+++ b/OvmfPkg/OvmfPkgX64.dsc
@@ -888,7 +888,6 @@ [Components]
   OvmfPkg/Csm/Csm16/Csm16.inf

 !endif

 

-!if $(TOOL_CHAIN_TAG) != "XCODE5"

   ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf {

 

   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE

@@ -897,7 +896,7 @@ [Components]
 

   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE

   }

-!endif

+

   ShellPkg/Application/Shell/Shell.inf {

 

   
ShellCommandLib|ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.inf

diff --git a/OvmfPkg/OvmfPkgX64.fdf b/OvmfPkg/OvmfPkgX64.fdf
index bfca1eff9e83..4cd4dd652a9f 100644
--- a/OvmfPkg/OvmfPkgX64.fdf
+++ b/OvmfPkg/OvmfPkgX64.fdf
@@ -289,10 +289,8 @@ [FV.DXEFV]
 INF  FatPkg/EnhancedFatDxe/Fat.inf

 INF  MdeModulePkg/Universal/Disk/UdfDxe/UdfDxe.inf

 

-!if $(TOOL_CHAIN_TAG) != "XCODE5"

 INF  ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf

 INF  OvmfPkg/LinuxInitrdDynamicShellCommand/LinuxInitrdDynamicShellCommand.inf

-!endif

 INF  ShellPkg/Application/Shell/Shell.inf

 

 INF MdeModulePkg/Logo/LogoDxe.inf

diff --git a/OvmfPkg/OvmfXen.dsc b/OvmfPkg/OvmfXen.dsc
index 3af0ee705407..0c11d4732628 100644
--- a/OvmfPkg/OvmfXen.dsc
+++ b/OvmfPkg/OvmfXen.dsc
@@ -670,7 +670,6 @@ [Components]
   OvmfPkg/Csm/Csm16/Csm16.inf

 !endif

 

-!if $(TOOL_CHAIN_TAG) != "XCODE5"

   ShellPkg/DynamicCommand/TftpDynamicCommand/TftpDynamicCommand.inf {

 

   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE

@@ -679,7 +678,7 @@ [Components]
 

   gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE

   }

-!endif

+

   ShellPkg/Application/Shell/Shell.inf {

 

   
ShellCommandLib|Sh

Re: [edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

Re: [edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

Re: [edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

Re: [edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

[edk2-devel] [PATCH 3/3] OvmfwPkg: Don't exclude XCODE Modules

5 matches

Site Navigation

Mail list logo

Footer information