Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

make install fixes #17

Merged
merged 2 commits into from

2 participants

@Benabik
Collaborator

The first commit fixes the fact that make install doesn't create PARROT_LIBRARY_DIR. This is a problem when someone (like me) uses DESTDIR to allow for clean install/removal. (Yay stow!)

The second one removes NQP_LANGUAGE_DIR which doesn't seem to be used anywhere. Feel free to drop this one if it serves some purpose I can't see.

Benabik added some commits
@Benabik Benabik [install] Create all directories before use
This mostly matters if someone is using DESTDIR
ad0564e
@Benabik Benabik [Makefile] Remove NQP_LANG_DIR variable
It doesn't seem to be used anywhere.
9764cc7
@moritz moritz merged commit 845d921 into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jul 12, 2011
  1. @Benabik

    [install] Create all directories before use

    Benabik authored
    This mostly matters if someone is using DESTDIR
  2. @Benabik

    [Makefile] Remove NQP_LANG_DIR variable

    Benabik authored
    It doesn't seem to be used anywhere.
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 2 deletions.
  1. +1 −2  tools/build/Makefile.in
View
3  tools/build/Makefile.in
@@ -41,7 +41,6 @@ CINCLUDES = -I$(PARROT_INCLUDE_DIR) -I$(PARROT_INCLUDE_DIR)/pmc
LINKARGS = $(LDFLAGS) $(LD_LOAD_FLAGS) $(LIBPARROT) @libs@ @icu_shared@
# nqp directories
-NQP_LANG_DIR = $(PARROT_LIB_DIR)/languages/nqp
DYNEXT_DIR = dynext
PMC_DIR = src/pmc
OPS_DIR = src/ops
@@ -220,7 +219,7 @@ CLEANUPS = \
all: $(NQP_EXE)
install: all
- $(MKPATH) $(DESTDIR)$(NQP_LANG_DIR)
+ $(MKPATH) $(DESTDIR)$(PARROT_LIBRARY_DIR)
$(CP) $(P6REGEX_PBC) $(DESTDIR)$(PARROT_LIBRARY_DIR)/$(P6REGEX_PBC)
$(CP) $(REGEX_PBC) $(DESTDIR)$(PARROT_LIBRARY_DIR)/$(REGEX_PBC)
$(CP) $(HLL_PBC) $(DESTDIR)$(PARROT_LIBRARY_DIR)/$(HLL_PBC)
Something went wrong with that request. Please try again.