Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2517)

Issue 885043: Added GYP build files. GYP files are needed for integration with chromium and... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 8 months ago by Alok Priyadarshi
Modified:
14 years, 8 months ago
Reviewers:
vangelis, kbr1, dgkoch
CC:
angleproject-review_googlegroups.com
Base URL:
http://angleproject.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Added GYP build files. GYP files are needed for integration with chromium and cross-platform build for glsl translator. It would really be nice if we did not have to maintain two build systems. For now I have only moved the GLSL translator to gyp build system. Please note that the gyp files awkwardly have build_ prefix so they do not conflict with the manually-maintained sln files.

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+803 lines, -704 lines) Patch
A DEPS View 1 chunk +20 lines, -0 lines 0 comments Download
A build/all.gyp View 1 chunk +25 lines, -0 lines 1 comment Download
A build/common.gypi View 1 chunk +97 lines, -0 lines 0 comments Download
A build/gyp_angle View 1 chunk +29 lines, -0 lines 0 comments Download
A samples/build_samples.gyp View 1 chunk +27 lines, -0 lines 0 comments Download
M samples/samples.sln View 2 chunks +0 lines, -18 lines 0 comments Download
M src/ANGLE.sln View 2 chunks +0 lines, -9 lines 0 comments Download
A src/build_angle.gyp View 1 chunk +121 lines, -0 lines 0 comments Download
M src/compiler/glslang.l View 2 chunks +10 lines, -16 lines 0 comments Download
M src/compiler/glslang.y View 65 chunks +473 lines, -477 lines 0 comments Download
M src/compiler/translator_common.vcproj View 1 chunk +1 line, -1 line 0 comments Download
D src/compiler/translator_glsl.vcproj View 1 chunk +0 lines, -183 lines 0 comments Download

Messages

Total messages: 7
Alok Priyadarshi
Grammar files needed to be changed so that they can be parsed by newer versions ...
14 years, 8 months ago (2010-04-05 15:43:26 UTC) #1
kbr1
LGTM
14 years, 8 months ago (2010-04-05 19:42:22 UTC) #2
kbr1
http://codereview.appspot.com/885043/diff/1/4 File build/all.gyp (right): http://codereview.appspot.com/885043/diff/1/4#newcode11 build/all.gyp:11: # build_ prefix should be removed from the gyp ...
14 years, 8 months ago (2010-04-05 19:42:32 UTC) #3
dgkoch
Do these grammer changes continue to work with the old flex and bison that are ...
14 years, 8 months ago (2010-04-05 20:29:11 UTC) #4
Alok Priyadarshi
Hi Daniel, The grammar changes are solely made so they continue to work with the ...
14 years, 8 months ago (2010-04-05 20:57:39 UTC) #5
dgkoch
Hi Alok, Thanks for verifying that the grammars still work with the existing flex and ...
14 years, 8 months ago (2010-04-06 15:14:21 UTC) #6
Alok Priyadarshi
14 years, 8 months ago (2010-04-06 15:41:05 UTC) #7
I am not a git expert but a few people on the chromium project use git
with GYP. Basically you just need to execute the python script in
gyp_angle to generate build files.

GYP generates .vcproj as well. It also supports visual studio express.

In the next CL, I will also add libEGL and libGLES2 so you can start
experimenting and see if it works for you.

On Tue, Apr 6, 2010 at 9:14 AM,  <daniel@transgaming.com> wrote:
> Hi Alok,
>
> Thanks for verifying that the grammars still work with the existing flex
> and bison.
>
> I have no problems with you adding the files for GYP support that if
> that is necessary for Chromium integration. It looks interesting
> (although sparsely documented) and I'm looking forward to trying it out
> once it's in place and I can experiment with it directly.  However, at
> this point I can't commit to us switching to GYP until we've had a
> chance to investigate further. The fact that we're using an internal git
> repository for our code will likely make this more complicated.  Would
> it simplify integration for you if we renamed the solution files that we
> are currently using?
>
> Does GYP generate the .vcproj files as well, or just the .sln files?
>
> If you like, we can continue to discuss this further offline.  As is,
> this patch set looks good to me.
>
> Thanks,
> Daniel
>
> On 2010/04/05 20:57:39, alokp wrote:
>>
>> Hi Daniel,
>
>> The grammar changes are solely made so they continue to work with the
>
> old flex
>>
>> and bison exes. I have ran hlsl translator on a couple of shaders and
>
> they seem
>>
>> to be working fine. In the new GYP build I am using flex and bison
>
> under cygwin.
>
>> It is pretty easy to switch to GYP. The only extra dependency is
>
> depot_tools
>>
>> which is a set of python utilities to maintain multiple svn
>
> repositories. You
>>
>> can set it up by following the instructions here:
>> http://www.chromium.org/developers/how-tos/install-gclient
>
>> After you have depot_tools in your path, do the following to setup
>
> your client:
>>
>> 1. Create a directory to hold your source code. This example assumes
>> c:\angleproject, but other names are fine. Important: Make sure the
>
> full
>>
>> directory path has no spaces.
>> 2. In a shell window, execute the following commands:
>>     cd c:\angleproject
>>     gclient config https://angleproject.googlecode.com/svn/trunk/
>
>> To checkout or update code, execute the following commands:
>>     cd c:\angleproject
>>     gclient sync
>
>> The sync command gets/updates the source code in the trunk and other
>> repositories specified in DEPS file (added in this CL). It also
>
> generates build
>>
>> files at the end. I think it will be nice to just use one build
>
> system. We HAVE
>>
>> to use GYP to integrate with chromium, and it is also nice for
>
> cross-platform
>>
>> builds. If you guys are willing to switch, I will be happy to maintain
>
> the gyp
>>
>> build system.
>
>
>
> http://codereview.appspot.com/885043/show
>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b