Bug 514900 – dev-libs/libxslt-1.1.28-r2: double prefix files installed (the python module)

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Michael Haubenwallner (RETIRED) 2014-06-24 11:15:04 UTC]

Not sure if this has changed recently, but python_get_sitedir() does contain EPREFIX already. Plus one comment for Prefix.

--- libxslt-1.1.28-r2.ebuild    18 Jun 2014 19:17:22 -0000      1.3
+++ libxslt-1.1.28-r2.ebuild    24 Jun 2014 11:12:11 -0000
@@ -52,6 +52,8 @@
        # and it is propably otherwise too if upstream generated with new
        # autoconf
 #      epunt_cxx
+       # But Prefix always needs elibtoolize if not eautoreconf'd.
+#      elibtoolize
 }
 
 multilib_src_configure() {
@@ -97,8 +99,8 @@
                PYTHON="${PYTHON}" \
                PYTHON_INCLUDES="${EPREFIX}/usr/include/${EPYTHON}" \
                PYTHON_LIBS="$(python-config --ldflags)" \
-               PYTHON_SITE_PACKAGES="${EPREFIX}$(python_get_sitedir)" \
-               pythondir="${EPREFIX}$(python_get_sitedir)" \
+               PYTHON_SITE_PACKAGES="$(python_get_sitedir)" \
+               pythondir="$(python_get_sitedir)" \
                PYTHON_VERSION=${EPYTHON/python} "$@"
        popd > /dev/null
 }


Thanks!

[Comment 1 Pacho Ramos 2014-06-24 12:35:10 UTC]

Go ahead please

[Comment 2 Alexandre Rostovtsev (RETIRED) 2014-06-24 12:36:33 UTC]

Please wait a bit, I would prefer to rewrite the old-fashioned python build setup in libxslt using libxml2 as a model.

[Comment 3 Alexandre Rostovtsev (RETIRED) 2014-06-24 13:14:37 UTC]

+*libxslt-1.1.28-r3 (24 Jun 2014)
+
+  24 Jun 2014; Alexandre Rostovtsev <[e-mail]>
+  +libxslt-1.1.28-r3.ebuild:
+  Modernize python bindings build setup, fixing prefix issues (bug #514900,
+  thanks to Michael Haubenwallner).

[Comment 4 Pacho Ramos 2014-06-24 13:24:37 UTC]

Thanks, I am reading the diff to learn a bit more about this python stuff :O

[Comment 5 Michael Haubenwallner (RETIRED) 2014-06-24 13:31:40 UTC]

Works. Keyworded. Thanks!