Bug 581914 – app-i18n/sunpinyin: pod2man error (no name given for document) and double prefix when intalled on gentoo::prefix

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description gtkgfxp 2016-05-03 06:45:10 UTC]

Created attachment 433012 [details]
fix:sunpinyin:pod2man error (no name given for document) and double prefix when intalled on gentoo::prefix

1 bug fix for https://bugs.gentoo.org/show_bug.cgi?id=553796

add this line:
sed -i -e 's/pod2man < $SOURCE > $TARGET/pod2man $SOURCE $TARGET/' 
according to:
https://bugs.gentoo.org/attachment.cgi?id=431980&action=diff
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=790287;msg=7

2 bug fix for double prefix when intalled on gentoo::prefix

escons --install-sandbox="${ED}" install modified to:
escons --install-sandbox="${EPREFIX}/${D}" install
according to:
Usually, all but one occurrences of ${D} in an ebuild (or eclass) have to be replaced by ${ED} to work properly in the prefix. Remember that when using make DESTDIR="${D}" install the ${D} should in general not be changed, as configure was called with --prefix="${EPREFIX}"/usr. 
ref:
https://wiki.gentoo.org/wiki/Project:Prefix/Technical_Documentation

notice:
${EPREFIX}/${D} is diff from ${ED}( It contains the value of ${D%/}${EPREFIX}/)

[Comment 1 gtkgfxp 2016-05-03 10:14:31 UTC]

fix:sunpinyin:pod2man error (no name given for document) and double prefix when intalled on gentoo::prefix

1 bug fix for https://bugs.gentoo.org/show_bug.cgi?id=553796
---
src_prepare() {
	epatch_user
	sed -i -e 's/pod2man < $SOURCE > $TARGET/pod2man $SOURCE $TARGET/' man/SConscript || die
}
---
add this line:
sed -i -e 's/pod2man < $SOURCE > $TARGET/pod2man $SOURCE $TARGET/' 
according to:
https://bugs.gentoo.org/attachment.cgi?id=431980&action=diff
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=790287;msg=7

2 bug fix for double prefix when intalled on gentoo::prefix

escons --install-sandbox="${ED}" install modified to:
escons --install-sandbox="${D}" install
according to:
Usually, all but one occurrences of ${D} in an ebuild (or eclass) have to be replaced by ${ED} to work properly in the prefix. Remember that when using make DESTDIR="${D}" install the ${D} should in general not be changed, as configure was called with --prefix="${EPREFIX}"/usr. 
ref:
https://wiki.gentoo.org/wiki/Project:Prefix/Technical_Documentation

[Comment 2 gtkgfxp 2016-05-03 10:32:10 UTC]

Created attachment 433074 [details]
ix:sunpinyin:pod2man error (no name given for document) and double prefix when intalled on gentoo::prefix

[Comment 3 Benda Xu 2016-05-10 05:48:12 UTC]

Fixed.  Thanks a lot for your bug report.

--- a/app-i18n/sunpinyin/sunpinyin-2.0.4_pre20130108.ebuild
+++ b/app-i18n/sunpinyin/sunpinyin-2.0.4_pre20130108.ebuild
@@ -1,4 +1,4 @@
-# Copyright 1999-2015 Gentoo Foundation
+# Copyright 1999-2016 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
 # $Id$

@@ -24,6 +24,7 @@ DEPEND="${RDEPEND}
 PDEPEND="app-i18n/sunpinyin-data"

 src_prepare() {
+       epatch "${FILESDIR}"/${PN}-2.0.4-pod2man.patch
        epatch_user
 }

@@ -40,7 +41,7 @@ src_compile() {
 }

 src_install() {
-       escons --install-sandbox="${ED}" install
+       escons --install-sandbox="${D}" install
        rm -rf "${D}"/usr/share/doc/${PN} || die
        dodoc doc/{README,SLM-inst.mk,SLM-train.mk}
 }