Allan McRae wrote:
> Add a check that the package does not contain references to the
> folder it was built in.
> 
> Fixes FS#14751
> 
> Signed-off-by: Allan McRae <[email protected]>
> ---
>  scripts/makepkg.sh.in |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)
> 
> diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in
> index 820d81f..a5b8f87 100644
> --- a/scripts/makepkg.sh.in
> +++ b/scripts/makepkg.sh.in
> @@ -971,6 +971,12 @@ check_package() {
>                       warning "$(gettext "Invalid backup entry : %s")" "$file"
>               fi
>       done
> +
> +     # check for references to the build directory
> +     grep -R "${srcdir}" "${pkgdir}" &>/dev/null
> +     if [ $? -eq 0 ]; then
> +             warning "$(gettext "Package contains reference to %s")" 
> "\$srcdir"
> +     fi
>  }
>  
>  create_package() {


I wonder why you do not use

if grep -R "${srcdir}" "${pkgdir}" &>/dev/null; then

Is there a special cause for that or is it only for cosmetic reasons? Thanks.

Reply via email to