Re: svn commit: r326792 - head/stand/uboot/common
On 12/12/17 20:27, Cy Schubert wrote: PR 224080??? Yes please. Regards -- Niclas ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
RE: svn commit: r326792 - head/stand/uboot/common
PR 224080??? --- Sent using a tiny phone keyboard. Apologies for any typos and autocorrect. Also, this old phone only supports top post. Apologies. Cy Schubert <cy.schub...@cschubert.com> or <c...@freebsd.org> The need of the many outweighs the greed of the few. --- -Original Message- From: Mateusz Guzik Sent: 12/12/2017 11:13 To: Warner Losh Cc: Antoine Brodin; svn-src-head@freebsd.org; svn-src-...@freebsd.org; src-committers Subject: Re: svn commit: r326792 - head/stand/uboot/common How about you both close one PR as a punishment and we pretend this did not happen. The thing to do was to reply to the offending commit or contact the committer on irc. Only in case of no timely response fixing on your own is the right thing to do. On Tue, Dec 12, 2017 at 8:03 PM, Warner Losh <i...@bsdimp.com> wrote: The email I wrote here also sucks. It was unduly harsh. While the commit message should have been better, I was too harsh in trying to express this and giving a better example. Warner On Tue, Dec 12, 2017 at 5:49 AM, Warner Losh <i...@bsdimp.com> wrote: > This commit message sucks. Please use a more descriptive one in the future > like "catch up to change rX that remove the argument from interact." If > you want to be snarky, add 'pounty hat to imp' to it. Thats infornative. > Your message is utterly useless. It doesn't even say what arch is broken. > > Warner > > Warner > > On Dec 12, 2017 1:47 AM, "Antoine Brodin" <anto...@freebsd.org> wrote: > >> Author: antoine >> Date: Tue Dec 12 09:46:53 2017 >> New Revision: 326792 >> URL: https://svnweb.freebsd.org/changeset/base/326792 >> >> Log: >> Attempt to unbreak buildworld >> >> Modified: >> head/stand/uboot/common/main.c >> >> Modified: head/stand/uboot/common/main.c >> >> == >> --- head/stand/uboot/common/main.c Tue Dec 12 06:56:21 2017 >> (r326791) >> +++ head/stand/uboot/common/main.c Tue Dec 12 09:46:53 2017 >> (r326792) >> @@ -500,7 +500,7 @@ main(int argc, char **argv) >> archsw.arch_readin = uboot_readin; >> archsw.arch_autoload = uboot_autoload; >> >> - interact(NULL); /* doesn't return */ >> + interact(); /* doesn't return */ >> >> return (0); >> } >> >> ___ svn-src-...@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org" -- Mateusz Guzik ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
Re: svn commit: r326792 - head/stand/uboot/common
On Tue, Dec 12, 2017 at 12:13 PM, Mateusz Guzikwrote: > How about you both close one PR as a punishment and we pretend this did > not happen. > I'll fix 6 CIDs as punishment. Warner > The thing to do was to reply to the offending commit or contact the > committer > on irc. Only in case of no timely response fixing on your own is the right > thing > to do. > > > On Tue, Dec 12, 2017 at 8:03 PM, Warner Losh wrote: > >> The email I wrote here also sucks. It was unduly harsh. While the commit >> message should have been better, I was too harsh in trying to express this >> and giving a better example. >> >> Warner >> >> On Tue, Dec 12, 2017 at 5:49 AM, Warner Losh wrote: >> >> > This commit message sucks. Please use a more descriptive one in the >> future >> > like "catch up to change rX that remove the argument from >> interact." If >> > you want to be snarky, add 'pounty hat to imp' to it. Thats infornative. >> > Your message is utterly useless. It doesn't even say what arch is >> broken. >> > >> > Warner >> > >> > Warner >> > >> > On Dec 12, 2017 1:47 AM, "Antoine Brodin" wrote: >> > >> >> Author: antoine >> >> Date: Tue Dec 12 09:46:53 2017 >> >> New Revision: 326792 >> >> URL: https://svnweb.freebsd.org/changeset/base/326792 >> >> >> >> Log: >> >> Attempt to unbreak buildworld >> >> >> >> Modified: >> >> head/stand/uboot/common/main.c >> >> >> >> Modified: head/stand/uboot/common/main.c >> >> >> >> == >> >> --- head/stand/uboot/common/main.c Tue Dec 12 06:56:21 2017 >> >> (r326791) >> >> +++ head/stand/uboot/common/main.c Tue Dec 12 09:46:53 2017 >> >> (r326792) >> >> @@ -500,7 +500,7 @@ main(int argc, char **argv) >> >> archsw.arch_readin = uboot_readin; >> >> archsw.arch_autoload = uboot_autoload; >> >> >> >> - interact(NULL); /* doesn't return */ >> >> + interact(); /* doesn't return */ >> >> >> >> return (0); >> >> } >> >> >> >> >> ___ >> svn-src-...@freebsd.org mailing list >> https://lists.freebsd.org/mailman/listinfo/svn-src-all >> To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org" >> > > > > -- > Mateusz Guzik > ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
Re: svn commit: r326792 - head/stand/uboot/common
How about you both close one PR as a punishment and we pretend this did not happen. The thing to do was to reply to the offending commit or contact the committer on irc. Only in case of no timely response fixing on your own is the right thing to do. On Tue, Dec 12, 2017 at 8:03 PM, Warner Loshwrote: > The email I wrote here also sucks. It was unduly harsh. While the commit > message should have been better, I was too harsh in trying to express this > and giving a better example. > > Warner > > On Tue, Dec 12, 2017 at 5:49 AM, Warner Losh wrote: > > > This commit message sucks. Please use a more descriptive one in the > future > > like "catch up to change rX that remove the argument from interact." > If > > you want to be snarky, add 'pounty hat to imp' to it. Thats infornative. > > Your message is utterly useless. It doesn't even say what arch is broken. > > > > Warner > > > > Warner > > > > On Dec 12, 2017 1:47 AM, "Antoine Brodin" wrote: > > > >> Author: antoine > >> Date: Tue Dec 12 09:46:53 2017 > >> New Revision: 326792 > >> URL: https://svnweb.freebsd.org/changeset/base/326792 > >> > >> Log: > >> Attempt to unbreak buildworld > >> > >> Modified: > >> head/stand/uboot/common/main.c > >> > >> Modified: head/stand/uboot/common/main.c > >> > >> == > >> --- head/stand/uboot/common/main.c Tue Dec 12 06:56:21 2017 > >> (r326791) > >> +++ head/stand/uboot/common/main.c Tue Dec 12 09:46:53 2017 > >> (r326792) > >> @@ -500,7 +500,7 @@ main(int argc, char **argv) > >> archsw.arch_readin = uboot_readin; > >> archsw.arch_autoload = uboot_autoload; > >> > >> - interact(NULL); /* doesn't return */ > >> + interact(); /* doesn't return */ > >> > >> return (0); > >> } > >> > >> > ___ > svn-src-...@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/svn-src-all > To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org" > -- Mateusz Guzik ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
Re: svn commit: r326792 - head/stand/uboot/common
The email I wrote here also sucks. It was unduly harsh. While the commit message should have been better, I was too harsh in trying to express this and giving a better example. Warner On Tue, Dec 12, 2017 at 5:49 AM, Warner Loshwrote: > This commit message sucks. Please use a more descriptive one in the future > like "catch up to change rX that remove the argument from interact." If > you want to be snarky, add 'pounty hat to imp' to it. Thats infornative. > Your message is utterly useless. It doesn't even say what arch is broken. > > Warner > > Warner > > On Dec 12, 2017 1:47 AM, "Antoine Brodin" wrote: > >> Author: antoine >> Date: Tue Dec 12 09:46:53 2017 >> New Revision: 326792 >> URL: https://svnweb.freebsd.org/changeset/base/326792 >> >> Log: >> Attempt to unbreak buildworld >> >> Modified: >> head/stand/uboot/common/main.c >> >> Modified: head/stand/uboot/common/main.c >> >> == >> --- head/stand/uboot/common/main.c Tue Dec 12 06:56:21 2017 >> (r326791) >> +++ head/stand/uboot/common/main.c Tue Dec 12 09:46:53 2017 >> (r326792) >> @@ -500,7 +500,7 @@ main(int argc, char **argv) >> archsw.arch_readin = uboot_readin; >> archsw.arch_autoload = uboot_autoload; >> >> - interact(NULL); /* doesn't return */ >> + interact(); /* doesn't return */ >> >> return (0); >> } >> >> ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
Re: svn commit: r326792 - head/stand/uboot/common
This commit message sucks. Please use a more descriptive one in the future like "catch up to change rX that remove the argument from interact." If you want to be snarky, add 'pounty hat to imp' to it. Thats infornative. Your message is utterly useless. It doesn't even say what arch is broken. Warner Warner On Dec 12, 2017 1:47 AM, "Antoine Brodin"wrote: > Author: antoine > Date: Tue Dec 12 09:46:53 2017 > New Revision: 326792 > URL: https://svnweb.freebsd.org/changeset/base/326792 > > Log: > Attempt to unbreak buildworld > > Modified: > head/stand/uboot/common/main.c > > Modified: head/stand/uboot/common/main.c > > == > --- head/stand/uboot/common/main.c Tue Dec 12 06:56:21 2017 > (r326791) > +++ head/stand/uboot/common/main.c Tue Dec 12 09:46:53 2017 > (r326792) > @@ -500,7 +500,7 @@ main(int argc, char **argv) > archsw.arch_readin = uboot_readin; > archsw.arch_autoload = uboot_autoload; > > - interact(NULL); /* doesn't return */ > + interact(); /* doesn't return */ > > return (0); > } > > ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
svn commit: r326792 - head/stand/uboot/common
Author: antoine Date: Tue Dec 12 09:46:53 2017 New Revision: 326792 URL: https://svnweb.freebsd.org/changeset/base/326792 Log: Attempt to unbreak buildworld Modified: head/stand/uboot/common/main.c Modified: head/stand/uboot/common/main.c == --- head/stand/uboot/common/main.c Tue Dec 12 06:56:21 2017 (r326791) +++ head/stand/uboot/common/main.c Tue Dec 12 09:46:53 2017 (r326792) @@ -500,7 +500,7 @@ main(int argc, char **argv) archsw.arch_readin = uboot_readin; archsw.arch_autoload = uboot_autoload; - interact(NULL); /* doesn't return */ + interact(); /* doesn't return */ return (0); } ___ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"