commit:     d5beda4d171865e8f8c4e6e23d7cea110bff40fb
Author:     Ulrich Müller <ulm <AT> gentoo <DOT> org>
AuthorDate: Sat Dec 19 14:15:05 2015 +0000
Commit:     Ulrich Müller <ulm <AT> gentoo <DOT> org>
CommitDate: Mon Dec 21 07:52:26 2015 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d5beda4d

readme.gentoo-r1.eclass: Do not inherit eutils.

This was only needed in readme.gentoo_create_doc() for a single call
of eshopts_{push,pop}. Replace it by saving the set of options in a
variable. Die if writing the temp file in readme.gentoo_create_doc()
fails.

 eclass/readme.gentoo-r1.eclass | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/eclass/readme.gentoo-r1.eclass b/eclass/readme.gentoo-r1.eclass
index c076650..07320c0 100644
--- a/eclass/readme.gentoo-r1.eclass
+++ b/eclass/readme.gentoo-r1.eclass
@@ -21,8 +21,6 @@
 if [[ -z ${_README_GENTOO_ECLASS} ]]; then
 _README_GENTOO_ECLASS=1
 
-inherit eutils
-
 case "${EAPI:-0}" in
        0|1|2|3)
                die "Unsupported EAPI=${EAPI:-0} (too old) for ${ECLASS}"
@@ -61,15 +59,16 @@ readme.gentoo_create_doc() {
        debug-print-function ${FUNCNAME} "${@}"
 
        if [[ -n "${DOC_CONTENTS}" ]]; then
-               eshopts_push
-               set -f
                if [[ -n "${DISABLE_AUTOFORMATTING}" ]]; then
-                       echo "${DOC_CONTENTS}" > "${T}"/README.gentoo
+                       echo "${DOC_CONTENTS}" > "${T}"/README.gentoo || die
                else
+                       local saved_flags=$-
+                       set -f                          # disable filename 
expansion in echo arguments
                        echo -e ${DOC_CONTENTS} | fold -s -w 70 \
                                | sed 's/[[:space:]]*$//' > "${T}"/README.gentoo
+                       assert
+                       set +f -${saved_flags}
                fi
-               eshopts_pop
        elif [[ -f "${FILESDIR}/README.gentoo-${SLOT%/*}" ]]; then
                cp "${FILESDIR}/README.gentoo-${SLOT%/*}" "${T}"/README.gentoo 
|| die
        elif [[ -f "${FILESDIR}/README.gentoo${README_GENTOO_SUFFIX}" ]]; then

Reply via email to