Re: git: 28472e9722d8 - main - Revert "x11-fonts/noto*: Restructure and update"
- Reply: Moin Rahman : "Re: git: 28472e9722d8 - main - Revert "x11-fonts/noto*: Restructure and update""
- Reply: Po-Chuan Hsieh : "Re: git: 28472e9722d8 - main - Revert "x11-fonts/noto*: Restructure and update""
- In reply to: Gleb Popov : "Re: git: 28472e9722d8 - main - Revert "x11-fonts/noto*: Restructure and update""
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 23 Mar 2024 15:38:03 UTC
Am Samstag, 23. März 2024, 15:14:01 CET schrieb Gleb Popov: > On Sat, Mar 23, 2024 at 5:04 PM Po-Chuan Hsieh <sunpoet@freebsd.org> wrote: > > The branch main has been updated by sunpoet: > > > > URL: > > https://cgit.FreeBSD.org/ports/commit/?id=28472e9722d808739a4d690f6d803a2 > > b1b8f5f98 > > > > commit 28472e9722d808739a4d690f6d803a2b1b8f5f98 > > Author: Po-Chuan Hsieh <sunpoet@FreeBSD.org> > > AuthorDate: 2024-03-23 14:01:07 +0000 > > Commit: Po-Chuan Hsieh <sunpoet@FreeBSD.org> > > CommitDate: 2024-03-23 14:01:07 +0000 > > > > Revert "x11-fonts/noto*: Restructure and update" > > > > This reverts commit 3b128ff9f74b28ada2f475f0b816480b1550e5b8. > > It is good manners to explain the reason for reverting when doing it. > Even better to first talk to the original committer and ask him to the > revert. Moin moin I completely agree. Given that there has been a phabricator review open for two months, with you, sunpoet@ as reviewer, this is looking like poor judgement on your side. As arrowd@ points out, you should have commented on the reverted commit first, and elaborate your reasoning for a revert. Simply reverting the work of others may lead to fewer contributions. Please take this into consideration in the future. mfg Tobias