Oopsie in sysvinit-2.92

[Benda Xu]

Hi Dmitry,

Dmitry Bogatov <KAction at debian.org> writes:

> [2018-11-24 09:31] KatolaZ <katolaz at freaknet.org>
>> > Dmitry, Benda, Ian: you find the relevant commit in sysvinit/master
>> > for review.
>
> Please, when you are requesting review, do not push to
> master. Remember, you can not amend `master' branch once you pushed to
> it, no matter what!
>
> I have following concerns about those three commits on `master'
> branch:
>
>  * New upstream release should be incorporate as merge with
>  `upstream-release' branch. Use `gbp import-orig --uscan'.
>
>  * Your commit messages and changelog entries could be improved. The
>  `gitlint' tool may provide some suggestions.
>
>  * You do not add [ J. Random hacker ] into changelog as long you are
>  the only person making changes.
>
> Given that 2.93 is already released, I believe the best course of action
> for us would be to revert your commits and package and upload 2.93
> instead. I did it and pushed into `wip/master'. If there is no
> objections, I will push it into `master' and upload to sid.
>
> Please, do not touch `master' for now.

While I agree with your commits on points for KatolaZ to improve, I
would like to remind that the git workflow is ultimately a personal
preference, not black-or-white, right-or-wrong.  I wish I misread your
tonality in this email and I sincerely hope that KatolaZ does not take
it personally.

Yours,
Benda