• Skip to sidebar navigation
  • Skip to content

Bitbucket

  • More
    ProjectsRepositories
  • Help
    • Online help
    • Learn Git
    • Welcome to Bitbucket
    • Keyboard shortcuts
  • Log In
Alexander Dahl
  1. Alexander Dahl

buildroot

Public
Actions
  • Clone
  • Compare

Learn more about cloning repositories

You have read-only access

Navigation
  • Source
  • Commits
  • Branches
  • All Branches Graph
  • Forks
  1. Alexander Dahl
  2. buildroot

Source

buildroot/package/kodi/0003-cmake-iconv-is-a-required-dependency.patch
Bernd KuhlsBernd Kuhls committed f6a33a73e2310 Nov 2017
Raw file
Source viewDiff to previous
 
1
From 70ab56d74aff8b2e3ac49fed6bdf3751c9b1457e Mon Sep 17 00:00:00 2001
2
From: Bernd Kuhls <bernd.kuhls@t-online.de>
3
Date: Sun, 12 Feb 2017 14:24:18 +0100
4
Subject: [PATCH] [cmake] iconv is a required dependency
5
​
6
This patch adds support for libiconv currently only provided by the
7
autoconf-based build system:
8
https://github.com/xbmc/xbmc/blob/Krypton/configure.ac#L1172
9
​
10
This commit fixes an error during linking with an uClibc-based
11
buildroot toolchain:
12
​
13
[100%] Linking CXX executable kodi.bin
14
/home/buildroot/br8_ffmpeg3_kodi17_github/output/host/usr/lib/gcc/i586-buildroot-linux-uclibc/6.3.0/../../../../i586-buildroot-linux-uclibc/bin/ld: build/utils/utils.a(CharsetConverter.cpp.o): undefined reference to symbol 'libiconv_open'
15
/home/buildroot/br8_ffmpeg3_kodi17_github/output/host/usr/i586-buildroot-linux-uclibc/sysroot/usr/lib32/libiconv.so.2: error adding symbols: DSO missing from command line
16
​
17
Backported to Krypton from master branch commit:
18
https://github.com/xbmc/xbmc/commit/9a64537543e8dc8609ca8a98181ba17f30c53493
19
​
20
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
21
---
22
 project/cmake/CMakeLists.txt          |  2 +-
23
 project/cmake/modules/FindIconv.cmake | 44 +++++++++++++++++++++++++++++++++++
24
 2 files changed, 45 insertions(+), 1 deletion(-)
25
 create mode 100644 project/cmake/modules/FindIconv.cmake
26
​
27
diff --git a/project/cmake/CMakeLists.txt b/project/cmake/CMakeLists.txt
28
index aeb1ff47c2..07c1d1a8d3 100644
29
--- a/project/cmake/CMakeLists.txt
30
+++ b/project/cmake/CMakeLists.txt
31
@@ -103,7 +103,7 @@ list(APPEND DEPLIBS ${CMAKE_THREAD_LIBS_INIT})
32
 
33
 # Required dependencies
34
 set(required_deps Sqlite3 FreeType PCRE Cpluff LibDvd
35
-                  TinyXML Python Yajl Cdio
36
+                  TinyXML Python Yajl Cdio Iconv
37
                   Lzo2 Fribidi TagLib FFMPEG CrossGUID)
38
 if(NOT WIN32)
39
   list(APPEND required_deps ZLIB)
40
diff --git a/project/cmake/modules/FindIconv.cmake b/project/cmake/modules/FindIconv.cmake
41
new file mode 100644
42
index 0000000000..8ee01fb6b8
43
--- /dev/null
44
+++ b/project/cmake/modules/FindIconv.cmake
45
@@ -0,0 +1,44 @@
46
+#.rst:
47
+# FindICONV
48
+# --------
49
+# Finds the ICONV library
50
+#
51
+# This will will define the following variables::
52
+#
53
+# ICONV_FOUND - system has ICONV
54
+# ICONV_INCLUDE_DIRS - the ICONV include directory
55
+# ICONV_LIBRARIES - the ICONV libraries
56
+#
57
+# and the following imported targets::
58
+#
59
+#   ICONV::ICONV   - The ICONV library
60
+
61
+find_path(ICONV_INCLUDE_DIR NAMES iconv.h)
62
+
63
+find_library(ICONV_LIBRARY NAMES iconv libiconv c)
  • Git repository management for enterprise teams powered by Atlassian Bitbucket
  • Atlassian Bitbucket v6.7.2
  • Documentation
  • Request a feature
  • About
  • Contact Atlassian
Atlassian

Everything looks good. We'll let you know here if there's anything you should know about.