Code review comment for ~logan/ubuntu/+source/xdg-utils:merge

Revision history for this message
Bryce Harrington (bryce) wrote :

I've confirmed the patch is still needed for this release of the package.

I would recommend adding DEP3 headers for the patch, specifically Origin, Bug-Ubuntu, and Bug-Debian (https://dep-team.pages.debian.net/deps/dep3/). The bug has already been raised with Debian at https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=975370, so that should be listed as Bug-Debian. It appears they are considering alternate solutions, but no recent activity it seems, so for now keeping the existing fix is the right course of action.

LGTM, +1

If you need sponsorship, let me know.

review: Approve

« Back to merge proposal