Nicholas D Steeves <sten@debian.org> writes: > Control: severity -1 normal > > Xiyue Deng <manphiz@gmail.com> writes: > >> Control: severity -1 minor >> thanks >> >> Nicholas D Steeves <sten@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? > I'm following the bug severity level suggestions[1]. The patch only touches documents without any code change, which I think fits the category of "minor": a problem which doesn't affect the package's usefulness, and is presumably trivial to fix. Also, I see that you changed the severity to "normal", so I think you agree it's not "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? > I found your arguments vague and exaggerating without any reference. Let's first make sure we are discussing the patch I dropped in commit[2]. The patch mainly does 2 things: replaces upstream instruction to install through ELPA with Debian instructions (with a suggest prefer version in sid); and a new instruction to report bug in the Debian way. I have no concern with the second change. By the first change, Are you suggesting that installing php-mode through ELPA breaks users in anyway? In my experience, installing addons both from ELPA and Debian work fine, even if there were dependency relationships. Also suggesting using sid on stable Debian systems creates FrankenDebian, which I believe a senior DD like you know is a bad thing. If a newer upstream version fixes any issues, and the newer version will not be backported to the stable distribution, the better way is to install the newer version through ELPA, instead of configuring a FrankenDebian. The is also an important reason I didn't try to rebase the patch. >> 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. And where is the link referencing the discussion? > Forwarded: not-needed means it's Debian-specific, not unimportant. > If in doubt, you should ask your teammates. > OK. >>> 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. > Adding a link to the upstream issue you mentioned would help me and other maintainers understand your concern better. >> 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. > The maintainer is the Emacsen team, but yes, I do have responsibilities as you mentioned, with help from other team members. > 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. > You assume that I dropped the patch because I didn't know how to do a 3-way merge. I gave my reason above. > Best, > Nicholas [1] https://www.debian.org/Bugs/Developer#severities [2] https://salsa.debian.org/emacsen-team/php-elisp/-/commit/08630e52c7ee41c022522543d127af42d3afe5c2 -- Regards, Xiyue Deng
Attachment:
signature.asc
Description: PGP signature