Page MenuHomeFreeBSD

games/anki: Update to 26.09.3
ClosedPublic

Authored by longros on Mon, Oct 19, 4:18 AM.
Tags
None
Referenced Files
F176597699: D60853.id.diff
Tue, Oct 20, 3:00 AM
F176573420: D60853.diff
Mon, Oct 19, 10:57 PM
F176565921: D60853.id190302.diff
Mon, Oct 19, 9:41 PM
F176565470: D60853.id190240.diff
Mon, Oct 19, 9:36 PM
F176557661: D60853.id190240.diff
Mon, Oct 19, 8:06 PM
F176506787: D60853.diff
Mon, Oct 19, 9:55 AM
Unknown Object (File)
Mon, Oct 19, 7:09 AM
Unknown Object (File)
Mon, Oct 19, 7:09 AM
Subscribers
None

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

longros created this revision.

Looks good.

  • Since there is a release page, it's better to reference it in the commit log. https://github.com/ankitects/anki/releases/tag/26.09.3
  • Normally we should wait for maintainer approval, but since the release page says, "Please upgrade to 26.09+ as soon as possible as it includes important security fixes." you can push sooner.
  • VuXML entry?
This revision is now accepted and ready to land.Mon, Oct 19, 11:25 AM
In D60853#1395620, @jrm wrote:

Looks good.

  • Since there is a release page, it's better to reference it in the commit log. https://github.com/ankitects/anki/releases/tag/26.09.3
  • Normally we should wait for maintainer approval, but since the release page says, "Please upgrade to 26.09+ as soon as possible as it includes important security fixes." you can push sooner.
  • VuXML entry?

Hi Joe, the VuXML entry was introduced on 14th September: https://vuxml.freebsd.org/freebsd/ce54d883-b060-11f1-b187-901b0edee044.html
I add only an entry that is not covered by the existing entry but it does not have a CVE or GSHA "Prevent external links in deck descriptions rendered in the overview/congrats screen from manipulating the page in certain cases when clicked." https://github.com/ankitects/anki/releases#release-26.09.2

make validate passes

This revision now requires review to proceed.Mon, Oct 19, 5:46 PM

This should be merged to the quarterly branch, too.

This revision is now accepted and ready to land.Mon, Oct 19, 6:08 PM
This revision was automatically updated to reflect the committed changes.