[erlang-questions] gproc patch

Alain O'Dea alain.odea@REDACTED
Fri Jan 21 18:01:36 CET 2011


It's more likely that the commit is present in both repositories, but one does not have it on the master branch.



On 2011-01-21, at 3:49, Ulf Wiger <ulf.wiger@REDACTED> wrote:

> 
> 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
> 
> 
> 
> 
> ________________________________________________________________
> erlang-questions (at) erlang.org mailing list.
> See http://www.erlang.org/faq.html
> To unsubscribe; mailto:erlang-questions-unsubscribe@REDACTED
> 


More information about the erlang-questions mailing list