Merge ~fourdollars/ipu6-camera-hal/+git/packaging:main into ~oem-solutions-engineers/ipu6-camera-hal/+git/packaging:main

Proposed by Shih-Yuan Lee
Status: Merged
Approved by: You-Sheng Yang
Approved revision: afbc62d926d76cc2c551c33aef97502710a1123d
Merged at revision: afbc62d926d76cc2c551c33aef97502710a1123d
Proposed branch: ~fourdollars/ipu6-camera-hal/+git/packaging:main
Merge into: ~oem-solutions-engineers/ipu6-camera-hal/+git/packaging:main
Diff against target: 179 lines (+23/-33)
5 files modified
CMakeLists.txt (+10/-5)
cmake/libcamhal.pc.cmakein (+2/-2)
debian/changelog (+11/-0)
debian/patches/series (+0/-2)
dev/null (+0/-24)
Reviewer Review Type Date Requested Status
You-Sheng Yang Approve
Review via email: mp+413083@code.launchpad.net
To post a comment you must log in.
Revision history for this message
You-Sheng Yang (vicamo) wrote :

Already pushed tag upstream/0_git202112130639.4585612 to lp:~oem-solutions-engineers/ipu6-camera-hal/+git/packaging.

Revision history for this message
You-Sheng Yang (vicamo) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/CMakeLists.txt b/CMakeLists.txt
2index 3b3b653..6a49e0e 100644
3--- a/CMakeLists.txt
4+++ b/CMakeLists.txt
5@@ -68,6 +68,8 @@ message(STATUS "install prefix: " ${CMAKE_INSTALL_PREFIX})
6
7 #------------------------- Compilation settings -------------------------
8
9+include(GNUInstallDirs)
10+
11 # -fPIC in xos way
12 set(CMAKE_POSITION_INDEPENDENT_CODE ON)
13
14@@ -243,6 +245,11 @@ set_target_properties(camhal_static PROPERTIES OUTPUT_NAME "camhal")
15 target_link_libraries(camhal ${CMAKE_DL_LIBS})
16 target_link_libraries(camhal_static ${CMAKE_DL_LIBS})
17
18+set (THREADS_PREFER_PTHREAD_FLAG ON)
19+find_package (Threads REQUIRED)
20+target_link_libraries(camhal ${CMAKE_THREAD_LIBS_INIT})
21+target_link_libraries(camhal_static ${CMAKE_THREAD_LIBS_INIT})
22+
23 target_link_libraries(camhal expat rt)
24 target_link_libraries(camhal_static expat rt)
25
26@@ -391,15 +398,13 @@ else()
27 endif()
28
29 # Install libraries
30-install(TARGETS camhal camhal_static
31- LIBRARY DESTINATION lib
32- ARCHIVE DESTINATION lib
33- )
34+install(TARGETS camhal camhal_static)
35
36 # Install package config file
37 configure_file(${PROJECT_SOURCE_DIR}/cmake/libcamhal.pc.cmakein
38 ${PROJECT_SOURCE_DIR}/libcamhal.pc @ONLY)
39-install(FILES libcamhal.pc DESTINATION lib/pkgconfig)
40+install(FILES libcamhal.pc
41+ DESTINATION ${CMAKE_INSTALL_LIBDIR}/pkgconfig)
42
43 endif() #NOT CAL_BUILD
44
45diff --git a/cmake/libcamhal.pc.cmakein b/cmake/libcamhal.pc.cmakein
46index fe9e6ab..27ad098 100644
47--- a/cmake/libcamhal.pc.cmakein
48+++ b/cmake/libcamhal.pc.cmakein
49@@ -1,7 +1,7 @@
50 prefix=@CMAKE_INSTALL_PREFIX@
51 exec_prefix=@CMAKE_INSTALL_PREFIX@
52-libdir=@CMAKE_INSTALL_PREFIX@/lib
53-includedir=@CMAKE_INSTALL_PREFIX@/include/libcamhal
54+libdir=${prefix}/@CMAKE_INSTALL_LIBDIR@
55+includedir=${prefix}/@CMAKE_INSTALL_INCLUDEDIR@/libcamhal
56
57 Name: libcamhal
58 Description: Camera HAL Library
59diff --git a/debian/changelog b/debian/changelog
60index 22a571d..c3a0f39 100644
61--- a/debian/changelog
62+++ b/debian/changelog
63@@ -1,3 +1,14 @@
64+ipu6-camera-hal (0~git202112130639.4585612-1) UNRELEASED; urgency=medium
65+
66+ [ You-Sheng Yang ]
67+ * cmake: fix multiarch installation path
68+ * cmake: link pthread properly
69+
70+ [ Shih-Yuan Lee (FourDollars) ]
71+ * debian: Revert "debian: add patches for unexpectedly reverted fixes"
72+
73+ -- Shih-Yuan Lee (FourDollars) <sylee@canonical.com> Mon, 13 Dec 2021 20:32:04 +0800
74+
75 ipu6-camera-hal (0~git202112030433.aa8c9d8-1) UNRELEASED; urgency=medium
76
77 * Intel tributo_20211202 release
78diff --git a/debian/patches/0001-fix-multiarch-build.patch b/debian/patches/0001-fix-multiarch-build.patch
79deleted file mode 100644
80index 7ca5245..0000000
81--- a/debian/patches/0001-fix-multiarch-build.patch
82+++ /dev/null
83@@ -1,55 +0,0 @@
84-From: You-Sheng Yang <vicamo@gmail.com>
85-Date: Mon, 28 Dec 2020 20:12:57 +0800
86-Subject: [PATCH] fix multiarch build
87-
88----
89- CMakeLists.txt | 10 +++++-----
90- cmake/libcamhal.pc.cmakein | 4 ++--
91- 2 files changed, 7 insertions(+), 7 deletions(-)
92-
93-diff --git a/CMakeLists.txt b/CMakeLists.txt
94-index 5a9a41d..19bb36c 100644
95---- a/CMakeLists.txt
96-+++ b/CMakeLists.txt
97-@@ -69,6 +69,8 @@ message(STATUS "install prefix: " ${CMAKE_INSTALL_PREFIX})
98-
99- #------------------------- Compilation settings -------------------------
100-
101-+include(GNUInstallDirs)
102-+
103- # -fPIC in xos way
104- set(CMAKE_POSITION_INDEPENDENT_CODE ON)
105-
106-@@ -371,15 +373,13 @@ else()
107- endif()
108-
109- # Install libraries
110--install(TARGETS camhal camhal_static
111-- LIBRARY DESTINATION lib
112-- ARCHIVE DESTINATION lib
113-- )
114-+install(TARGETS camhal camhal_static)
115-
116- # Install package config file
117- configure_file(${PROJECT_SOURCE_DIR}/cmake/libcamhal.pc.cmakein
118- ${PROJECT_SOURCE_DIR}/libcamhal.pc @ONLY)
119--install(FILES libcamhal.pc DESTINATION lib/pkgconfig)
120-+install(FILES libcamhal.pc
121-+ DESTINATION ${CMAKE_INSTALL_LIBDIR}/pkgconfig)
122-
123- endif() #NOT CAL_BUILD
124-
125-diff --git a/cmake/libcamhal.pc.cmakein b/cmake/libcamhal.pc.cmakein
126-index fe9e6ab..27ad098 100644
127---- a/cmake/libcamhal.pc.cmakein
128-+++ b/cmake/libcamhal.pc.cmakein
129-@@ -1,7 +1,7 @@
130- prefix=@CMAKE_INSTALL_PREFIX@
131- exec_prefix=@CMAKE_INSTALL_PREFIX@
132--libdir=@CMAKE_INSTALL_PREFIX@/lib
133--includedir=@CMAKE_INSTALL_PREFIX@/include/libcamhal
134-+libdir=${prefix}/@CMAKE_INSTALL_LIBDIR@
135-+includedir=${prefix}/@CMAKE_INSTALL_INCLUDEDIR@/libcamhal
136-
137- Name: libcamhal
138- Description: Camera HAL Library
139diff --git a/debian/patches/0003-cmake-link-pthread-properly.patch b/debian/patches/0003-cmake-link-pthread-properly.patch
140deleted file mode 100644
141index 8dbe406..0000000
142--- a/debian/patches/0003-cmake-link-pthread-properly.patch
143+++ /dev/null
144@@ -1,24 +0,0 @@
145-From: You-Sheng Yang <vicamo@gmail.com>
146-Date: Mon, 28 Dec 2020 22:15:59 +0800
147-Subject: [PATCH] cmake: link pthread properly
148-
149----
150- CMakeLists.txt | 5 +++++
151- 1 file changed, 5 insertions(+)
152-
153-diff --git a/CMakeLists.txt b/CMakeLists.txt
154-index 8aaddda..1146af3 100644
155---- a/CMakeLists.txt
156-+++ b/CMakeLists.txt
157-@@ -243,6 +243,11 @@ set_target_properties(camhal_static PROPERTIES OUTPUT_NAME "camhal")
158- target_link_libraries(camhal ${CMAKE_DL_LIBS})
159- target_link_libraries(camhal_static ${CMAKE_DL_LIBS})
160-
161-+set (THREADS_PREFER_PTHREAD_FLAG ON)
162-+find_package (Threads REQUIRED)
163-+target_link_libraries(camhal ${CMAKE_THREAD_LIBS_INIT})
164-+target_link_libraries(camhal_static ${CMAKE_THREAD_LIBS_INIT})
165-+
166- target_link_libraries(camhal expat rt)
167- target_link_libraries(camhal_static expat rt)
168-
169diff --git a/debian/patches/series b/debian/patches/series
170index be69491..554fc13 100644
171--- a/debian/patches/series
172+++ b/debian/patches/series
173@@ -1,6 +1,4 @@
174-0001-fix-multiarch-build.patch
175 0002-cmake-specify-library-version.patch
176-0003-cmake-link-pthread-properly.patch
177 0004-cmake-correct-CAMERA_DEFAULT_CFG_PATH.patch
178 0005-cmake-link-to-corresponding-ia_imaging-gcss-libs.patch
179 0006-cmake-build-libcamhal.so-from-static-lib.patch

Subscribers

People subscribed via source and target branches