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

Issue 4288001: Add explicit build dependencies from npchrome_frame to chrome and chrome_dll.... (Closed)

Created:
10 years, 1 month ago by robertshield
Modified:
9 years, 6 months ago
Reviewers:
amit
CC:
chromium-reviews, amit
Visibility:
Public.

Description

Add explicit build dependencies from npchrome_frame to chrome and chrome_dll. This is to avoid having non-versioned matched components in the build directory if only buildingg npchrome_frame which causes all manner of trouble. BUG=version-mismatched binaries can reside in the build folder when building only npchrome_frame TEST=NONE Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=64759

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M chrome_frame/chrome_frame.gyp View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
robertshield
10 years, 1 month ago (2010-11-02 03:12:43 UTC) #1
amit
10 years, 1 month ago (2010-11-02 16:49:30 UTC) #2
lg, thanks!

On Mon, Nov 1, 2010 at 8:12 PM, <robertshield@chromium.org> wrote:

> Reviewers: amit,
>
> Description:
> Add explicit build dependencies from npchrome_frame to chrome and
> chrome_dll.
> This is to avoid having non-versioned matched components in the build
> directory
> if only buildingg npchrome_frame which causes all manner of trouble.
>
> BUG=version-mismatched binaries can reside in the build folder when
> building
> only npchrome_frame
> TEST=NONE
>
>
> Please review this at http://codereview.chromium.org/4288001/show
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src/
>
> Affected files:
>  M     chrome_frame/chrome_frame.gyp
>
>
> Index: chrome_frame/chrome_frame.gyp
> ===================================================================
> --- chrome_frame/chrome_frame.gyp       (revision 64609)
> +++ chrome_frame/chrome_frame.gyp       (working copy)
> @@ -839,6 +839,8 @@
>         '../build/temp_gyp/googleurl.gyp:googleurl',
>         'chrome_frame_launcher.gyp:chrome_frame_helper',
>         'chrome_frame_launcher.gyp:chrome_frame_helper_dll',
> +        '../chrome/chrome.gyp:chrome',
> +        '../chrome/chrome.gyp:chrome_dll',
>         '../chrome/chrome.gyp:chrome_version_header',
>         '../chrome/chrome.gyp:common',
>         '../chrome/chrome.gyp:utility',
>
>
>

Powered by Google App Engine
This is Rietveld 408576698