Merge lp:~alan-griffiths/miral/debian-control-files into lp:miral

Proposed by Alan Griffiths on 2016-08-08
Status: Merged
Approved by: Alan Griffiths on 2016-08-10
Approved revision: 267
Merged at revision: 267
Proposed branch: lp:~alan-griffiths/miral/debian-control-files
Merge into: lp:miral
Diff against target: 228 lines (+133/-5)
13 files modified
CMakeLists.txt (+2/-0)
debian/changelog (+5/-0)
debian/compat (+1/-0)
debian/control (+67/-0)
debian/copyright (+20/-0)
debian/libmiral-dev.install (+3/-0)
debian/libmiral1.install (+1/-0)
debian/miral-doc.install (+1/-0)
debian/miral-examples.install (+5/-0)
debian/rules (+24/-0)
miral-kiosk/CMakeLists.txt (+1/-1)
miral-shell/CMakeLists.txt (+1/-1)
miral/CMakeLists.txt (+2/-3)
To merge this branch: bzr merge lp:~alan-griffiths/miral/debian-control-files
Reviewer Review Type Date Requested Status
Alexandros Frantzis (community) 2016-08-08 Approve on 2016-08-10
Review via email: mp+302274@code.launchpad.net

Commit Message

First cut at debian files

To post a comment you must log in.
Alexandros Frantzis (afrantzis) wrote :

Basic structure is good. A few issues:

+ -- Alan Griffiths <alan@Octopull-desktop> Mon, 08 Aug 2016 10:41:55 +0100

Invalid email address.

+Standards-Version: 3.9.4

You can use the latest standards version 3.9.8 (only available on yakkety lintian, but it can be used with harmless warning in previous releases too).

+DEB_HOST_ARCH ?= $(shell dpkg-architecture -qDEB_HOST_ARCH)
+DEB_HOST_ARCH_ENDIAN ?= $(shell dpkg-architecture -qDEB_HOST_ARCH_ENDIAN)
+EXACT_PACKAGE_VERSION = $(shell dpkg-parsechangelog | grep Version | cut -d' ' -f 2)

Not used.

+COMMON_CONFIGURE_OPTIONS = \

The configure options are set, but not used.

> miral source: no-debian-copyright

Need a debian/copyright file.

> libmiral1: arch-dependent-file-not-in-arch-specific-directory usr/lib/libmiral.so.1

Should be in /usr/lib/<arch>/... especially since we claim to support multi-arch. This is an effect of not using COMMON_CONFIGURE_OPTIONS.

> miral source: no-debian-copyright

Need a debian/copyright file.

> libmiral1: arch-dependent-file-not-in-arch-specific-directory usr/lib/libmiral.so.1

Should be in /usr/lib/<arch>/... especially since we claim to support multi-arch.
A few other lintian complaints:

> miral source: no-debian-copyright

Need a debian/copyright file.

> libmiral1: arch-dependent-file-not-in-arch-specific-directory usr/lib/libmiral.so.1

Should be in /usr/lib/<arch>/... especially since we claim to support multi-arch.

> libmiral-dev: extended-description-line-too-long

Self-explanatory.

review: Needs Fixing
Alexandros Frantzis (afrantzis) wrote :

Somehow the lintian complaints and notes got duplicated in the previous comment...

262. By Alan Griffiths on 2016-08-08

Fix email address

263. By Alan Griffiths on 2016-08-08

Attempt to fix review comments

264. By Alan Griffiths on 2016-08-08

Fix CMakeLists.txt to match

265. By Alan Griffiths on 2016-08-08

Remove debug code

Alexandros Frantzis (afrantzis) wrote :

Nits:

+ cmake-data,

Not needed, cmake depend on cmake-data.

+Standards-Version: 3.9.4

Still using an ancient version (not an error, but lintian complains "ancient-standards-version")

review: Approve
266. By Alan Griffiths on 2016-08-10

Fix nits

267. By Alan Griffiths on 2016-08-10

