On 24.05.19 15:30, Peter Robinson wrote:
On Fri, May 24, 2019 at 6:19 AM Heinrich Schuchardt <xypron.g...@gmx.de> wrote:
On 5/24/19 3:02 AM, AKASHI Takahiro wrote:
Heinrich,
On Thu, May 16, 2019 at 07:34:54PM +0200, Heinrich Schuchardt wrote:
In EFI 1.10 a version of the Unicode collation protocol using ISO 639-2
language codes existed. This protocol is not part of the UEFI specification
any longer. Unfortunately it is required to run the UEFI Self Certification
Test (SCT) II, version 2.6, 2017. So we implement it here for the sole
purpose of running the SCT. It can be removed once a compliant SCT is
available.
I remember that Alex always rejected this kind of patch, saying
that an upstream (SCT in this case) should be fixed in the first place.
Do you want to change this policy?
(I'm just asking.)
I am using SCT a lot to test my patches. I want to be able to run the
tests on the final code.
Wouldn't it be better to patch/fork the upstream SCT that blindly
pulling in obsolete code that is basically already dead?
I have raised a ticket for upstream SCT but did not see any reaction up
to now.
Got a reference to the ticket?
Making this deprecated protocol a config option deselected by default is
the most plausible solution to me. In the Kconfig comment I made it
clear that this protocol is going to be removed when the SCT has been
corrected.
I disagree, I think if it's obsolete not having the code in the first
case in the most plausible option IMO.
Since it's hidden behind a default-n config option, I'd not be terribly
opposed to it. I do agree that fixing SCT is the better path forward.
Leif, how long do you think fixing this properly in SCT is going to take?
Alex
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot