You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by sh...@apache.org on 2019/03/10 00:32:30 UTC

[arrow] branch master updated: ARROW-4801: [GLib] Suppress Meson warnings

This is an automated email from the ASF dual-hosted git repository.

shiro pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new 17500f7  ARROW-4801: [GLib] Suppress Meson warnings
17500f7 is described below

commit 17500f781282f21a68ea818d21a9006df3ee0d42
Author: Kouhei Sutou <ko...@clear-code.com>
AuthorDate: Sun Mar 10 09:32:15 2019 +0900

    ARROW-4801: [GLib] Suppress Meson warnings
    
    Messages:
    
        arrow-glib/meson.build:211: DEPRECATION: Library arrow-glib was passed to the "libraries" keyword argument of a previous call to generate() method instead of first positional argument. Adding arrow-glib to "Requires" field, but this is a deprecated behaviour that will change in a future version of Meson. Please report the issue if this warning cannot be avoided in your case.
        arrow-cuda-glib/meson.build:53: DEPRECATION: Library arrow-cuda-glib was passed to the "libraries" keyword argument of a previous call to generate() method instead of first positional argument. Adding arrow-cuda-glib to "Requires" field, but this is a deprecated behaviour that will change in a future version of Meson. Please report the issue if this warning cannot be avoided in your case.
    
    Author: Kouhei Sutou <ko...@clear-code.com>
    
    Closes #3840 from kou/glib-meson-suppress-warning and squashes the following commits:
    
    3c0b0bbc <Kouhei Sutou>  Suppress Meson warnings
---
 c_glib/arrow-cuda-glib/meson.build |  6 +++---
 c_glib/arrow-glib/meson.build      |  6 +++---
 c_glib/gandiva-glib/meson.build    |  6 +++---
 c_glib/parquet-glib/meson.build    | 12 ++++++------
 c_glib/plasma-glib/meson.build     |  6 +++---
 5 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/c_glib/arrow-cuda-glib/meson.build b/c_glib/arrow-cuda-glib/meson.build
index e5b9f47..a29f12b 100644
--- a/c_glib/arrow-cuda-glib/meson.build
+++ b/c_glib/arrow-cuda-glib/meson.build
@@ -50,12 +50,12 @@ arrow_cuda_glib = declare_dependency(link_with: libarrow_cuda_glib,
                                      include_directories: base_include_directories,
                                      dependencies: dependencies)
 
-pkgconfig.generate(filebase: 'arrow-cuda-glib',
+pkgconfig.generate(libarrow_cuda_glib,
+                   filebase: 'arrow-cuda-glib',
                    name: 'Apache Arrow CUDA GLib',
                    description: 'C API for Apache Arrow CUDA based on GLib',
                    version: version,
-                   requires: ['arrow-glib', 'arrow-cuda'],
-                   libraries: [libarrow_cuda_glib])
+                   requires: ['arrow-glib', 'arrow-cuda'])
 
 gir_dependencies = [
   declare_dependency(sources: arrow_glib_gir),
diff --git a/c_glib/arrow-glib/meson.build b/c_glib/arrow-glib/meson.build
index 14126be..4e8e1cc 100644
--- a/c_glib/arrow-glib/meson.build
+++ b/c_glib/arrow-glib/meson.build
@@ -208,12 +208,12 @@ arrow_glib = declare_dependency(link_with: libarrow_glib,
                                 dependencies: dependencies,
                                 sources: enums_header)
 
-pkgconfig.generate(filebase: meson.project_name(),
+pkgconfig.generate(libarrow_glib,
+                   filebase: meson.project_name(),
                    name: 'Apache Arrow GLib',
                    description: 'C API for Apache Arrow based on GLib',
                    version: version,
-                   requires: ['gio-2.0', 'arrow'],
-                   libraries: [libarrow_glib])
+                   requires: ['gio-2.0', 'arrow'])
 if have_arrow_orc
   pkgconfig.generate(filebase: 'arrow-orc-glib',
                      name: 'Apache Arrow GLib ORC',
diff --git a/c_glib/gandiva-glib/meson.build b/c_glib/gandiva-glib/meson.build
index 7c288e4..11ef8d6 100644
--- a/c_glib/gandiva-glib/meson.build
+++ b/c_glib/gandiva-glib/meson.build
@@ -58,12 +58,12 @@ gandiva_glib = declare_dependency(link_with: libgandiva_glib,
                                   dependencies: dependencies,
                                   sources: enums_header)
 
-pkgconfig.generate(filebase: project_name,
+pkgconfig.generate(libgandiva_glib,
+                   filebase: project_name,
                    name: 'Apache Arrow Gandiva GLib',
                    description: 'C API for Apache Arrow Gandiva based on GLib',
                    version: version,
-                   requires: ['gandiva', 'arrow-glib'],
-                   libraries: [libgandiva_glib])
+                   requires: ['gandiva', 'arrow-glib'])
 
 gnome.generate_gir(libgandiva_glib,
                    dependencies: declare_dependency(sources: arrow_glib_gir),
diff --git a/c_glib/parquet-glib/meson.build b/c_glib/parquet-glib/meson.build
index afa6d21..a52b328 100644
--- a/c_glib/parquet-glib/meson.build
+++ b/c_glib/parquet-glib/meson.build
@@ -56,12 +56,12 @@ parquet_glib = declare_dependency(link_with: libparquet_glib,
                                   include_directories: base_include_directories,
                                   dependencies: dependencies)
 
-pkgconfig.generate(filebase: project_name,
+pkgconfig.generate(libparquet_glib,
+                   filebase: project_name,
                    name: 'Apache Parquet GLib',
                    description: 'C API for Apache Parquet based on GLib',
                    version: version,
-                   requires: ['parquet', 'arrow-glib'],
-                   libraries: [libparquet_glib])
+                   requires: ['parquet', 'arrow-glib'])
 
 gnome.generate_gir(libparquet_glib,
                    dependencies: declare_dependency(sources: arrow_glib_gir),
@@ -72,10 +72,10 @@ gnome.generate_gir(libparquet_glib,
                    symbol_prefix: 'gparquet',
                    export_packages: 'parquet-glib',
                    includes: [
-		     'Arrow-1.0',
+                     'Arrow-1.0',
                    ],
                    install: true,
                    extra_args: [
-		     '--warn-all',
-		     '--include-uninstalled=./arrow-glib/Arrow-1.0.gir',
+                     '--warn-all',
+                     '--include-uninstalled=./arrow-glib/Arrow-1.0.gir',
                    ])
diff --git a/c_glib/plasma-glib/meson.build b/c_glib/plasma-glib/meson.build
index 75ebce8..31a5233 100644
--- a/c_glib/plasma-glib/meson.build
+++ b/c_glib/plasma-glib/meson.build
@@ -81,12 +81,12 @@ plasma_glib = declare_dependency(link_with: libplasma_glib,
                                  include_directories: base_include_directories,
                                  dependencies: dependencies)
 
-pkgconfig.generate(filebase: project_name,
+pkgconfig.generate(libplasma_glib,
+                   filebase: project_name,
                    name: 'Apache Arrow Plasma GLib',
                    description: 'C API for Apache Arrow Plasma based on GLib',
                    version: version,
-                   requires: pkg_config_requires,
-                   libraries: [libplasma_glib])
+                   requires: pkg_config_requires)
 
 gnome.generate_gir(libplasma_glib,
                    dependencies: gir_dependencies,