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

Unified Diff: build/linux/system.gyp

Issue 11437025: Linux: add library loader for GNOME keyring. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix for bots Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/password_manager/native_backend_gnome_x.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/linux/system.gyp
diff --git a/build/linux/system.gyp b/build/linux/system.gyp
index 041afd54072d664bd83734d37e0832142c721c1e..6c4488ae73ca3a059c44aa7878992afab1bf4361 100644
--- a/build/linux/system.gyp
+++ b/build/linux/system.gyp
@@ -493,9 +493,15 @@
},
{
'target_name': 'gnome_keyring',
- 'type': 'none',
+ 'type': 'static_library',
'conditions': [
['use_gnome_keyring==1', {
+ 'dependencies': [
+ '../../base/base.gyp:base',
+ ],
+ 'cflags': [
+ '<!@(<(pkg-config) --cflags gnome-keyring-1)',
+ ],
'direct_dependent_settings': {
'cflags': [
'<!@(<(pkg-config) --cflags gnome-keyring-1)',
@@ -503,33 +509,58 @@
'defines': [
'USE_GNOME_KEYRING',
],
- 'conditions': [
- ['linux_link_gnome_keyring==0', {
- 'defines': ['DLOPEN_GNOME_KEYRING'],
- }],
+ 'include_dirs': [
+ '<(SHARED_INTERMEDIATE_DIR)',
],
},
- 'conditions': [
- ['linux_link_gnome_keyring!=0', {
- 'link_settings': {
+ 'link_settings': {
+ 'conditions': [
+ ['linux_link_gnome_keyring==1', {
'ldflags': [
'<!@(<(pkg-config) --libs-only-L --libs-only-other gnome-keyring-1)',
],
'libraries': [
'<!@(<(pkg-config) --libs-only-l gnome-keyring-1)',
],
+ }, { # linux_link_gnome_keyring==0
+ 'libraries': [
+ '-ldl',
+ ],
+ }],
+ ],
+ },
+ 'hard_dependency': 1,
+ 'actions': [
+ {
+ 'variables': {
+ 'output_h': '<(SHARED_INTERMEDIATE_DIR)/library_loaders/libgnome-keyring.h',
+ 'output_cc': '<(INTERMEDIATE_DIR)/libgnome_keyring_loader.cc',
+ 'generator': '../../tools/generate_library_loader/generate_library_loader.py',
},
- }, {
- 'conditions': [
- ['OS=="linux"', {
- 'link_settings': {
- 'libraries': [
- '-ldl',
- ],
- },
- }],
+ 'action_name': 'generate_libgnome_keyring_loader',
+ 'inputs': [
+ '<(generator)',
],
- }],
+ 'outputs': [
+ '<(output_h)',
+ '<(output_cc)',
+ ],
+ 'action': ['python',
+ '<(generator)',
+ '--name', 'LibGnomeKeyringLoader',
+ '--output-h', '<(output_h)',
+ '--output-cc', '<(output_cc)',
+ '--header', '<gnome-keyring.h>',
+ '--link-directly=<(linux_link_gnome_keyring)',
+ 'gnome_keyring_is_available',
+ 'gnome_keyring_store_password',
+ 'gnome_keyring_delete_password',
+ 'gnome_keyring_find_itemsv',
+ 'gnome_keyring_result_to_message',
+ ],
+ 'message': 'Generating libgnome-keyring library loader.',
+ 'process_outputs_as_sources': 1,
+ },
],
}],
],
« no previous file with comments | « no previous file | chrome/browser/password_manager/native_backend_gnome_x.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698