> Your argumentation why to introduce another test framework and why one
> can't replace the other should go here as well. To some degree it was in
> the cover letter, but here it would be easier to find later on via git. 

Sure, I will extend the commit description in V2.

> BUT: You do not explain to us anywhere why installing bats from a
> distro, Debian or Ubuntu in CI, is not sufficient. Why do we need to
> import it?

Older versions of Debian and Ubuntu ship the discontinued version [1], while 
newer versions have switched to "the" fork
[2]. However, we would still be missing the following packages:

> > - bats-support [3]: CC0-1.0
> > - bats-assert [4]: CC0-1.0

They are available on Arch Linux, but not on Ubuntu/Debian.

I could adjust the tests and get rid of [3] and [4] (I guess we wouldn't miss 
out on that much), thus allowing us to get
rid of [1] as well.

Personally, I don't mind either way. We already depend on a git submodule for 
testing purposes, which is why I didn't
hesitate to introduce a few more. Pandora's box had already been opened ;)

Let me know what you think and I'll proceed accordingly.

Kind regards,
        Michael

[1] https://github.com/sstephenson/bats
[2] https://github.com/bats-core/bats-core

-- 
Michael Adler

Siemens AG
T RDA IOT SES-DE
Otto-Hahn-Ring 6
81739 München, Deutschland

Siemens Aktiengesellschaft: Vorsitzender des Aufsichtsrats: Jim Hagemann Snabe; 
Vorstand: Roland Busch, Vorsitzender; Klaus Helmrich, Cedrik Neike, Matthias 
Rebellius, Ralf P. Thomas, Judith Wiese; Sitz der Gesellschaft: Berlin und 
München, Deutschland; Registergericht: Berlin-Charlottenburg, HRB 12300, 
München, HRB 6684; WEEE-Reg.-Nr. DE 23691322

-- 
You received this message because you are subscribed to the Google Groups "EFI 
Boot Guard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/efibootguard-dev/20211021065550.w6hgisxbq6dtnrdx%40kratos.

Reply via email to