Linux: adjust include path for system jsoncpp.

third_party/jsoncpp/jsoncpp.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': {
     'use_system_jsoncpp%': 0,
   },
   'conditions': [
     ['use_system_jsoncpp==0', {
@@ skipping 48 matching lines @@
               'json/forwards.h',
               'json/json.h',
               'json/reader.h',
               'json/value.h',
               'json/writer.h',
             ],
           },
           'includes': [
             '../../build/shim_headers.gypi',
           ],
+          'direct_dependent_settings': {
+            'include_dirs': [
+              '/usr/include/jsoncpp',

[tommi (sloooow) - chröme, 2013/02/19 10:30:48]

Would it make sense to have an OS check before adding this path?

[Paweł Hajdan Jr., 2013/02/19 10:37:08]

When non-unices start shipping FOSS libraries as part of the system, sure. ;)

I'm also assuming you're fully aware use_system_foo switches have never been
officially supported.

[tommi (sloooow) - chröme, 2013/02/19 10:54:28]

Nope, I'm not aware of it.  I only stumbled across this change when looking at
build breakage on the waterfall.
Since this is an os specific include path it feels to me that an OS check should
be here (as is the case in other gypi files).

Think of it another way - if this path was
'%PROGRAMFILES%\\TheSDKPath\\Include', wouldn't that at least raise one of your
eyebrows? :)

+            ],
+          },
           'link_settings': {
             'libraries': [
               '-ljsoncpp',
             ],
           },
         }
       ],
     }],
   ],
 }
 
 # Local Variables:
 # tab-width:2
 # indent-tabs-mode:nil
 # End:
 # vim: set expandtab tabstop=2 shiftwidth=2: