Benda Xu:
> Hi Fellows,
> 
> This is a trivial patch for kernel-2.eclass to support Prefix.  Does it
> look good to be commited?
> 
> Thanks,
> Benda
> 
> Bug: 478436
> --- kernel-2.eclass   2016-02-17 22:46:25.235543840 +0900
> +++ kernel-2.eclass   2016-05-24 01:14:22.246809021 +0900
> @@ -539,8 +522,8 @@
>  #==============================================================
>  kernel_header_destdir() {
>       [[ ${CTARGET} == ${CHOST} ]] \
> -             && echo /usr/include \
> -             || echo /usr/${CTARGET}/usr/include
> +             && echo "${EPREFIX}"/usr/include \
> +             || echo "${EPREFIX}"/usr/${CTARGET}/usr/include
>  }
>  
>  cross_pre_c_headers() {
> @@ -667,7 +650,7 @@
>  
>               # autoconf.h isnt generated unless it already exists. plus, we 
> have
>               # no guarantee that any headers are installed on the system...
> -             [[ -f ${ROOT}/usr/include/linux/autoconf.h ]] \
> +             [[ -f ${EROOT}/usr/include/linux/autoconf.h ]] \
EROOT already ends with a trailing slash.

Also there are inconsistencies in the rest of the patch w.r.t. quoting
EROOT and EPREFIX.

>                       || touch include/linux/autoconf.h
>  
>               # if K_DEFCONFIG isn't set, force to "defconfig"
> @@ -741,15 +724,15 @@
>       # Do not use "linux/*" as that can cause problems with very long
>       # $S values where the cmdline to cp is too long
>       pushd "${S}" >/dev/null
> -     dodir ${ddir}/linux
> +     dodir ${ddir#${EPREFIX}}/linux
>       cp -pPR "${S}"/include/linux "${D}"/${ddir}/ || die
>       rm -rf "${D}"/${ddir}/linux/modules
>  
> -     dodir ${ddir}/asm
> -     cp -pPR "${S}"/include/asm/* "${D}"/${ddir}/asm
> +     dodir ${ddir#${EPREFIX}}/asm
> +     cp -pPR "${S}"/include/asm/* "${ED}"/${ddir}/asm
>  
>       if kernel_is 2 6 ; then
> -             dodir ${ddir}/asm-generic
> +             dodir ${ddir#${EPREFIX}}/asm-generic
>               cp -pPR "${S}"/include/asm-generic/* "${D}"/${ddir}/asm-generic
>       fi
>  
> @@ -784,7 +767,7 @@
>                       > "${S}"/patches.txt
>       fi
>  
> -     mv ${WORKDIR}/linux* "${D}"/usr/src
> +     mv ${WORKDIR}/linux* "${ED}"/usr/src
>  
>       if [[ -n "${UNIPATCH_DOCS}" ]] ; then
>               for i in ${UNIPATCH_DOCS}; do
> @@ -819,22 +799,22 @@
>  
>       # if we are to forcably symlink, delete it if it already exists first.
>       if [[ ${K_SYMLINK} > 0 ]]; then
> -             [[ -h ${ROOT}usr/src/linux ]] && rm ${ROOT}usr/src/linux
> +             [[ -h ${EROOT}usr/src/linux ]] && rm ${EROOT}usr/src/linux
>               MAKELINK=1
>       fi
>  
>       # if the link doesnt exist, lets create it
> -     [[ ! -h ${ROOT}usr/src/linux ]] && MAKELINK=1
> +     [[ ! -h ${EROOT}usr/src/linux ]] && MAKELINK=1
>  
>       if [[ ${MAKELINK} == 1 ]]; then
> -             cd "${ROOT}"usr/src
> +             cd "${EROOT}"usr/src
>               ln -sf linux-${KV_FULL} linux
>               cd ${OLDPWD}
>       fi
>  
>       # Don't forget to make directory for sysfs
> -     [[ ! -d ${ROOT}sys ]] && kernel_is 2 6 && mkdir ${ROOT}sys
> +     [[ ! -d ${EROOT}sys ]] && kernel_is 2 6 && mkdir ${EROOT}sys
>  
>       echo
>       elog "If you are upgrading from a previous kernel, you may be 
> interested"
>       elog "in the following document:"
> @@ -1353,11 +1310,11 @@
>       [[ ${ETYPE} == headers ]] && return 0
>  
>       # If there isn't anything left behind, then don't complain.
> -     [[ -e ${ROOT}usr/src/linux-${KV_FULL} ]] || return 0
> +     [[ -e ${EROOT}usr/src/linux-${KV_FULL} ]] || return 0
>       echo
>       ewarn "Note: Even though you have successfully unmerged "
>       ewarn "your kernel package, directories in kernel source location: "
> -     ewarn "${ROOT}usr/src/linux-${KV_FULL}"
> +     ewarn "${EROOT}usr/src/linux-${KV_FULL}"
>       ewarn "with modified files will remain behind. By design, package 
> managers"
>       ewarn "will not remove these modified files and the directories they 
> reside in."
>       echo
> 


Reply via email to