Merge ~aaronhoneycutt/kubuntu-packaging/+git/kalzium:kubuntu_unstable into ~kubuntu-packagers/kubuntu-packaging/+git/kalzium:kubuntu_unstable

Proposed by Aaron Honeycutt on 2016-11-27
Status: Merged
Merged at revision: 03a3b6296735c102ed745efa801a4fcabd9abe51
Proposed branch: ~aaronhoneycutt/kubuntu-packaging/+git/kalzium:kubuntu_unstable
Merge into: ~kubuntu-packagers/kubuntu-packaging/+git/kalzium:kubuntu_unstable
Diff against target: 37 lines (+14/-0)
2 files modified
debian/changelog (+1/-0)
debian/control (+13/-0)
Reviewer Review Type Date Requested Status
Simon Quigley 2016-11-27 Approve on 2016-11-27
Review via email: mp+311889@code.launchpad.net

Description of the Change

Added:
- libkf5i18n-dev
- libkf5kdelibs4support-dev
- libkf5khtml-dev
- libkf5newstuff-dev
- libkf5parts-dev
- libkf5plotting-dev
- libkf5solid-dev
- libkf5unitconversion-dev
- libkf5widgetsaddons-dev
- libkf5archive-dev
- libkf5config-dev
- libkf5coreaddons-dev
- kdoctools-dev

To post a comment you must log in.
Simon Quigley (tsimonq2) wrote :

At this point with the changelog, I would use a more general entry and just one entry total. It makes the changelog less cluttered. If you can find out why we need these all of a sudden, put that there as well. :)

Otherwise, lgtm.

review: Needs Fixing
Aaron Honeycutt (aaronhoneycutt) wrote :

I'm not sure why it needs all of them now, the logs are just saying that it needs a ton of cmake files and those packages provide them.

Simon Quigley (tsimonq2) wrote :

Alright, then say something like "Upstream has new build deps that we need"

Simon Quigley (tsimonq2) wrote :

Ok, fair enough, I'll merge. Thanks Aaron! :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index c920bb2..9294c01 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -10,6 +10,7 @@ kalzium (4:16.04.3-0ubuntu2) UNRELEASED; urgency=medium
6 * Added qtscript5-dev as build dep
7 * Added libqt5opengl5-dev as build dep
8 * Added libqt5svg5-dev as build dep
9+ * Added lots of libkf5 packages and kdoctools-dev
10
11 -- José Manuel Santamaría Lema <panfaust@gmail.com> Sat, 29 Oct 2016 17:16:39 +0200
12
13diff --git a/debian/control b/debian/control
14index 05b20bb..efeed35 100644
15--- a/debian/control
16+++ b/debian/control
17@@ -9,7 +9,20 @@ Build-Depends: chemical-mime-data,
18 cmake,
19 debhelper (>= 9),
20 extra-cmake-modules,
21+ kdoctools-dev,
22 libavogadro-dev [!armhf],
23+ libkf5archive-dev,
24+ libkf5config-dev,
25+ libkf5coreaddons-dev,
26+ libkf5i18n-dev,
27+ libkf5kdelibs4support-dev,
28+ libkf5khtml-dev,
29+ libkf5newstuff-dev,
30+ libkf5parts-dev,
31+ libkf5plotting-dev,
32+ libkf5solid-dev,
33+ libkf5unitconversion-dev,
34+ libkf5widgetsaddons-dev,
35 libqt5opengl5-dev,
36 libqt5svg5-dev,
37 pkg-config,

Subscribers

People subscribed via source and target branches