[LLVMdev] Removing $(LLVM_SRC_ROOT)/autoconf dependensies in Stacker, llvm-java [PATCH]

[Vladimir Merzliakov]

Hi!

In current LLVM CVS:

llvm/projects/Stacker/Makefile.common.in
llvm/projects/sample/Makefile.common.in
llvm-java/llvm-java/Makefile.common.in

have line:
include $(LLVM_OBJ_ROOT)/Makefile.common

that have line:
include $(LLVM_OBJ_ROOT)/Makefile.config

(also $(LLVM_OBJ_ROOT)/Makefile.config used in llvm-test/Makefile.config.in)

and

llvm/Makefile.config.in have lines:

INSTALL_SH := $(LLVM_SRC_ROOT)/autoconf/install-sh
MKDIR      := $(LLVM_SRC_ROOT)/autoconf/mkinstalldirs

But all this projects (LLVM, Stacker, sample, llvm-java) have
autoconf/install-sh and autoconf/mkinstalldirs in own sources.

I think at building, for example, llvm-java use
llvm-java/autoconf/install-sh is more correct (synchronized version with
other autoconf files and configure in project sources)

Is patch replacing

INSTALL_SH := $(LLVM_SRC_ROOT)/autoconf/install-sh
MKDIR      := $(LLVM_SRC_ROOT)/autoconf/mkinstalldirs

by

INSTALL_SH := $(PROJ_SRC_ROOT)/autoconf/install-sh
MKDIR      := $(PROJ_SRC_ROOT)/autoconf/mkinstalldirs

acceptable?

LLVM (including Stacker, sample), llvm-java build and installed with patch 
and without problems.
llvm-test have same install-sh and mkinstalldirs files as Stacker.

Vladimir
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Makefile.config.in.patch
Type: application/octet-stream
Size: 572 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20050215/2e68dbb6/attachment.obj>