Date: Sun, 12 Feb 2012 21:47:03 +0000 From: Chris Rees <utisoft@gmail.com> To: Steve Kargl <sgk@troutmask.apl.washington.edu> Cc: Stephen Montgomery-Smith <stephen@missouri.edu>, freebsd-ports@freebsd.org, Andriy Gapon <avg@freebsd.org> Subject: Re: Please test your commits Message-ID: <CADLo839J3vQppA4V8cMcgVjNT0yQt5-4kCLEqzhJYApB5miVMQ@mail.gmail.com> In-Reply-To: <20120212213750.GA93817@troutmask.apl.washington.edu> References: <20120212193927.GA86426@troutmask.apl.washington.edu> <CADLo83_UZR0K15LQWoXb-AAhd1wj25fbeKSAPch5ji_Q=41f8A@mail.gmail.com> <20120212201624.GA86650@troutmask.apl.washington.edu> <4F3820AD.4090702@FreeBSD.org> <20120212204118.GA86775@troutmask.apl.washington.edu> <CADLo83_w1A7tv3jgK_LQZCgi7odbEniE4Qfn1HJwgjWBNVn4ig@mail.gmail.com> <20120212204526.GB86775@troutmask.apl.washington.edu> <CADLo83_Ni55YqyM6H3Tn7gWMx1pWe6TqtvtRB-WVuiB5dU6KnA@mail.gmail.com> <20120212211744.GC86775@troutmask.apl.washington.edu> <4F383004.2000800@missouri.edu> <20120212213750.GA93817@troutmask.apl.washington.edu>
next in thread | previous in thread | raw e-mail | index | archive | help
On 12 Feb 2012 21:37, "Steve Kargl" <sgk@troutmask.apl.washington.edu> wrote: > > On Sun, Feb 12, 2012 at 03:32:52PM -0600, Stephen Montgomery-Smith wrote: > > > > > >But, the 2nd issue with too many arguments in a function call is > > >clearly evident on amd64 because I justed test that on FreeBSD 10. > > > > Yes. But the issue isn't whether someone else was correct in why the > > port might or might not have built in a particular environment. > > > > The issue is whether you were too hasty in your initial accusation that > > the committer didn't test their commit. And another issue is whether > > you should apologize to them for attempting to publicly humiliate them. > > > > I've now tested on i386 and amd64, and the port fails > to build on both architectures. Given the code for the > malloc.h failure, this port will fail on all non-amd64 > platforms that freebsd runs (dating back FreeBSD 5.0). > The evidence suggests that this commit was not tested. Whatever. Others have pointed out that they can't reproduce your error, so save the posturing, and get a PR in. Thank you. Chris
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CADLo839J3vQppA4V8cMcgVjNT0yQt5-4kCLEqzhJYApB5miVMQ>