merge lp:miral

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2016-08-04 10:35:41 +0000
3+++ CMakeLists.txt 2016-08-10 09:40:43 +0000
4@@ -5,6 +5,8 @@
5 set(CMAKE_INSTALL_PREFIX "/usr" CACHE PATH "/usr install prefix" FORCE)
6 endif()
7
8+include (GNUInstallDirs)
9+
10 cmake_policy(SET CMP0015 NEW)
11 cmake_policy(SET CMP0022 NEW)
12
13
14=== added directory 'debian'
15=== added file 'debian/changelog'
16--- debian/changelog 1970-01-01 00:00:00 +0000
17+++ debian/changelog 2016-08-10 09:40:43 +0000
18@@ -0,0 +1,5 @@
19+miral (0.1.0) UNRELEASED; urgency=medium
20+
21+ * Initial release. (Closes: #XXXXXX)
22+
23+ -- Alan Griffiths <alan.griffiths@canonical.com> Mon, 08 Aug 2016 10:41:55 +0100
24
25=== added file 'debian/compat'
26--- debian/compat 1970-01-01 00:00:00 +0000
27+++ debian/compat 2016-08-10 09:40:43 +0000
28@@ -0,0 +1,1 @@
29+9
30
31=== added file 'debian/control'
32--- debian/control 1970-01-01 00:00:00 +0000
33+++ debian/control 2016-08-10 09:40:43 +0000
34@@ -0,0 +1,67 @@
35+Source: miral
36+Section: x11
37+Priority: optional
38+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
39+XSBC-Original-Maintainer: Alan Griffiths <alan.griffiths@canonical.com>
40+Build-Depends: cmake,
41+ pkg-config,
42+ debhelper (>= 9),
43+ doxygen,
44+ xsltproc,
45+ graphviz,
46+ pkg-config,
47+ python-pil,
48+ google-mock (>= 1.6.0+svn437),
49+ python3:any,
50+ libmirserver-dev,
51+ libmirclient-dev,
52+ mirtest-dev,
53+Standards-Version: 3.9.8
54+Homepage: https://launchpad.net/miral
55+# If you aren't a member of ~mir-team but need to upload packaging changes,
56+# just go ahead. ~mir-team will notice and sync up the code again.
57+Vcs-Bzr: https://code.launchpad.net/miral
58+
59+Package: libmiral1
60+Section: libs
61+Architecture: linux-any
62+Multi-Arch: same
63+Pre-Depends: ${misc:Pre-Depends}
64+Depends: ${misc:Depends},
65+ ${shlibs:Depends},
66+Description: Display server for Ubuntu - ABI preserving abstraction layer
67+ Mir is a display server running on linux systems, with a focus on efficiency,
68+ robust operation and a well-defined driver model.
69+ .
70+ Contains the shared library containing MirAL abstraction layer.
71+
72+Package: libmiral-dev
73+Section: libdevel
74+Architecture: linux-any
75+Multi-Arch: same
76+Pre-Depends: ${misc:Pre-Depends}
77+Depends: libmiral1 (= ${binary:Version}),
78+ ${misc:Depends},
79+Description: Display server for Ubuntu - ABI preserving abstraction layer
80+ Mir is a display server running on linux systems, with a focus on efficiency
81+ robust operation and a well-defined driver model.
82+ .
83+ Contains header files required for development using the MirAL abstraction
84+ layer.
85+
86+Package: miral-examples
87+Architecture: linux-any
88+Depends: ${misc:Depends},
89+ ${shlibs:Depends},
90+Description: Display server for Ubuntu - demonstration programs
91+ Mir is a display server running on linux systems, with a focus on efficiency,
92+ robust operation and a well-defined driver model.
93+ .
94+ Contains demo applications that use the MirAL abstraction layer
95+
96+Package: miral-doc
97+Section: doc
98+Architecture: all
99+Depends: ${misc:Depends},
100+Description: API documentation for MirAL
101+ This package installs the MirAL abstraction layer documentation.
102
103=== added file 'debian/copyright'
104--- debian/copyright 1970-01-01 00:00:00 +0000
105+++ debian/copyright 2016-08-10 09:40:43 +0000
106@@ -0,0 +1,20 @@
107+Format: http://dep.debian.net/deps/dep5/
108+Upstream-Name: MirAL
109+Upstream-Contact: Alan Griffiths <alan.griffiths@canonical.com>
110+Source: http://launchpad.net/miral
111+
112+Files: *
113+Copyright: 2016, Canonical Ltd.
114+License: GPL-3
115+ This program is free software: you can redistribute it and/or modify
116+ it under the terms of the GNU General Public License version 3 as
117+ published by the Free Software Foundation.
118+ .
119+ This program is distributed in the hope that it will be useful,
120+ but WITHOUT ANY WARRANTY; without even the implied warranty of
121+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
122+ GNU General Public License for more details.
123+ .
124+ On Debian systems, the full text of the GNU General Public
125+ License version 3 can be found in the file
126+ `/usr/share/common-licenses/GPL-3'.
127
128=== added file 'debian/libmiral-dev.install'
129--- debian/libmiral-dev.install 1970-01-01 00:00:00 +0000
130+++ debian/libmiral-dev.install 2016-08-10 09:40:43 +0000
131@@ -0,0 +1,3 @@
132+usr/include/miral
133+usr/lib/*/pkgconfig/miral.pc
134+usr/lib/*/libmiral.so
135
136=== added file 'debian/libmiral1.install'
137--- debian/libmiral1.install 1970-01-01 00:00:00 +0000
138+++ debian/libmiral1.install 2016-08-10 09:40:43 +0000
139@@ -0,0 +1,1 @@
140+usr/lib/*/libmiral.so.1
141
142=== added file 'debian/miral-doc.install'
143--- debian/miral-doc.install 1970-01-01 00:00:00 +0000
144+++ debian/miral-doc.install 2016-08-10 09:40:43 +0000
145@@ -0,0 +1,1 @@
146+usr/share/doc/miral-doc/
147
148=== added file 'debian/miral-examples.install'
149--- debian/miral-examples.install 1970-01-01 00:00:00 +0000
150+++ debian/miral-examples.install 2016-08-10 09:40:43 +0000
151@@ -0,0 +1,5 @@
152+usr/bin/miral-shell
153+usr/bin/miral-run
154+usr/bin/miral-kiosk
155+usr/share/applications/miral-shell.desktop
156+usr/share/icons/hicolor/scalable/apps/ubuntu-logo.svg
157
158=== added file 'debian/rules'
159--- debian/rules 1970-01-01 00:00:00 +0000
160+++ debian/rules 2016-08-10 09:40:43 +0000
161@@ -0,0 +1,24 @@
162+#!/usr/bin/make -f
163+
164+DEB_HOST_MULTIARCH ?= $(shell dpkg-architecture -qDEB_HOST_MULTIARCH)
165+
166+export DPKG_GENSYMBOLS_CHECK_LEVEL = 4
167+
168+%:
169+ dh $@ --parallel --fail-missing --with python3
170+
171+COMMON_CONFIGURE_OPTIONS = \
172+ -DCMAKE_INSTALL_LIBEXECDIR="lib/$(DEB_HOST_MULTIARCH)/miral"\
173+
174+ifeq ($(filter noopt,$(DEB_BUILD_OPTIONS)),noopt)
175+ COMMON_CONFIGURE_OPTIONS += -DMIRAL_USE_LTO=OFF
176+else
177+ COMMON_CONFIGURE_OPTIONS += -DMIRAL_USE_LTO=ON
178+endif
179+
180+override_dh_auto_configure:
181+ dh_auto_configure -- $(COMMON_CONFIGURE_OPTIONS)
182+
183+# Only build the docs when we need them
184+override_dh_auto_build-indep:
185+ dh_auto_build -- doc
186
187=== modified file 'miral-kiosk/CMakeLists.txt'
188--- miral-kiosk/CMakeLists.txt 2016-04-16 23:13:59 +0000
189+++ miral-kiosk/CMakeLists.txt 2016-08-10 09:40:43 +0000
190@@ -7,5 +7,5 @@
191 target_link_libraries(miral-kiosk miral)
192
193 install(TARGETS miral-kiosk
194- DESTINATION ${CMAKE_INSTALL_PREFIX}/bin
195+ DESTINATION ${CMAKE_INSTALL_BINDIR}
196 )
197
198=== modified file 'miral-shell/CMakeLists.txt'
199--- miral-shell/CMakeLists.txt 2016-07-25 09:02:46 +0000
200+++ miral-shell/CMakeLists.txt 2016-08-10 09:40:43 +0000
201@@ -22,5 +22,5 @@
202 )
203
204 install(TARGETS miral-shell
205- DESTINATION ${CMAKE_INSTALL_PREFIX}/bin
206+ DESTINATION ${CMAKE_INSTALL_BINDIR}
207 )
208
209=== modified file 'miral/CMakeLists.txt'
210--- miral/CMakeLists.txt 2016-08-04 16:26:20 +0000
211+++ miral/CMakeLists.txt 2016-08-10 09:40:43 +0000
212@@ -49,7 +49,6 @@
213 LINK_DEPENDS ${symbol_map}
214 )
215
216-set(LIBDIR "${CMAKE_INSTALL_PREFIX}/lib")
217 set(INCLUDEDIR "${CMAKE_INSTALL_PREFIX}/include/miral")
218
219 configure_file(${CMAKE_CURRENT_SOURCE_DIR}/miral.pc.in
220@@ -57,6 +56,6 @@
221 @ONLY
222 )
223
224-install(TARGETS miral LIBRARY DESTINATION "${LIBDIR}")
225+install(TARGETS miral LIBRARY DESTINATION "${CMAKE_INSTALL_FULL_LIBDIR}")
226 install(DIRECTORY ${CMAKE_SOURCE_DIR}/include/ DESTINATION "${INCLUDEDIR}")
227-install(FILES ${CMAKE_CURRENT_BINARY_DIR}/miral.pc DESTINATION "${LIBDIR}/pkgconfig")
228+install(FILES ${CMAKE_CURRENT_BINARY_DIR}/miral.pc DESTINATION "${CMAKE_INSTALL_FULL_LIBDIR}/pkgconfig")

Subscribers

People subscribed via source and target branches