[LLVMbugs] [Bug 7809] New: llvm/Config/config.h pollutes user namespace

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=7809

           Summary: llvm/Config/config.h pollutes user namespace
           Product: libraries
           Version: trunk
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P
         Component: Target Description Classes
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: jdenny at etinternational.com
                CC: llvmbugs at cs.uiuc.edu


The LLVM headers llvm-c/Target.h and llvm/Target/TargetSelect.h include 
the header llvm/Config/config.h, which defines common macros, such as 
PACKAGE_BUGREPORT.  If a project that uses LLVM includes any of these 
headers, the project's own macros of the same names can be redefined.

Most advice I have seen says that installed headers should never define 
such common macros.  Thus, an internal config.h should never be installed:

  http://stackoverflow.com/questions/1810216/autoconf-where-does-config-h-go
  http://lists.gnu.org/archive/html/autoconf/2004-09/msg00151.html

Would you please consider replacing the current llvm/Config/config.h with 
a trimmed down version that contains only the definitions that must be 
present in LLVM's installed headers?  PACKAGE_* macros certainly do not 
need to be there.

-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.