<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
<title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
On 03/06/2011 07:20 AM, Jason House wrote:
<blockquote
cite="mid:9642EC32-DD8B-4AA0-9710-DA90131A35CC@gmail.com"
type="cite">
<div>Short of having more Don's, is there anything that can be
done to reduce how much time it takes to process patches?</div>
<div><br>
</div>
<div>Automated test suite against dmd with the patch? Detection of
patches that apply cleanly?</div>
<div><br>
</div>
<div>A less automated thing is someone to check if patches come
with new tests that verify they do what is intended.</div>
<div><br>
</div>
</blockquote>
<br>
Even that could be automated to some extent:<br>
<br>
- apply any part of the patch that updates the tests<br>
- run the tests<br>
- apply the rest<br>
- run the test again<br>
<br>
If any test go from pass to fail, flag that. If no test go from fail
to pass, flag that.<br>
<br>
This wouldn't work with all requests, but some sort of annotation
standard for the comments would allow for overriding as well as
automation of things like updating bugzilla items.<br>
<br>
<blockquote
cite="mid:9642EC32-DD8B-4AA0-9710-DA90131A35CC@gmail.com"
type="cite">
<div>Etc...<br>
<br>
Sent from my iPhone</div>
<div><br>
On Mar 6, 2011, at 1:43 AM, Walter Bright <<a
moz-do-not-send="true" href="mailto:walter@digitalmars.com">walter@digitalmars.com</a>>
wrote:<br>
<br>
</div>
<blockquote type="cite">
<div>
I know, I know, but this piles up faster and faster. 8 new
bugzilla
reports <b>just today</b>. Many come with patches. I fixed
one.<br>
<br>
Brad Roberts wrote:
<blockquote cite="mid:4D7307E4.3060502@puremagic.com"
type="cite">
<pre wrap="">Walter, as gatekeeper to dmd, this is primarily addressed to you. A major portion of engaging with a community of
developers is being responsive to their desires to help out. One of the primary reasons we switched to github was
exactly to better engage with the community. I really think taking more than a week to respond to a pull request is
excessive. Over a month, rather rude. It's certainly not encouraging anyone to continue to invest their time.
So, would you take some time this weekend to catch up? If there's something that you'd like either Don or I to do with
them, speak up. I'm willing to invest a bit of time if it'd help get the change merged in a more timely manner.
Here's the open pull requests for dmd:
<a moz-do-not-send="true" href="https://github.com/D-Programming-Language/dmd/pull/3">https://github.com/D-Programming-Language/dmd/pull/3</a> Feb 6, 2011
const(Object) ref
<a moz-do-not-send="true" href="https://github.com/D-Programming-Language/dmd/pull/5">https://github.com/D-Programming-Language/dmd/pull/5</a> Feb 9, 2011
Issue 4360 - Allow intrinsics in core.bitop to operate as intrinsics
<a moz-do-not-send="true" href="https://github.com/D-Programming-Language/dmd/pull/6">https://github.com/D-Programming-Language/dmd/pull/6</a> Feb 13, 2011
Issue 4833 - map file location obeys -od/-of flags
<a moz-do-not-send="true" href="https://github.com/D-Programming-Language/dmd/pull/7">https://github.com/D-Programming-Language/dmd/pull/7</a> Feb 13, 2011
Remove old druntime interface
<a moz-do-not-send="true" href="https://github.com/D-Programming-Language/dmd/pull/10">https://github.com/D-Programming-Language/dmd/pull/10</a> Feb 20, 2011
Clarify tuple length error message
<a moz-do-not-send="true" href="https://github.com/D-Programming-Language/dmd/pull/11">https://github.com/D-Programming-Language/dmd/pull/11</a> Feb 26, 2011
Issue 3541 - Add -oq to dmd (use fully qualified module name as object filename)
<a moz-do-not-send="true" href="https://github.com/D-Programming-Language/dmd/pull/12">https://github.com/D-Programming-Language/dmd/pull/12</a> Mar 3, 2011
Bug fixes I threw patches in the past
(NOTE: looks like at least part of this one needs to be delayed. Thanks for reviewing it Don!)
If you don't think that the code is ready to be pulled, at a minimum, a comment needs to be added about what needs to be
done.
Thanks,
Brad
_______________________________________________
dmd-internals mailing list
<a moz-do-not-send="true" href="mailto:dmd-internals@puremagic.com">dmd-internals@puremagic.com</a>
<a moz-do-not-send="true" href="http://lists.puremagic.com/mailman/listinfo/dmd-internals">http://lists.puremagic.com/mailman/listinfo/dmd-internals</a>
</pre>
</blockquote>
</div>
</blockquote>
<blockquote type="cite">
<div><span>_______________________________________________</span><br>
<span>dmd-internals mailing list</span><br>
<span><a moz-do-not-send="true"
href="mailto:dmd-internals@puremagic.com">dmd-internals@puremagic.com</a></span><br>
<span><a moz-do-not-send="true"
href="http://lists.puremagic.com/mailman/listinfo/dmd-internals">http://lists.puremagic.com/mailman/listinfo/dmd-internals</a></span></div>
</blockquote>
<pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
dmd-internals mailing list
<a class="moz-txt-link-abbreviated" href="mailto:dmd-internals@puremagic.com">dmd-internals@puremagic.com</a>
<a class="moz-txt-link-freetext" href="http://lists.puremagic.com/mailman/listinfo/dmd-internals">http://lists.puremagic.com/mailman/listinfo/dmd-internals</a></pre>
</blockquote>
<br>
</body>
</html>