[Buildroot] [PATCH v2 1/1] package/golang: new package
Yann E. MORIN
yann.morin.1998 at free.fr
Thu Jul 30 16:37:53 UTC 2015
Christian, All,
On 2015-07-29 15:37 -0700, Christian Stewart spake thusly:
> This patch adds golang, the Go compiler-and-interpreter.
Please, do not resend so fast.
After a review, it is usually good to wait a bit, to give others a
chance to review, too.
Re-sending too fast means others would review the previous iteration
(bad) but that you would also have a lot of re-spin to do. Also,
potential reviewr (and committer) may get wary of this many respins.
I know you are probably eager to see that merged soon, but re-sending
too fast is not the best way to make that happen.
Unless the committer asks for a simple fixup and a prompt respin, of
course. Otherwise, let it settle for a day...
Beside, in that case, your respin is not correct, because it does not
address the rm-rf that Thomas pointed out, so I'll have to repin my
version...
Don't get me wrong: I too am eager to see it merged, and we all do
appreciate your work on this! :-)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list