Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use MdeModulePkg BDS in OVMF #62

Closed
jljusten opened this issue Mar 3, 2016 · 6 comments
Closed

Use MdeModulePkg BDS in OVMF #62

jljusten opened this issue Mar 3, 2016 · 6 comments

Comments

@jljusten
Copy link
Member

jljusten commented Mar 3, 2016

OVMF should migrate to use MdeModulePkg/Universal/BdsDxe rather than IntelFrameworkModulePkg/Universal/BdsDxe.
@lersek
Copy link
Member

lersek commented Mar 3, 2016

See the previous discussion at:

http://thread.gmane.org/gmane.comp.bios.edk2.devel/759

@lersek
Copy link
Member

lersek commented Apr 25, 2016

v3 series from @niruiyu : http://thread.gmane.org/gmane.comp.bios.edk2.devel/10859/focus=11032

@lersek
Copy link
Member

lersek commented May 2, 2016

v4 series from Ray: http://news.gmane.org/find-root.php?message_id=1461913264-137700-1-git-send-email-ruiyu.ni@intel.com

niruiyu added a commit to niruiyu/edk2 that referenced this issue May 3, 2016
@niruiyu
OvmfPkg: Use MdeModulePkg/BDS
da84640 
By default the new MdeModulePkg/BDS is used.
If USE_OLD_BDS is defined to TRUE, IntelFrameworkModulePkg/BDS
is used.

Fixes: tianocore#62

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
@lersek
Copy link
Member

lersek commented May 3, 2016

v5 series from Ray: http://thread.gmane.org/gmane.comp.bios.edk2.devel/11566

@niruiyu niruiyu closed this as completed in 79c098b May 4, 2016
@lersek lersek mentioned this issue May 4, 2016
Closed
@lersek
Copy link
Member

lersek commented May 4, 2016

To be clear, the commit range for the full series is d0a23f9..1a4232b.

@lersek
Copy link
Member

lersek commented Jul 21, 2016

This (closed) issue has been manually migrated to
https://tianocore.acgmultimedia.com/show_bug.cgi?id=81

gguo11837463 added a commit to gguo11837463/edk2 that referenced this issue Apr 27, 2023
@gguo11837463
.azurepipelines: Choose container that have installed lcov
5da3076 
Azure should install code coverage tool (lcov), it didn't
exist on Fedora and Ubuntu by default.

