public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* Proposal to make EccCheck non-destructive
@ 2020-10-04 23:25 Bret Barkelew
  2020-10-06 10:53 ` [edk2-devel] " Laszlo Ersek
  0 siblings, 1 reply; 2+ messages in thread
From: Bret Barkelew @ 2020-10-04 23:25 UTC (permalink / raw)
  To: devel@edk2.groups.io

[-- Attachment #1: Type: text/plain, Size: 1232 bytes --]

In regards to this bug:
2986 – EccCheck should not revert staged and local changes (tianocore.org)<https://bugzilla.tianocore.org/show_bug.cgi?id=2986>

I have a proposal:

  1.  git diff --name-only <starting_commit>..<ending_commit/HEAD>
  2.  Copy list of changed files to a temp directory under Build/
     *   Perhaps something like Build/EccCheck or Build/EccCheckTemp
  3.  Remove files/directories that are in the exclusion list
  4.  Run EccCheck against that directory
  5.  Filter the report to remove the temp directory path so that the reported errors are root-based paths

The trade off of this approach is that EccCheck will run against multiple patches in aggregate, but – frankly – this isn’t any different than the other CI plugins.

The destructive nature of this plugin has caused headaches for everyone on our team that has worked with the new codebase. We all use local CI to validate changes as we make them, and having to make “test commits” just to ensure our work doesn’t go away isn’t a satisfactory process.

Thoughts? If this passes muster, I’m happy to code up some quick patches (if for no other reason than we can stage them in Mu while we’re waiting for full adoption).

- Bret


[-- Attachment #2: Type: text/html, Size: 5210 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-10-06 10:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-10-04 23:25 Proposal to make EccCheck non-destructive Bret Barkelew
2020-10-06 10:53 ` [edk2-devel] " Laszlo Ersek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox