Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(659)

Issue 2118005: Don't probe virtual devices before they exist (Closed)

Created:
10 years, 7 months ago by Nasser Grainawi
Modified:
9 years, 7 months ago
Reviewers:
Micah C, Jason Glasgow, borr
CC:
chromium-os-reviews_chromium.org
Base URL:
git://git.chromium.org/modemmanager.git
Visibility:
Public.

Description

Don't probe virtual devices before they exist Prevent ModemManager from probing virtual devices before their associated files exist in /dev tree. Contributed by Nasser Grainawi <nasser@codeaurora.org>;

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -0 lines) Patch
M src/mm-plugin-base.c View 3 chunks +20 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Nasser Grainawi
10 years, 7 months ago (2010-05-15 01:48:16 UTC) #1
Jason Glasgow
LGTM
10 years, 7 months ago (2010-05-17 22:42:48 UTC) #2
Nasser Grainawi
jglasgow@chromium.org wrote: > LGTM > > http://codereview.chromium.org/2118005/show Thanks! Can someone please commit?
10 years, 7 months ago (2010-05-17 22:54:38 UTC) #3
jglasgow
Yes. I will do so. -Jason On Mon, May 17, 2010 at 6:54 PM, Nasser ...
10 years, 7 months ago (2010-05-17 23:17:06 UTC) #4
jglasgow
10 years, 7 months ago (2010-05-17 23:26:05 UTC) #5
Done. -jason

On Mon, May 17, 2010 at 7:17 PM, Jason Glasgow <jglasgow@google.com> wrote:

> Yes.  I will do so. -Jason
>
>
> On Mon, May 17, 2010 at 6:54 PM, Nasser Grainawi <nasser@codeaurora.org>wrote:
>
>> jglasgow@chromium.org wrote:
>>
>>> LGTM
>>>
>>> http://codereview.chromium.org/2118005/show
>>>
>>
>> Thanks! Can someone please commit?
>>
>
>

Powered by Google App Engine
This is Rietveld 408576698