Comment 58 for bug 1700373

15:38 <rbasak> chiluk: around? Reviewing intel-microcode.

15:39 <rbasak> In principle the change seems OK, but I'm confused by things mismatching the descriptions in debian/changelog

15:39 <rbasak> "source: remove unneeded intel-ucode/ directory" - removed from where? I don't see this in the current packages for Xenial nor in Zesty.

15:40 <rbasak> "source: remove superseded upstream data file: 20170511" - this is not true for Xenial. You remove other files instead.

15:41 <rbasak> Though if we're removing superseded files, why do some remain? Are we following a pattern of leaving old files there, or removing old files? I'm not sure I follow why your proposed SRUs appear to do both. Is there some nuance I'm missing?