Gentoo Archives: gentoo-dev

From: Alexis Ballier <aballier@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] [PATCH 2/3] Introduce cmake-multilib wrapper for cmake-utils.
Date: Wed, 06 Feb 2013 01:33:14
Message-Id: 20130205223259.68e4c0c1@gentoo.org
In Reply to: [gentoo-dev] [PATCH 2/3] Introduce cmake-multilib wrapper for cmake-utils. by "Michał Górny"
1 On Tue, 5 Feb 2013 21:19:23 +0100
2 Michał Górny <mgorny@g.o> wrote:
3
4 > +cmake-multilib_src_install() {
5 > + cmake-multilib_secure_install() {
6 > + cmake-utils_src_install
7 > +
8 > + # Make sure all headers are the same for each ABI.
9 > + cmake-multilib_cksum() {
10 > + find "${ED}"usr/include -type f \
11 > + -exec cksum {} + | sort -k2
12 > + }
13 > +
14 > + local cksum=$(cmake-multilib_cksum)
15 > + local cksum_file=${T}/.cmake-multilib_cksum
16 > +
17 > + if [[ -f ${cksum_file} ]]; then
18 > + local cksum_prev=$(< "${cksum_file}")
19 > +
20 > + if [[ ${cksum} != ${cksum_prev} ]]; then
21 > + echo "${cksum}" > "${cksum_file}.new"
22 > +
23 > + eerror "Header files have changed
24 > between ABIs." +
25 > + if type -p diff &>/dev/null; then
26 > + eerror "$(diff -du
27 > "${cksum_file}" "${cksum_file}.new")"
28 > + else
29 > + eerror "Old checksums in:
30 > ${cksum_file}"
31 > + eerror "New checksums in:
32 > ${cksum_file}.new"
33 > + fi
34 > +
35 > + die "Header checksum mismatch,
36 > aborting."
37 > + fi
38 > + else
39 > + echo "${cksum}" > "${cksum_file}"
40 > + fi
41 > + }
42
43 > The eclass uses the same header consistency check as
44 > autotools-multilib
45 > (therefore, I move the function to multilib-build).
46
47
48 You probably forgot to use it here

Replies