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

Update waf script (so it can run on Python 3) #144

Closed
wants to merge 5 commits into from
Closed

Conversation

Noeda
Copy link
Contributor

@Noeda Noeda commented Oct 29, 2019

The waf script that was in the repo was incompatible with Python 3.

I pulled the latest waf (2.0.18) and updated it.

Looks like macOS fails now because libarchive has become keg-only (so configure script can't find it because
it's not in usual location at /usr/local/XXX

I've added an extra brew link command to force it to put libarchive in the expected locations.
I was going try EXPORT-method to work around macOS brew bug but forgot
to remove this part of code before I committed in previous commit
the 'brew link' way instead.
MacOS is problematic with libarchive and homebrew no longer installs
libarchive as a non-keg-only package.

This commit adds dirty hacks to try cope with the Keg-only libarchive.
This hopefully makes travis tests still work and make user experience
of installing traildb from source less of a hassle.
@Noeda
Copy link
Contributor Author

Noeda commented Nov 1, 2019

I merged this PR manually and squashed it. Closing the PR.

@Noeda Noeda closed this Nov 1, 2019
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

Successfully merging this pull request may close these issues.

None yet

1 participant