First Last Prev Next    No search results available
Details
: [install] Some header files not installed when src root !...
Bug#: 220
: Build scripts
: Makefiles
Status: VERIFIED
Resolution: FIXED
: All
: All
: 1.0
: P2
: normal
: 1.2

:
: build-problem
:
:
  Show dependency tree - Show dependency graph
People
Reporter: Reid Spencer <rspencer@reidspencer.com>
Assigned To: Brian R. Gaeke <gaeke+bugs@uiuc.edu>
:

Attachments
Output of "make install" (21.48 KB, text/plain)
2004-02-02 17:22, Reid Spencer
Details


Note

You need to log in before you can comment on or make changes to this bug.

Related actions


Description:   Opened: 2004-02-02 17:17
I tried the new "make install" function from the root of the source tree. While
the library files installed correctly, the only header to install was
llvm/Config/config.h.

I suspect this is a result of my use of separate source and object directories.

My configuration:

Linux: 2.4.20-28.9smp (RedHat 9.0 with updates)
gcc: 3.3.1
autoconf: 2.57
gmake: 3.79.1
configure command:

../llvm/configure \
    --prefix=/proj/install \
    --with-llvmgccdir=/proj/work/llvm/cfrontend/install

output from "make install" will be attached.
------- Comment #1 From Reid Spencer 2004-02-02 17:22:05 -------
Created an attachment (id=71) [details]
Output of "make install"

Just shows that the only header file installed was Config/config.h (early in
the attached file).
------- Comment #2 From Brian R. Gaeke 2004-02-08 20:31:51 -------
install was not implemented in 1.1.

Fixing this might just involve duplicating the "pax" line in llvm/Makefile to do
the same thing for BUILD_SRC_ROOT/include as it does for include.
------- Comment #3 From Brian R. Gaeke 2004-02-09 19:00:36 -------
I have duplicated the incorrect behavior, btw.
------- Comment #4 From Brian R. Gaeke 2004-02-09 19:13:49 -------
Can you give it another shot with the current llvm/Makefile? I just checked in a
fix. If you do not agree that it fixes the bug, please re-open it.

Patch is here:

http://mail.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20040209/011409.html

Thanks for letting us know about this.
------- Comment #5 From Reid Spencer 2004-02-10 01:56:09 -------
Confirmed. The header files are now installed properly. 

One thing I noted is that the path I need to use to #include the llvm library
headers is:

#include <llvm/llvm/header.h>

This seems a little awkward with the duplicate llvm but its not a big deal.

Marking this as verified.

------- Comment #6 From Chris Lattner 2004-02-10 09:54:51 -------
> One thing I noted is that the path I need to use to #include the llvm library
> headers is:
> #include <llvm/llvm/header.h>

You're right that that is wrong, but it can only be fixed when Bug 183 is...

-Chris

First Last Prev Next    No search results available