Media Galleries API Metadata: Image metadata

third_party/libexif/libexif.gyp

 # Copyright (c) 2012 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.
 
 {
   'variables': {
     'conditions': [
-      # TODO(kmadhusu): We are not ready to build this library on Android.
-      # Resolve the issues and build on Android.
-      ['os_posix==1 and OS!="mac"', {
-        'use_system_libexif%': 0,
-      }, {  # os_posix != 1 or OS == "mac"
+      ['os_posix == 1 and OS != "mac" and OS != "ios" and OS != "android"', {

[Lei Zhang, 2014/03/12 22:49:46]

Maybe just do: OS == Linux ?

[tommycli, 2014/03/12 23:27:21]

Works with me. What's the situation with ChromeOS if we do this?

+        'use_system_libexif%': 1,
+      }, {  # os_posix == 1 and OS != "mac" and OS != "ios" and OS != "android" 
         'use_system_libexif%': 0,
       }],
     ],
   },
   'conditions': [
     ['use_system_libexif==0', {
       'targets': [
         {
           'target_name': 'libexif',
           'type': 'shared_library',
@@ skipping 26 matching lines @@
           ],
           'include_dirs': [
             'sources',
           ],
           'direct_dependent_settings': {
             'include_dirs': [
               'sources',
             ],
           },
           'conditions': [
+            ['clang==1', {
+              'cflags': ['-Wno-enum-conversion', '-Wno-switch'],
+            }],
             ['os_posix==1 and OS!="mac"', {
               'cflags!': ['-fvisibility=hidden'],
             }],
             ['OS=="mac"', {
               'conditions': [
                ['mac_breakpad==1', {
                   'variables': {
                     'mac_real_dsym': 1,
                   },
                }],
@@ skipping 51 matching lines @@
             ],
             'libraries': [
               '<!@(<(pkg-config) --libs-only-l libexif)',
             ],
           },
         }
       ],
     }],
   ]
 }