[erlang-questions] gproc patch

[Ulf Wiger]

How funny! I checked by simply substituting 'uwiger' for 'esl'
in the commit URL:

https://github.com/uwiger/gproc/commit/2742634c2ec608c341d425827eef7f6421849ea5

…and there it was. 

Is this some weird artefact of how github interprets its URIs?

Anyway, I consider github.com/esl/gproc to be the official one,
even though I will strive to keep the uwiger version in synch.

BR,
Ulf

On 20 Jan 2011, at 18:42, Garret Smith wrote:

> I was using the version at https://github.com/uwiger/gproc.git which
> looks identical except for that patch :)
> 
> -G
> 
> On Thu, Jan 20, 2011 at 9:02 AM, Ulf Wiger
> <ulf.wiger@REDACTED> wrote:
>> 
>> Hi Garrett,
>> 
>> Which version of gproc are you using?
>> 
>> That particular bug was fixed on Jan 11 in http://github.com/esl/gproc
>> (That is, quite recently. :)
>> 
>> Sending pull requests to that project will work just fine.
>> 
>> BR,
>> Ulf W
>> 
>> On 19 Jan 2011, at 01:21, Garret Smith wrote:
>> 
>>> Fixed a small issue with gproc in the 'await' function when no timeout
>>> is specified.
>>> 
>>> see branch 'fix-await-notimeout' in the repository
>>> git://github.com/garret-smith/gproc.git
>>> 
>>> or the patch:
>>> 
>>> --- a/src/gproc.erl
>>> +++ b/src/gproc.erl
>>> @@ -312,7 +312,7 @@ request_wait({n,C,_} = Key, Timeout) when C==l; C==g ->
>>>     WRef = call({await,Key,self()}, C),
>>>     receive
>>>         {gproc, WRef, registered, {_K, Pid, V}} ->
>>> -           case Timeout of
>>> +           case TRef of
>>>                no_timer -> ignore;
>>>                _ -> erlang:cancel_timer(TRef)
>>>            end,
>>> 
>>> 
>>> I don't know of any guidelines for patch submission since this is not
>>> OTP, so let me know if there is a better way I can provide patches now
>>> or in the future.
>>> 
>>> -G
>>> 
>>> ________________________________________________________________
>>> erlang-questions (at) erlang.org mailing list.
>>> See http://www.erlang.org/faq.html
>>> To unsubscribe; mailto:erlang-questions-unsubscribe@REDACTED
>>> 
>> 
>> Ulf Wiger, CTO, Erlang Solutions, Ltd.
>> http://erlang-solutions.com
>> 
>> 
>> 
>> 

Ulf Wiger, CTO, Erlang Solutions, Ltd.
http://erlang-solutions.com