Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Files are installed into the root directory #69

Closed
yurivict opened this issue Mar 17, 2018 · 3 comments
Closed

Files are installed into the root directory #69

yurivict opened this issue Mar 17, 2018 · 3 comments

Comments

@yurivict
Copy link

@lvandeve

This commit on Sep 12, 2017 broke Makefile:
be5491d

Now it doesn't install libraries into $(PREFIX). It installs them into root.

Now installed files look like this:

/highwayhash/arch_specific.h
/highwayhash/c_bindings.h
/highwayhash/compiler_specific.h
/highwayhash/data_parallel.h
/highwayhash/endianess.h
/highwayhash/hh_avx2.h
/highwayhash/hh_buffer.h
/highwayhash/hh_portable.h
/highwayhash/hh_sse41.h
/highwayhash/hh_types.h
/highwayhash/hh_vsx.h
/highwayhash/highwayhash.h
/highwayhash/highwayhash_target.h
/highwayhash/highwayhash_test_target.h
/highwayhash/iaca.h
/highwayhash/instruction_sets.h
/highwayhash/load3.h
/highwayhash/nanobenchmark.h
/highwayhash/os_specific.h
/highwayhash/profiler.h
/highwayhash/robust_statistics.h
/highwayhash/scalar.h
/highwayhash/scalar_sip_tree_hash.h
/highwayhash/sip_hash.h
/highwayhash/sip_tree_hash.h
/highwayhash/state_helpers.h
/highwayhash/tsc_timer.h
/highwayhash/vector128.h
/highwayhash/vector256.h
/highwayhash/vector_test_target.h
/libhighwayhash.a
/libhighwayhash.so
/libhighwayhash.so.0
@yurivict yurivict changed the title Filed Files are installed into the root directory Mar 17, 2018
@jan-wassenberg
Copy link
Member

Oops, looks like we somehow removed the following from Makefile:

PREFIX ?= /usr/local	
INCDIR ?= $(PREFIX)/include	
LIBDIR ?= $(PREFIX)/lib

I've added this to TODO - thanks for reporting!

@yurivict
Copy link
Author

This is still broken.

@jan-wassenberg
Copy link
Member

At long last :) Sorry about the delay, we have been very busy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants