Discussion:
An apology about my commits
Neel Chauhan
2021-05-28 03:19:14 UTC
Permalink
Hi freebsd-ports@, ports-committers@,

I hope all is well with you.

You may know I have prematurely made Ports commits without review,
namely 507359c509389602b4060b2c5e203c99911c3578 and
0bc7478682b2d7c9393f2dd095d99072070a2f65.

I'm really sorry if I pushed these commits without approval.

I feel extremely guilty for making these commits without approval, and
almost felt I will lose my commit bit. I even thought at times that I
will have to switch my PCs/servers to Windows or Linux if all goes wrong
since I will feel shame for even **using** FreeBSD [1].

Next time, I will try not to bypass review, especially for big commits
that touch major packages like devel/glib20.

However, to unbreak the fetching for GNOME Ports that have been updated
to 40.x, we will need to make one change to Mk/bsd.sites.mk. This patch
is in https://reviews.freebsd.org/D30514 . This may need portmgr
approval, but I won't commit it without approval.

Happy hacking!

-Neel Chauhan (nc@)

Footnotes:

[1] - For reference, I work at Microsoft and use Windows at work, but
used FreeBSD almost exclusively at home since 2012, back when I was only
15 (I'm 24 now), and more-or-less the only server OS I have run my home
servers on since I set that up in 2013.
Kubilay Kocak
2021-05-28 03:25:07 UTC
Permalink
Post by Neel Chauhan
I hope all is well with you.
You may know I have prematurely made Ports commits without review,
namely 507359c509389602b4060b2c5e203c99911c3578 and
0bc7478682b2d7c9393f2dd095d99072070a2f65.
I'm really sorry if I pushed these commits without approval.
I feel extremely guilty for making these commits without approval, and
almost felt I will lose my commit bit. I even thought at times that I
will have to switch my PCs/servers to Windows or Linux if all goes wrong
since I will feel shame for even **using** FreeBSD [1].
Next time, I will try not to bypass review, especially for big commits
that touch major packages like devel/glib20.
However, to unbreak the fetching for GNOME Ports that have been updated
to 40.x, we will need to make one change to Mk/bsd.sites.mk. This patch
is in https://reviews.freebsd.org/D30514 . This may need portmgr
approval, but I won't commit it without approval.
Happy hacking!
[1] - For reference, I work at Microsoft and use Windows at work, but
used FreeBSD almost exclusively at home since 2012, back when I was only
15 (I'm 24 now), and more-or-less the only server OS I have run my home
servers on since I set that up in 2013.
It's your thoughts and care that count Neel, don't feel guilty.

The responses you received could have been a bit more 'matter of fact'.

What was *actually* important about the replies was 'we should be QA'ing
everything and getting more than one persons eyes on things'.


Note (for everyone), some of the biggest impacts are caused by the
smallest commits.

The idea of a 'trivial update' needs to be related to the dustbin, and
we're still hearing it often.
Kubilay Kocak
2021-05-28 03:26:47 UTC
Permalink
Post by Kubilay Kocak
Post by Neel Chauhan
I hope all is well with you.
You may know I have prematurely made Ports commits without review,
namely 507359c509389602b4060b2c5e203c99911c3578 and
0bc7478682b2d7c9393f2dd095d99072070a2f65.
I'm really sorry if I pushed these commits without approval.
I feel extremely guilty for making these commits without approval, and
almost felt I will lose my commit bit. I even thought at times that I
will have to switch my PCs/servers to Windows or Linux if all goes
wrong since I will feel shame for even **using** FreeBSD [1].
Next time, I will try not to bypass review, especially for big commits
that touch major packages like devel/glib20.
However, to unbreak the fetching for GNOME Ports that have been
updated to 40.x, we will need to make one change to Mk/bsd.sites.mk.
This patch is in https://reviews.freebsd.org/D30514 . This may need
portmgr approval, but I won't commit it without approval.
Happy hacking!
[1] - For reference, I work at Microsoft and use Windows at work, but
used FreeBSD almost exclusively at home since 2012, back when I was
only 15 (I'm 24 now), and more-or-less the only server OS I have run
my home servers on since I set that up in 2013.
It's your thoughts and care that count Neel, don't feel guilty.
The responses you received could have been a bit more 'matter of fact'.
What was *actually* important about the replies was 'we should be QA'ing
everything and getting more than one persons eyes on things'.
Note (for everyone), some of the biggest impacts are caused by the
smallest commits.
The idea of a 'trivial update' needs to be related to the dustbin, and
we're still hearing it often.
ugh, relegated*
Neel Chauhan
2021-05-28 03:42:30 UTC
Permalink
Post by Kubilay Kocak
It's your thoughts and care that count Neel, don't feel guilty.
Thanks!
Post by Kubilay Kocak
The responses you received could have been a bit more 'matter of fact'.
What was *actually* important about the replies was 'we should be
QA'ing everything and getting more than one persons eyes on things'.
True, especially for big packages like GNOME, KDE, Mozilla, Xorg, or
LibreOffice.

Sometimes, I get trigger-happy. But I really shouldn't, well unless it's
a hobby project on GitHub.
Post by Kubilay Kocak
Note (for everyone), some of the biggest impacts are caused by the
smallest commits.
The idea of a 'trivial update' needs to be related to the dustbin, and
we're still hearing it often.
+1

I've had "trivial updates" that broke things, both in and outside
FreeBSD.

I've had code broken at my $DAYJOB, and that a Microsoft SaaS product
(not a household name product like Windows or Word, but still). I was a
major Tor contributor from 2017-2020 and had bad patches that broke
things or add bugs get in, and that with Tor being very
security-focused.

And hey, it's not like I'm pulling a University of Minnesota "research
project" where the goal is to intentionally add bugs to the Linux
kernel.

-Neel

Loading...