Update docker setting, pick below solution between 47addc9 and 3b3eb8f
3b3eb8f Fixes and improvements to dev containers (tianocore#69)
54e5bd1 Enable GTK on Fedora QEMU (tianocore#63)
f1c7a20 Fedora: install code coverage tools for GCC (tianocore#62)
2ce82af Ubuntu-22: Add initial Ubuntu-22 image (tianocore#61)
14d2aba Add Fedora 37 image with gcc12 (tianocore#60)
5b8a008 Add dotnet runtime to fedora build (tianocore#57)
f5c874a Fix platform build file name for EDK2 change (tianocore#58)
48540ad Ubuntu-20: Fix dev image entrypoint (tianocore#55)
98e849d Fedora-35: Add Powershell to build image (tianocore#52)

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Michael Kubacki <mikuback@linux.microsoft.com>
Cc: Oliver Steffen <osteffen@redhat.com>
Cc: Chris Fernald <chfernal@microsoft.com>
Signed-off-by: Gua Guo <gua.guo@intel.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Michael Kubacki <mikuback@linux.microsoft.com>
Reviewed-by: Chris Fernald <chfernal@microsoft.com>
gguo11837463 added a commit to gguo11837463/edk2 that referenced this issue Apr 27, 2023
@gguo11837463
.azurepipelines: Choose container that have installed lcov
cbbffc9 
Azure should install code coverage tool (lcov), it didn't
exist on Fedora and Ubuntu by default.

Update docker setting, pick below solution between 47addc9 and 3b3eb8f
3b3eb8f Fixes and improvements to dev containers (tianocore#69)
54e5bd1 Enable GTK on Fedora QEMU (tianocore#63)
f1c7a20 Fedora: install code coverage tools for GCC (tianocore#62)
2ce82af Ubuntu-22: Add initial Ubuntu-22 image (tianocore#61)
14d2aba Add Fedora 37 image with gcc12 (tianocore#60)
5b8a008 Add dotnet runtime to fedora build (tianocore#57)
f5c874a Fix platform build file name for EDK2 change (tianocore#58)
48540ad Ubuntu-20: Fix dev image entrypoint (tianocore#55)
98e849d Fedora-35: Add Powershell to build image (tianocore#52)

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Michael Kubacki <mikuback@linux.microsoft.com>
Cc: Oliver Steffen <osteffen@redhat.com>
Cc: Chris Fernald <chfernal@microsoft.com>
Signed-off-by: Gua Guo <gua.guo@intel.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Michael Kubacki <mikuback@linux.microsoft.com>
Reviewed-by: Chris Fernald <chfernal@microsoft.com>
gguo11837463 added a commit to gguo11837463/edk2 that referenced this issue Apr 27, 2023
@gguo11837463
.azurepipelines: Choose container that have installed lcov
0b7449f 
Azure should install code coverage tool (lcov), it didn't
exist on Fedora and Ubuntu by default.

Update docker setting, pick below solution between 47addc9 and 3b3eb8f
3b3eb8f Fixes and improvements to dev containers (tianocore#69)
54e5bd1 Enable GTK on Fedora QEMU (tianocore#63)
f1c7a20 Fedora: install code coverage tools for GCC (tianocore#62)
2ce82af Ubuntu-22: Add initial Ubuntu-22 image (tianocore#61)
14d2aba Add Fedora 37 image with gcc12 (tianocore#60)
5b8a008 Add dotnet runtime to fedora build (tianocore#57)
f5c874a Fix platform build file name for EDK2 change (tianocore#58)
48540ad Ubuntu-20: Fix dev image entrypoint (tianocore#55)
98e849d Fedora-35: Add Powershell to build image (tianocore#52)

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Michael Kubacki <mikuback@linux.microsoft.com>
Cc: Oliver Steffen <osteffen@redhat.com>
Cc: Chris Fernald <chfernal@microsoft.com>
Signed-off-by: Gua Guo <gua.guo@intel.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Michael Kubacki <mikuback@linux.microsoft.com>
Reviewed-by: Chris Fernald <chfernal@microsoft.com>
mergify bot pushed a commit that referenced this issue Apr 27, 2023
@gguo11837463 @mergify
.azurepipelines: Choose container that have installed lcov
04f8371 
Azure should install code coverage tool (lcov), it didn't
exist on Fedora and Ubuntu by default.

Update docker setting, pick below solution between 47addc9 and 3b3eb8f
3b3eb8f Fixes and improvements to dev containers (#69)
54e5bd1 Enable GTK on Fedora QEMU (#63)
f1c7a20 Fedora: install code coverage tools for GCC (#62)
2ce82af Ubuntu-22: Add initial Ubuntu-22 image (#61)
14d2aba Add Fedora 37 image with gcc12 (#60)
5b8a008 Add dotnet runtime to fedora build (#57)
f5c874a Fix platform build file name for EDK2 change (#58)
48540ad Ubuntu-20: Fix dev image entrypoint (#55)
98e849d Fedora-35: Add Powershell to build image (#52)

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Michael Kubacki <mikuback@linux.microsoft.com>
Cc: Oliver Steffen <osteffen@redhat.com>
Cc: Chris Fernald <chfernal@microsoft.com>
Signed-off-by: Gua Guo <gua.guo@intel.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Michael Kubacki <mikuback@linux.microsoft.com>
Reviewed-by: Chris Fernald <chfernal@microsoft.com>
MrChromebox pushed a commit to MrChromebox/edk2 that referenced this issue Sep 15, 2023
@gguo11837463 @MrChromebox
.azurepipelines: Choose container that have installed lcov
f2dd35c 
Azure should install code coverage tool (lcov), it didn't
exist on Fedora and Ubuntu by default.

Update docker setting, pick below solution between 47addc9 and 3b3eb8f
3b3eb8f Fixes and improvements to dev containers (tianocore#69)
54e5bd1 Enable GTK on Fedora QEMU (tianocore#63)
f1c7a20 Fedora: install code coverage tools for GCC (tianocore#62)
2ce82af Ubuntu-22: Add initial Ubuntu-22 image (tianocore#61)
14d2aba Add Fedora 37 image with gcc12 (tianocore#60)
5b8a008 Add dotnet runtime to fedora build (tianocore#57)
f5c874a Fix platform build file name for EDK2 change (tianocore#58)
48540ad Ubuntu-20: Fix dev image entrypoint (tianocore#55)
98e849d Fedora-35: Add Powershell to build image (tianocore#52)

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Sean Brogan <sean.brogan@microsoft.com>
Cc: Michael Kubacki <mikuback@linux.microsoft.com>
Cc: Oliver Steffen <osteffen@redhat.com>
Cc: Chris Fernald <chfernal@microsoft.com>
Signed-off-by: Gua Guo <gua.guo@intel.com>
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Michael Kubacki <mikuback@linux.microsoft.com>
Reviewed-by: Chris Fernald <chfernal@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jljusten @lersek