https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110153

Nathaniel Shead <nshead at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |16.0
           Assignee|unassigned at gcc dot gnu.org      |nicolas.werner at 
hotmail dot de
         Resolution|---                         |FIXED
                 CC|                            |nshead at gcc dot gnu.org
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #2 from Nathaniel Shead <nshead at gcc dot gnu.org> ---
This looks to have been fixed by r16-3137-g041248ca43beb1:

commit 041248ca43beb158afae8350117378c98e9191ad
Author: Nicolas Werner <nicolas.wer...@hotmail.de>
Date:   Sun Aug 3 16:38:08 2025 +0200

    c++: Quoting in -fmodules-mapper

    Users might be using a space in their build directory path. To allow
    specifying such a root for the module mapper started by GCC, we need the
    command to allow quotes. Previously quoting a path passed to the module
    mapper was not possible, so replace the custom argv parsing with
    the argv parsing logic from libiberty, that supports fairly standard
    shell quoting using single and double quotes.

    The primary purpose of this patch is to allow passing paths with spaces
    to the --root parameter of the module mapper.

    No test is included as spaces in build directories are tricky cross
    platform. The patch was tested manually on my system.

    gcc/cp/ChangeLog:

            * mapper-client.cc (spawn_mapper_program): change argv parsing

So closing as fixed, thanks!

Reply via email to