Sure. Is this the forked repo you mean? tianocore/edk2-basetools: EDK II BaseTools Python tools as a PIP module (github.com) Or this one: matthewfcarlson/edk2: EDK II (github.com)

Hey Matthew! I've found Lazlo's guide to be incredibly helpful https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-unkempt-git-guide-for-edk2-contributors-and-maintainers I also ran into si

Thanks for catching this!

Hey Liming! I added the two reviewed by and pushed to the PR here: https://github.com/tianocore/edk2/pull/933 Feel free to use that PR to merge in

Just pinging this thread to see what needs to get done next. Thank you Liming for the reviewed by on the MdeModulePkg changes.

Will do.

You are correct, I'll fix it in v6

I'll fix this for v6. Good spot.

Thank you for the helpful comments Lazlo! I sent out an updated series (v5) that fixes the things you mentioned. I added more description, so hopefully that helps. Sorry I didn't notice BhyvePkg, I th

Thanks Guomin! I sent out a version 5 of this series.

I included a null lib and I was including the wrong version of Rng anyway. I think that would make sense to split them into two different DSC files. Typo fixed, good spot!

Hey Mike! Thanks for the comments. I've addressed the feedback below. 1. All good comments. I think I've addressed them all. New patch series should be coming soon. 2. This is a problem for BaseCryptL

I sent out a new patch series. Not sure, it might be waiting in the

Great point. I'll fix this.

Thanks Mike!

Based on this RFC: https://edk2.groups.io/g/rfc/topic/rfc_basetools_source_python/74009714?p=,,,20,0,0,0::recentpostdate%2Fsticky,,,20,2,0,74009714 We'd like to request a repository be made under tian

I think you've got it. The version of basetools will be carried via a pip-requirements file. Where would "pip install -r pip-requirements.txt" *fetch* the required basetools version from? I believe th

Hey Lazlo, This is a great point of discussion. Just to make sure we're on the same page, let me paraphrase your scenario. Two features at the same time or a improperly tested new feature were committ

So it's not required by OpenSSL, it's required by the compiler whenever floating point is used, which can be in multiple places. For example, this is used in mu_plus (the Microsoft UEFI value add to E