Control: severity -1 normal Xiyue Deng <manp...@gmail.com> writes:
> Control: severity -1 minor > thanks > > Nicholas D Steeves <s...@debian.org> writes: > >> Package: php-elisp >> Version: 1.27.0-1 >> Severity: important > > The patch only touches README.md, so I fail to see it as "a bug which > has a major effect on the usability of a package". Changed to minor. Right, that's in the subject of this bug. Is your bias that our users don't read README.md, and/or that documentation isn't important? When users follow the installation instructions in the documentation and it renders the package unusable or breaks non-related packages, then it's more severe than "minor". This has happened disproportionately often with this package compared to others. "Minor" also means trivial to fix, which begs that question: why did you create a problem that would have been trivial to avoid? > Do you have any reference to the discussions? I saw it marked as > "Forwarded: not-needed" and the changes seem to be Debian specific, > which seems possible that upstream may not be aware of it. Yes. Forwarded: not-needed means it's Debian-specific, not unimportant. If in doubt, you should ask your teammates. >> I'd be happy to fix up the patch header to be more informative, and I >> concede that I missed this important step when I orphaned the package. >> > > Adding links to the discussions would help. Help with what? The old patch addresses the issues raised both upstream and in private. > Also, this is still a team maintained package, so feel free to make > changes as you see fit. As the maintainer of this package, you are now responsible to our users, and to upstream. Also, in my checklist of your competencies, you're still missing a couple of categories. Being able to do a three-way merge (I recommend ediff) and maintain a patch is one of them. Maintaining good relations with users and upstreams are two others. Best, Nicholas
signature.asc
Description: PGP signature
