diff options
| author | hiroshige <hiroshige@chromium.org> | 2015-02-23 23:54:06 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-02-24 07:54:29 +0000 |
| commit | e6d374cf7ead69b3ff110f9d7dfba0dee921e6b7 (patch) | |
| tree | e23bc296abe16faa3f2b5f96c918689f3929b261 /third_party/libexif | |
| parent | 57b85942d40ea1e9fe0d933434392bc67b23e8a3 (diff) | |
| download | chromium_src-e6d374cf7ead69b3ff110f9d7dfba0dee921e6b7.zip chromium_src-e6d374cf7ead69b3ff110f9d7dfba0dee921e6b7.tar.gz chromium_src-e6d374cf7ead69b3ff110f9d7dfba0dee921e6b7.tar.bz2 | |
Revert of add gn_all, gyp_all targets to build files, remove GN's //:root (patchset #18 id:350001 of https://codereview.chromium.org/936193004/)
Reason for revert:
Suspecting this breaks Mac builds.
http://build.chromium.org/p/chromium/builders/Mac/builds/33993
Original issue's description:
> add gn_all, gyp_all targets to the build files, remove GN's //:root
>
> In order to be able to make sure we are building the same list of targets
> in both the GYP and GN builds, this patch adds new 'gn_all' and 'gyp_all'
> targets.
>
> Eventually, we should make sure that if one builds 'gn_all', then building
> 'gyp_all', 'All', or 'all' then has nothing to do. We're not there yet, but this
> is a start.
>
> The 'gn_all' target replaces the //:root target in the GN builds.
>
> R=brettw@chromium.org
> BUG=461019
> CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg
>
> Committed: https://crrev.com/ff30e3de333305ccdaa103aa7f9e46765da83e99
> Cr-Commit-Position: refs/heads/master@{#317760}
TBR=brettw@chromium.org,dpranke@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=461019
Review URL: https://codereview.chromium.org/944573003
Cr-Commit-Position: refs/heads/master@{#317763}
Diffstat (limited to 'third_party/libexif')
| -rw-r--r-- | third_party/libexif/BUILD.gn | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/third_party/libexif/BUILD.gn b/third_party/libexif/BUILD.gn index de9b2ef..be8caf6 100644 --- a/third_party/libexif/BUILD.gn +++ b/third_party/libexif/BUILD.gn @@ -4,19 +4,10 @@ # Linux should use system libexif if (!is_linux || is_chromeos) { + # TODO(thestig): Properly support building on Linux. config("libexif_config") { include_dirs = [ "sources" ] - if (is_clang) { - cflags = [ - "-Wno-enum-conversion", - "-Wno-switch", - "-Wno-format", - - # libexif uses fabs(int) to cast to float. - "-Wno-absolute-value", - ] - } } static_library("libexif") { @@ -60,7 +51,9 @@ if (!is_linux || is_chromeos) { ldflags = [ "/DEF:" + rebase_path("libexif.def") ] - cflags = [ "/wd4018" ] # size/unsigned mismatch + cflags = [ + "/wd4018", # size/unsigned mismatch + ] # TODO(GYP): Handle /analyze switch, when it will be used in GN. } else if (is_linux) { |