Ilya,

Am 25.02.20 um 21:13 schrieb Илья Шипицин:
> it turned out that FreeBSD image is slightly different from packages.
> since we cannot change image, let us just suppress that warning.

I'd like to point out this paragraph from the CONTRIBUTING document:

>    As a rule of thumb, your patch MUST NEVER be made only of a subject line,
>    it *must* contain a description. Even one or two lines, or indicating
>    whether a backport is desired or not. It turns out that single-line commits
>    are so rare in the Git world that they require special manual (hence
>    painful) handling when they are backported, and at least for this reason
>    it's important to keep this in mind.

When reading your patch it was unclear to me which kind of version check
needs to be ignored (and why). At the very least I would have expected a
copy the actual error message from CI (that will be fixed after applying
the patch) in the commit message. I'll copy it here for posterity.

> Updating FreeBSD repository catalogue...
> Fetching meta.conf: . done
> Fetching packagesite.txz: .......... done
> Processing entries: 
> Newer FreeBSD version for package p5-Bio-GFF3:
> To ignore this error set IGNORE_OSVERSION=yes
> - package: 1300081
> - running kernel: 1300078
> Ignore the mismatch and continue? [Y/n]: pkg: repository FreeBSD contains 
> packages for wrong OS version: FreeBSD:13:amd64
> Processing entries... done
> Unable to update repository FreeBSD
> Error updating repositories!

I'm not a FreeBSD guy, but my understanding after a short session with
the search engine of my choice indicated that apparently the Cirrus CI
VM is running an outdated kernel or possibly an outdated FreeBSD jail.
I'm not sure whether that is correct or not. In any case an explanation
why suddenly this error message popped up and why it's safe to ignore
that specific error is safe would have been helpful in the commit
message as well.

Best regards
Tim Düsterhus

Reply via email to