[erlang-questions] gproc patch

Garret Smith garret.smith@REDACTED
Fri Jan 21 18:40:27 CET 2011


Thanks, I'll start tracking the esl version.

-G

On Fri, Jan 21, 2011 at 9:01 AM, Alain O'Dea <alain.odea@REDACTED> wrote:
> 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