Gentoo Archives: gentoo-dev

From: Ulrich Mueller <ulm@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] RFC: Update to elisp-common.eclass
Date: Mon, 19 May 2014 11:17:15
Message-Id: 21369.59443.540769.48885@a1i15.kph.uni-mainz.de
In Reply to: Re: [gentoo-dev] RFC: Update to elisp-common.eclass by Alex Xu
1 >>>>> On Mon, 19 May 2014, Alex Xu wrote:
2
3 > On 18/05/14 02:13 PM, Ulrich Mueller wrote:
4 >> if [[ ${EBUILD_PHASE} = *rm && ! -e ${sitelisp}/site-gentoo.el ]]; then
5 >> ewarn "Refusing to create site-gentoo.el in ${EBUILD_PHASE} phase."
6 >> return 0
7 >> fi
8 >>
9 >> + [[ -d ${sitelisp} ]] \
10 >> + || die "elisp-site-regen: Directory ${sitelisp} does not exist"
11 >> +
12 >> + [[ -d ${T} ]] \
13 >> + || die "elisp-site-regen: Temporary directory ${T} does not exist"
14 >> +
15 > !qefs
16
17 > if ROOT, EPREFIX, or SITELISP has whitespace, this will throw a syntax
18 > error.
19
20 Where? In the snippet you quoted, I don't see where whitespace in
21 ${sitelisp} could cause any problems.
22
23 "Word splitting and filename expansion are not performed on the words
24 between the `[[' and `]]'" -- GNU Bash Reference Manual
25
26 Ulrich

Replies

Subject Author
Re: [gentoo-dev] RFC: Update to elisp-common.eclass Alex Xu <alex_y_xu@×××××.ca>