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

Restore compatibility with latest django-haystack #160

Merged
merged 5 commits into from Oct 30, 2016

Conversation

claudep
Copy link
Collaborator

@claudep claudep commented Oct 14, 2016

Please double-check the second commit of this series.

@coveralls
Copy link

coveralls commented Oct 14, 2016

Coverage Status

Coverage decreased (-0.04%) to 96.605% when pulling bbb4cde on claudep:25compat into ea2d9fe on notanumber:master.

@coveralls
Copy link

coveralls commented Oct 14, 2016

Coverage Status

Coverage decreased (-0.04%) to 96.605% when pulling e0e6a11 on claudep:25compat into ea2d9fe on notanumber:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 96.605% when pulling e0e6a11 on claudep:25compat into ea2d9fe on notanumber:master.

@jorgecarleitao
Copy link
Collaborator

Thanks a lot for this @claudep. I haven't forgotten. I am trying to fix the test problem in Python 3.

@claudep
Copy link
Collaborator Author

claudep commented Oct 26, 2016

Yes, would be great if you could sort the xapian 4/python 3 install issue!

@jorgecarleitao jorgecarleitao merged commit e0e6a11 into notanumber:master Oct 30, 2016
@jorgecarleitao
Copy link
Collaborator

This has been merged to master as I was able to fix the Python 3 install issue. Essentially, during installation, we were running Python while we were in a removed directory. This was an issue in Python 3 (Python issue 22834). The fix was to cd to another directory before running Python.

This fixed all issues for upgrading for Xapian 1.4.1 and therefore Xapian-Haystack now supports Xapian 1.4.1. Thanks again for the help @claudep.

@claudep
Copy link
Collaborator Author

claudep commented Oct 30, 2016

Thanks for merging and for solving the remaining issue!

@claudep claudep deleted the 25compat branch October 30, 2016 18:46
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

3 participants