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

Issue 1740001: Windows bots build and use OSMesa... (Closed)

Created:
10 years, 8 months ago by apatrick_chromium
Modified:
9 years, 7 months ago
Reviewers:
Nicolas Sylvain
CC:
chromium-reviews
Visibility:
Public.

Description

Windows bots build and use OSMesa This is a software GL renderer that can be used by bots that do not have access to native GL. It will be used for running tests that use GL, such as Pepper 3D, WebGL and accelerated compositing. Only working on Windows so far. TEST=trybots BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=45241

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+454 lines, -0 lines) Patch
M build/all.gyp View 1 chunk +3 lines, -0 lines 0 comments Download
A third_party/mesa/mesa.gyp View 1 chunk +451 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
apatrick_chromium
10 years, 8 months ago (2010-04-21 00:02:05 UTC) #1
Nicolas Sylvain
Try bots are not using that configuration, so they did not test it. I think ...
10 years, 8 months ago (2010-04-21 17:16:03 UTC) #2
apatrick
The Windows trybots appeared to build osmesa. Search for "osmesa.c" in http://build.chromium.org/buildbot/try-server/builders/win/builds/28691/steps/compile/logs/stdio. I don't want ...
10 years, 8 months ago (2010-04-21 18:47:13 UTC) #3
Nicolas Sylvain
10 years, 8 months ago (2010-04-22 00:00:18 UTC) #4
On Wed, Apr 21, 2010 at 11:47 AM, Alastair Patrick <apatrick@google.com>wrote:

> The Windows trybots appeared to build osmesa. Search for "osmesa.c" in
>
http://build.chromium.org/buildbot/try-server/builders/win/builds/28691/steps....
> I don't want to add it to the main section of all.gyp because then everyone
> will have to build it after clobbering, which takes a fair amount of time,
> and because then they will be using a slower software renderer when using
> the 3D accelerated features of Chrome like WebGL.

The try bots should be building the main section of all.gyp. So if this is
built, users who build all.gyp will also get it.  (except windows users who
build chrome.sln instead of all.sln... or linux users who build chrome
instead of all)

Nicolas


>
> On Wed, Apr 21, 2010 at 10:16 AM, <nsylvain@chromium.org> wrote:
>
>> Try bots are not using that configuration, so they did not test it. I
>> think you
>> also need to add it to the main section in all.gyp if you want them to
>> test it.
>>
>> the current code lgtm, feel free to commit when you feel confident that it
>> works
>>
>>
>> http://codereview.chromium.org/1740001/show
>>
>
>

Powered by Google App Engine
This is Rietveld 408576698