Package: qa.debian.org Severity: normal User: qa.debian....@packages.debian.org Usertags: udd
Hi, When using https://udd.debian.org/patches.cgi, I notice that whenever the Forwarded field contains anything other than "no", "not-needed", "yes" or an URL, it gets marked as invalid. That includes cases where it begins with "yes", but is complemented with other data, as can be seen by contrasting a search on Debian Sources about those packages [1] with their respective patch metadata status on UDD. In its current form, patches.cgi marks as invalid patches that include useful information on the Forwarded field, such as the mail address to which the patch was forwarded, when upstream doesn't have a public mailing list archive or a web pull/merge request tracker, for example. According to DEP-3 [2], if the Forwarded field is present, any other value other than "no" or "not-needed" should be considered valid: Any value other than "no" or "not-needed" means that the patch has been forwarded upstream. Ideally the value is an URL proving that it has been forwarded and where one can find more information about its inclusion status. As such, I think it would be interesting to either propose changes to DEP-3 or to adhere more closely to it. Thanks, David [1] https://codesearch.debian.net/search?q=file%3Adebian%2Fpatches%2F*.patch+Forwarded%3A+yes+.%2B&literal=0 [2] https://dep-team.pages.debian.net/deps/dep3/
