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 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Alexandros Frantzis (community) | 2016-08-08 | Approve on 2016-08-10 | |
|
Review via email:
|
|||
Commit Message
First cut at debian files
To post a comment you must log in.
| Alexandros Frantzis (afrantzis) wrote : | # |
Somehow the lintian complaints and notes got duplicated in the previous comment...
lp:~alan-griffiths/miral/debian-control-files
updated
on 2016-08-08
- 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-
review:
Approve
lp:~alan-griffiths/miral/debian-control-files
updated
on 2016-08-10
- 266. By Alan Griffiths on 2016-08-10
-
Fix nits
- 267. By Alan Griffiths on 2016-08-10
-
merge lp:miral

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) ARCH_ENDIAN ?= $(shell dpkg-architecture -qDEB_HOST_ ARCH_ENDIAN) PACKAGE_ VERSION = $(shell dpkg-parsechangelog | grep Version | cut -d' ' -f 2)
+DEB_HOST_
+EXACT_
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.