Re: [gentoo-dev] [PATCH 2/3] Introduce cmake-multilib wrapper for cmake-utils.

[Michał Górny]

On Tue, 5 Feb 2013 22:32:59 -0300
Alexis Ballier <aballier@g.o> wrote:

> On Tue,  5 Feb 2013 21:19:23 +0100
> Michał Górny <mgorny@g.o> wrote:
>
> > +cmake-multilib_src_install() {
> > +	cmake-multilib_secure_install() {
> > +		cmake-utils_src_install
> > +
> > +		# Make sure all headers are the same for each ABI.
> > +		cmake-multilib_cksum() {
> > +			find "${ED}"usr/include -type f \
> > +				-exec cksum {} + | sort -k2
> > +		}
> > +
> > +		local cksum=$(cmake-multilib_cksum)
> > +		local cksum_file=${T}/.cmake-multilib_cksum
> > +
> > +		if [[ -f ${cksum_file} ]]; then
> > +			local cksum_prev=$(< "${cksum_file}")
> > +
> > +			if [[ ${cksum} != ${cksum_prev} ]]; then
> > +				echo "${cksum}" > "${cksum_file}.new"
> > +
> > +				eerror "Header files have changed
> > between ABIs." +
> > +				if type -p diff &>/dev/null; then
> > +					eerror "$(diff -du
> > "${cksum_file}" "${cksum_file}.new")"
> > +				else
> > +					eerror "Old checksums in:
> > ${cksum_file}"
> > +					eerror "New checksums in:
> > ${cksum_file}.new"
> > +				fi
> > +
> > +				die "Header checksum mismatch,
> > aborting."
> > +			fi
> > +		else
> > +			echo "${cksum}" > "${cksum_file}"
> > +		fi
> > +	}
>
> > The eclass uses the same header consistency check as
> > autotools-multilib
> > (therefore, I move the function to multilib-build).
>
>
> You probably forgot to use it here

lolcorrect.

-- 
Best regards,
Michał Górny