[llvm-bugs] [Bug 27743] New: Spaces in paths from llvm-config should be quoted

[via llvm-bugs]

https://llvm.org/bugs/show_bug.cgi?id=27743

            Bug ID: 27743
           Summary: Spaces in paths from llvm-config should be quoted
           Product: tools
           Version: trunk
          Hardware: PC
                OS: Windows NT
            Status: NEW
          Severity: normal
          Priority: P
         Component: llvm-config
          Assignee: unassignedbugs at nondot.org
          Reporter: taricorp at gmail.com
                CC: llvm-bugs at lists.llvm.org
    Classification: Unclassified

When LLVM is installed in a path containing spaces, paths output from
llvm-config have no way to indicate breaks between parameters and spaces within
paths. For example, --cflags on Windows for MSVC might include "-IC:\Program
Files\LLVM\include" which parses as "-IC:\Program" "Files\LLVM\include".

This could be fixed with escaping or quoting by llvm-config, perhaps as simple
as quoting each individual element it emits. I believe quoting paths is
portable to all major shells, though this would involve trading one error case
(space in path) for another (quote in path) unless shell escaping logic were
added.

Adding quotes to the output might break some existing tools, but given the
usual use case is feeding the output to a shell it may not be worth worrying
about.

Source issue:
https://bitbucket.org/tari/llvm-sys.rs/issues/9/include-path-for-llvm-headers-on-windows

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20160514/0efd76cc/attachment.html>