Merge ~athos-ribeiro/ubuntu/+source/isc-dhcp:keama into ubuntu/+source/isc-dhcp:ubuntu/devel

Proposed by Athos Ribeiro
Status: Merged
Merged at revision: 9f7d845b6166dbc2263f949a631b31f1863b94cd
Proposed branch: ~athos-ribeiro/ubuntu/+source/isc-dhcp:keama
Merge into: ubuntu/+source/isc-dhcp:ubuntu/devel
Diff against target: 84 lines (+32/-0)
6 files modified
debian/changelog (+7/-0)
debian/control (+12/-0)
debian/isc-dhcp-keama.docs (+1/-0)
debian/isc-dhcp-keama.install (+1/-0)
debian/isc-dhcp-keama.manpages (+1/-0)
debian/rules (+10/-0)
Reviewer Review Type Date Requested Status
Sergio Durigan Junior (community) Approve
Canonical Server Pending
git-ubuntu import Pending
Review via email: mp+443236@code.launchpad.net

Description of the change

This introduces a new binary package in the isc-dhcp package, as proposed in LP: #2020086. The new binary (keama) is a standalone binary to assist users during their config migration to kea.

PPA: https://launchpad.net/~athos-ribeiro/+archive/ubuntu/keama/+packages

To post a comment you must log in.
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thanks for the MP, Athos.

It looks great. The package builds fine, and I verified that the new binary is correctly installed under /usr/sbin. I was going to ask if it's really necessary to install it under sbin, but I guess it doesn't hurt to keep it this way.

I tried to enable testing by executing "dh_auto_test -- -C keama", but it didn't work because apparently there's no Makefile target for it. I wonder if upstream intends to implement this.

I'm leaving some minor suggestions below, but otherwise I'm approving the MP. Thanks.

review: Approve
Revision history for this message
Athos Ribeiro (athos-ribeiro) wrote :

Thanks, Sergio!

Uploaded!

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 ede5c7f..c591c7c 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+isc-dhcp (4.4.3-P1-2ubuntu2) mantic; urgency=medium
7+
8+ * Ship keama, the KEA Migration Assistant in a new isc-dhcp-keama binary
9+ package. (LP: #2020086)
10+
11+ -- Athos Ribeiro <athos.ribeiro@canonical.com> Thu, 18 May 2023 21:30:57 -0300
12+
13 isc-dhcp (4.4.3-P1-2ubuntu1) mantic; urgency=low
14
15 * Merge from Debian unstable, remaining changes:
16diff --git a/debian/control b/debian/control
17index c1498e5..a664acd 100644
18--- a/debian/control
19+++ b/debian/control
20@@ -195,3 +195,15 @@ Description: ISC DHCP relay daemon
21 .
22 More information can be found in the ISC official announcement:
23 https://www.isc.org/blogs/dhcp-client-relay-eom/
24+
25+Package: isc-dhcp-keama
26+Priority: optional
27+Architecture: any
28+Depends:
29+ ${misc:Depends},
30+ ${shlibs:Depends},
31+Description: KEA Migration Assistant
32+ This is an ISC DHCP server to ISC KEA Migration Assistant.
33+ .
34+ The KEA Migration Assistant (aka keama) is an experimental tool
35+ which helps to translate ISC DHCP configurations to Kea.
36diff --git a/debian/isc-dhcp-keama.docs b/debian/isc-dhcp-keama.docs
37new file mode 100644
38index 0000000..8e9cf54
39--- /dev/null
40+++ b/debian/isc-dhcp-keama.docs
41@@ -0,0 +1 @@
42+keama/README.md
43diff --git a/debian/isc-dhcp-keama.install b/debian/isc-dhcp-keama.install
44new file mode 100644
45index 0000000..8e8233c
46--- /dev/null
47+++ b/debian/isc-dhcp-keama.install
48@@ -0,0 +1 @@
49+usr/sbin/keama
50diff --git a/debian/isc-dhcp-keama.manpages b/debian/isc-dhcp-keama.manpages
51new file mode 100644
52index 0000000..3d42c55
53--- /dev/null
54+++ b/debian/isc-dhcp-keama.manpages
55@@ -0,0 +1 @@
56+debian/tmp/usr/share/man/man8/keama.8
57diff --git a/debian/rules b/debian/rules
58index f6141e8..dee414a 100755
59--- a/debian/rules
60+++ b/debian/rules
61@@ -52,6 +52,9 @@ endif
62 %:
63 dh $@ --parallel --with autoreconf
64
65+override_dh_clean:
66+ dh_clean $@ -Xkeama/tests/samples/*
67+
68 # Ensure that the bind config.guess/config.sub are unpacked first,
69 # otherwise they won't be updated and isc-dhcp will FTBFS on new arches
70 # Thanks to Paul Wise
71@@ -81,6 +84,13 @@ override_dh_auto_build:
72 test -f Makefile && $(MAKE) distclean || true
73 ./configure CFLAGS="$(CFLAGS) -DNSUPDATE" $(CONFFLAGS) LIBS="$(LIBS) -latomic"
74 $(MAKE)
75+ # keama build
76+ $(MAKE) -C keama
77+
78+override_dh_auto_install:
79+ dh_auto_install
80+ # install keama
81+ dh_auto_install -- -C keama
82
83 override_dh_install:
84 # rename some upstream files

Subscribers

People subscribed via source and target branches