Correctly assign configuration to origin files with pip config

Author: chrysle

news/12099.bugfix.rst

@@ -0,0 +1 @@
+This change will fix incorrect output of the ``pip config debug`` command.

src/pip/_internal/cli/parser.py

@@ -185,23 +185,38 @@ def _get_ordered_configuration_items(
         section_items: Dict[str, List[Tuple[str, Any]]] = {
             name: [] for name in override_order
         }
-        for section_key, val in self.config.items():
-            # ignore empty values
-            if not val:
-                logger.debug(
-                    "Ignoring configuration key '%s' as it's value is empty.",
-                    section_key,
-                )
-                continue
 
-            section, key = section_key.split(".", 1)
-            if section in override_order:
-                section_items[section].append((key, val))
+        for key, value in self.config.items():
+            if not isinstance(value, dict):
+                # ignore empty values
+                if not value:
+                    logger.debug(
+                        "Ignoring configuration key '%s' as it's value is empty.",
+                        key,
+                    )
+                    continue
 
-        # Yield each group in their override order
-        for section in override_order:
-            for key, val in section_items[section]:
-                yield key, val
+                section, key = key.split(".", 1)
+                if section in override_order:
+                    section_items[section].append((key, value))
+            else:
+                for section_key, val in value.items():
+                    # ignore empty values
+                    if not val:
+                        logger.debug(
+                            "Ignoring configuration key '%s' as it's value is empty.",
+                            section_key,
+                        )
+                        continue
+
+                    section, key = section_key.split(".", 1)
+                    if section in override_order:
+                        section_items[section].append((key, val))
+
+            # Yield each group in their override order
+            for section in override_order:
+                for key, val in section_items[section]:
+                    yield key, val
 
     def _update_defaults(self, defaults: Dict[str, Any]) -> Dict[str, Any]:
         """Updates the given defaults with values from the config files and

src/pip/_internal/commands/configuration.py

@@ -172,7 +172,11 @@ def list_values(self, options: Values, args: List[str]) -> None:
         self._get_n_args(args, "list", n=0)
 
         for key, value in sorted(self.configuration.items()):
-            write_output("%s=%r", key, value)
+            if isinstance(value, dict):
+                for key, value in sorted(value.items()):
+                    write_output("%s=%r", key, value)
+            else:
+                write_output("%s=%r", key, value)
 
     def get_name(self, options: Values, args: List[str]) -> None:
         key = self._get_n_args(args, "get [name]", n=1)
@@ -206,13 +210,15 @@ def list_config_values(self, options: Values, args: List[str]) -> None:
                     file_exists = os.path.exists(fname)
                     write_output("%s, exists: %r", fname, file_exists)
                     if file_exists:
-                        self.print_config_file_values(variant)
+                        self.print_config_file_values(variant, fname)
 
-    def print_config_file_values(self, variant: Kind) -> None:
+    def print_config_file_values(self, variant: Kind, fname: str) -> None:
         """Get key-value pairs from the file of a variant"""
         for name, value in self.configuration.get_values_in_config(variant).items():
             with indent_log():
-                write_output("%s: %s", name, value)
+                if name == fname:
+                    for confname, confvalue in value.items():
+                        write_output("%s: %s", confname, confvalue)
 
     def print_env_var_values(self) -> None:
         """Get key-values pairs present as environment variables"""

src/pip/_internal/configuration.py

@@ -114,7 +114,7 @@ def __init__(self, isolated: bool, load_only: Optional[Kind] = None) -> None:
         self._parsers: Dict[Kind, List[Tuple[str, RawConfigParser]]] = {
             variant: [] for variant in OVERRIDE_ORDER
         }
-        self._config: Dict[Kind, Dict[str, Any]] = {
+        self._config: Dict[Kind, Dict[str, Dict[str, Any]]] = {
             variant: {} for variant in OVERRIDE_ORDER
         }
         self._modified_parsers: List[Tuple[str, RawConfigParser]] = []
@@ -145,7 +145,14 @@ def get_value(self, key: str) -> Any:
         orig_key = key
         key = _normalize_name(key)
         try:
-            return self._dictionary[key]
+            clean_config = {}
+            for k, value in self._dictionary.items():
+                if isinstance(value, dict):
+                    for k, v in value.items():
+                        clean_config[k] = v
+                else:
+                    clean_config[k] = value
+            return clean_config[key]
         except KeyError:
             # disassembling triggers a more useful error message than simply
             # "No such key" in the case that the key isn't in the form command.option
@@ -168,7 +175,11 @@ def set_value(self, key: str, value: Any) -> None:
                 parser.add_section(section)
             parser.set(section, name, value)
 
-        self._config[self.load_only][key] = value
+        exists = self._config[self.load_only].get(fname)
+        if not exists:
+            self._config[self.load_only][fname] = {}
+
+        self._config[self.load_only][fname][key] = value
         self._mark_as_modified(fname, parser)
 
     def unset_value(self, key: str) -> None:
@@ -178,11 +189,14 @@ def unset_value(self, key: str) -> None:
         self._ensure_have_load_only()
 
         assert self.load_only
-        if key not in self._config[self.load_only]:
-            raise ConfigurationError(f"No such key - {orig_key}")
-
         fname, parser = self._get_parser_to_modify()
 
+        if (
+            key not in self._config[self.load_only][fname]
+            and key not in self._config[self.load_only]
+        ):
+            raise ConfigurationError(f"No such key - {orig_key}")
+
         if parser is not None:
             section, name = _disassemble_key(key)
             if not (
@@ -197,8 +211,10 @@ def unset_value(self, key: str) -> None:
             if not parser.items(section):
                 parser.remove_section(section)
             self._mark_as_modified(fname, parser)
-
-        del self._config[self.load_only][key]
+        try:
+            del self._config[self.load_only][fname][key]
+        except KeyError:
+            del self._config[self.load_only][key]
 
     def save(self) -> None:
         """Save the current in-memory state."""
@@ -270,7 +286,10 @@ def _load_file(self, variant: Kind, fname: str) -> RawConfigParser:
 
         for section in parser.sections():
             items = parser.items(section)
-            self._config[variant].update(self._normalized_keys(section, items))
+            exists = self._config[variant].get(fname)
+            if not exists:
+                self._config[variant][fname] = {}
+            self._config[variant][fname].update(self._normalized_keys(section, items))
 
         return parser