Bug 290254 – sys-apps/portage-2.2.00.14701: wrong path for var/lib/portage/config

• Product: Gentoo/Alt
• Status: RESOLVED FIXED

[Description Nicolas Olivier 2009-10-23 16:05:46 UTC]

var/lib/portage/config is generated under ${EPREFIX}/${EPREFIX}/var/lib/portage/config instead of ${EPREFIX}/var/lib/portage/config

Reproducible: Always

Steps to Reproduce:
1. emerge portage (it updates var/lib/portage/config for etc/dispatch-conf & co)

Actual Results:  
/${EPREFIX}/${EPREFIX}/var/lib/portage/config is updated
/${EPREFIX}/var/lib/portage/config is inexistent or not updated

Expected Results:  
/${EPREFIX}/var/lib/portage/config should be updated

[Comment 1 Fabian Groffen 2009-10-24 09:27:57 UTC]

thanks for spotting that, seems critical to me

[Comment 2 Fabian Groffen 2009-10-24 11:00:03 UTC]

this patch should do it

Index: pym/portage/dbapi/vartree.py
===================================================================
--- pym/portage/dbapi/vartree.py        (revision 14698)
+++ pym/portage/dbapi/vartree.py        (working copy)
@@ -3075,7 +3075,7 @@
                        os.path.sep
                dest_root_len = len(dest_root) - 1
 
-               conf_mem_file = os.path.join(dest_root, EPREFIX_LSTRIP, CONFIG_MEMORY_FILE)
+               conf_mem_file = os.path.join(dest_root, CONFIG_MEMORY_FILE)
                cfgfiledict = grabdict(conf_mem_file)
                stale_confmem = []
 

[Comment 3 Nicolas Olivier 2009-10-24 11:59:00 UTC]

it seems the problem is still present for me

[Comment 4 Fabian Groffen 2009-10-24 12:01:13 UTC]

after applying the patch?

[Comment 5 Nicolas Olivier 2009-10-24 12:09:14 UTC]

yes, and checked again to be sure

[Comment 6 Fabian Groffen 2009-10-24 12:15:56 UTC]

and if you apply also this patch on top?

Index: pym/portage/dbapi/vartree.py
===================================================================
--- pym/portage/dbapi/vartree.py        (revision 14720)
+++ pym/portage/dbapi/vartree.py        (working copy)
@@ -4462,7 +4462,7 @@
                self.updateprotect()
 
                #if we have a file containing previously-merged config file md5sums, grab it.
-               conf_mem_file = os.path.join(destroot, EPREFIX_LSTRIP, CONFIG_MEMORY_FILE)
+               conf_mem_file = os.path.join(destroot, CONFIG_MEMORY_FILE)
                cfgfiledict = grabdict(conf_mem_file)
                cfgfiledict_orig = cfgfiledict.copy()
                if "NOCONFMEM" in self.settings:

[Comment 7 Nicolas Olivier 2009-10-24 12:28:28 UTC]

it works, thanks :)

[Comment 8 Fabian Groffen 2009-10-24 13:36:12 UTC]

thanks for testing, this is now in 2.2.00.14724