Add BUILD.gn for libflac.

BUILD.gn

+# Copyright 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+source_set("flac") {
+  sources = [
+    "include/FLAC/all.h",
+    "include/FLAC/assert.h",
+    "include/FLAC/callback.h",
+    "include/FLAC/export.h",
+    "include/FLAC/format.h",
+    "include/FLAC/metadata.h",
+    "include/FLAC/ordinals.h",
+    "include/FLAC/stream_decoder.h",
+    "include/FLAC/stream_encoder.h",
+    "include/share/alloc.h",
+    "src/libFLAC/alloc.c",
+    "src/libFLAC/bitmath.c",
+    "src/libFLAC/bitreader.c",
+    "src/libFLAC/bitwriter.c",
+    "src/libFLAC/cpu.c",
+    "src/libFLAC/crc.c",
+    "src/libFLAC/fixed.c",
+    "src/libFLAC/float.c",
+    "src/libFLAC/format.c",
+    "src/libFLAC/lpc.c",
+    "src/libFLAC/md5.c",
+    "src/libFLAC/memory.c",
+    "src/libFLAC/stream_decoder.c",
+    "src/libFLAC/stream_encoder.c",
+    "src/libFLAC/stream_encoder_framing.c",
+    "src/libFLAC/window.c",
+    "src/libFLAC/include/private/all.h",
+    "src/libFLAC/include/private/bitmath.h",
+    "src/libFLAC/include/private/bitreader.h",
+    "src/libFLAC/include/private/bitwriter.h",
+    "src/libFLAC/include/private/cpu.h",
+    "src/libFLAC/include/private/crc.h",
+    "src/libFLAC/include/private/fixed.h",
+    "src/libFLAC/include/private/float.h",
+    "src/libFLAC/include/private/format.h",
+    "src/libFLAC/include/private/lpc.h",
+    "src/libFLAC/include/private/md5.h",
+    "src/libFLAC/include/private/memory.h",
+    "src/libFLAC/include/private/metadata.h",
+    "src/libFLAC/include/private/stream_encoder_framing.h",
+    "src/libFLAC/include/private/window.h",
+    "src/libFLAC/include/protected/all.h",
+    "src/libFLAC/include/protected/stream_decoder.h",
+    "src/libFLAC/include/protected/stream_encoder.h",
+  ]
+
+  config("flac_config") {

[brettw, 2014/04/29 21:35:41]

I'd put this at the top, usually they wouldn't be nested.

[awong, 2014/04/29 21:43:52]

Huh... I misunderstood the style guide. Moved.

Should declarations like this even be allowed to be nested?  That's just
confusing if :flac can name a :flac_config that's lexically subordinate to
:flac.

Also, is it worth name-mangling "flac_" int the config name? It's already scoped
by //third_party/flac:....

[brettw, 2014/04/29 21:47:10]

Yeah, I'm thinking we should disallow nesting but haven't done it.

I think "flac_config" because plain "config" looks weird to me and sometimes
you'll add more later and should have unique names.

+    defines = [
+      "FLAC__NO_DLL",
+    ]
+  }
+
+  configs -= [ "//build/config/compiler:chromium_code" ]
+  configs += [
+    ":flac_config",
+    "//build/config/compiler:no_chromium_code",
+  ]
+  direct_dependent_configs = [ ":flac_config" ]
+
+  include_dirs = [
+    "include",
+    "src/libFLAC/include",
+  ]
+
+  defines = [
+    "FLAC__OVERFLOW_DETECT",
+    "VERSION=\"1.2.1\"",
+  ]
+
+  if (is_clang) {
+    # libflac converts between FLAC__StreamDecoderState and
+    # FLAC__StreamDecoderInitStatus a lot in stream_decoder.c.
+    cflags = [
+      "-Wno-conversion",
+    ]
+  }
+}
+
+# Local Variables:

[brettw, 2014/04/29 21:35:41]

I don't think we want this in all of the build files.

[awong, 2014/04/29 21:43:52]

It's in a bunch of the gyp files...happy to take it out, but at some point it'd
be nice to have good .vim and .emacs settings.

+# tab-width:2
+# indent-tabs-mode:nil
+# End:
+# vim: set expandtab tabstop=2 shiftwidth=2: