fix broken Windows build (#12)

* ad build-test workflow GitHub action

* version bump

Author: escherba

.github/workflows/build-test.yml

@@ -0,0 +1,36 @@
+# This is a partial whorkflow. Use wheels.yml for more
+# complete workflow.
+name: Build and Test
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    types:
+      - opened
+      - synchronize
+      - reopened
+
+jobs:
+  build:
+    strategy:
+      matrix:
+        os: [macos-10.15, windows-2019, ubuntu-latest]
+        python-version: [3.6, 3.7, 3.8, 3.9]
+
+    runs-on: ${{ matrix.os }}
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v2
+      with:
+        python-version: ${{ matrix.python-version }}
+        architecture: x64
+    - name: Install dependencies
+      run: |
+        pip install setuptools wheel
+        pip install -r requirements.txt
+        pip install .
+    - name: Test with pytest
+      run: python -m pytest

README.rst

@@ -34,18 +34,18 @@ To use this package in your program, simply type
 
 
 After that, you should be able to import the module and do things with it (see
-Example Usage below).
+usage example below).
 
 Usage Examples
 --------------
 
-Stateless Hashing
+Stateless hashing
 ~~~~~~~~~~~~~~~~~
 
 This package provides Python interfaces to 64- and 128-bit implementations of
 MetroHash algorithm. For stateless hashing, it exports ``metrohash64`` and
-``metrohash128`` functions. Both take a value to be hashed (either string or
-unicode) and an optional ``seed`` parameter:
+``metrohash128`` functions. Both take a value to be hashed and an optional
+``seed`` parameter:
 
 .. code-block:: python
 
@@ -57,7 +57,7 @@ unicode) and an optional ``seed`` parameter:
     182995299641628952910564950850867298725
 
 
-Incremental Hashing
+Incremental hashing
 ~~~~~~~~~~~~~~~~~~~
 
 For incremental hashing, use ``MetroHash64`` and ``MetroHash128`` classes.
@@ -82,7 +82,7 @@ Note that the resulting hash value above is the same as in:
     >>> mh.intdigest()
     7851180100622203313
 
-Buffer Protocol Support
+Buffer protocol support
 ~~~~~~~~~~~~~~~~~~~~~~~
 
 The methods in this module support Python `Buffer Protocol
@@ -129,8 +129,8 @@ extensions, see `CityHash <https://github.com/escherba/python-cityhash>`__ and
 
 Authors
 -------
-The original MetroHash algorithm was designed by J. Andrew Rogers. The Python
-bindings in this package were written by Eugene Scherba.
+The MetroHash algorithm and C++ implementation is due to J. Andrew Rogers. The
+Python bindings for it were written by Eugene Scherba.
 
 License
 -------

pip-freeze.txt

@@ -10,7 +10,7 @@ ipdb==0.13.9
 ipython==7.30.1
 jedi==0.18.1
 matplotlib-inline==0.1.3
--e git+https://github.com/escherba/python-metrohash@58720ab4383131ba9942cf728f237f0747f5286d#egg=metrohash
+-e git+https://github.com/escherba/python-metrohash@2e8b2c0ea0712339c65cda0c47649f9f1321243a#egg=metrohash
 numpy==1.21.5
 packaging==21.3
 parso==0.8.3
@@ -21,7 +21,7 @@ pluggy==1.0.0
 prompt-toolkit==3.0.24
 ptyprocess==0.7.0
 py==1.11.0
-Pygments==2.10.0
+Pygments==2.11.0
 pyparsing==3.0.6
 pytest==6.2.5
 six==1.16.0
@@ -30,4 +30,4 @@ traitlets==5.1.1
 typing_extensions==4.0.1
 virtualenv==20.11.2
 wcwidth==0.2.5
-zipp==3.6.0
+zipp==3.7.0

python.mk

@@ -81,7 +81,7 @@ env: env/bin/activate  ## set up a virtual environment
 env/bin/activate: setup.py requirements.txt
 	test -f $@ || virtualenv $(VENV_OPTS) env
 	export SETUPTOOLS_USE_DISTUTILS=stdlib; $(PYENV) curl https://bootstrap.pypa.io/ez_setup.py | $(INTERPRETER)
-	$(PIP) install -U pip
+	$(PIP) install -U pip wheel
 	$(PIP) install -r requirements.txt
 	$(PIP) install -e .
 	$(PIP) freeze > pip-freeze.txt

requirements.txt

@@ -5,4 +5,3 @@ numpy
 pytest
 setuptools
 virtualenv
-wheel

setup.py

@@ -1,5 +1,6 @@
 #!/usr/bin/env python
 # -*- coding: utf-8 -*-
+import os
 import warnings
 from os.path import join, dirname
 from setuptools import setup
@@ -29,11 +30,14 @@ def is_pure(self):
         return False
 
 
-CXXFLAGS = """
--O3
--Wno-unused-value
--Wno-unused-function
-""".split()
+CXXFLAGS = ["-O3"]
+
+if os.name != "nt":
+    CXXFLAGS.extend([
+        "-Wno-unused-value",
+        "-Wno-unused-function",
+    ])
+
 
 if HAVE_SSE42:
     warnings.warn("Compiling with SSE4.2 enabled")
@@ -82,7 +86,7 @@ def is_pure(self):
         )
 
 
-VERSION = '0.1.0.post6'
+VERSION = '0.1.1'
 URL = "https://github.com/escherba/python-metrohash"
 
 

src/metrohash.cpp

@@ -10,7 +10,7 @@ Python wrapper for MetroHash, a fast non-cryptographic hashing algorithm
 
 __author__  = "Eugene Scherba"
 __email__   = "escherba+metrohash@gmail.com"
-__version__ = "0.1.0.post6"
+__version__ = "0.1.1"
 __all__     = [
     "metrohash64",
     "metrohash128",