On 25-07-2023 04:47, Michel Alexandre Salim wrote:
Once this lands in F38, Fedora Review Service should be fixed,
unless
I'm missing something.
First of all, thanks for bringing fedora-review back. I rely on it quite
a bit reviewing my own packages as well as for official reviews.
I noticed that in f39 builds in Copr the directory containing the
results is now named after the package. Comparing that to the
"traditional" fedora-review directory in f38 builds, I miss the
`licensecheck.txt` file.
The contents of the fedora-review directory ($PACKAGE_NAME) in f39 look
more alike to a local fedora-review, which is fine. But
`licensecheck.txt` is invaluable for checking licenses of generated
files. Just now I finished a review using the f38 build, which detected
two additional licenses in generated files, that went unnoticed by the
submitter.
Can `licensecheck.txt` be added back to f39 fedora-review runs, please?
-- Sandro