Merge pull request bytedance#64 from bytedance/build-ci

add build check ci

Author: godweiyang

.github/workflows/build_check.yml

@@ -29,3 +29,34 @@ jobs:
       - name: check format
         run: |
           pre-commit run -a --show-diff-on-failure
+
+  build_wheel:
+    runs-on: ubuntu-latest
+    container: taka23/lightseq:build-linux
+
+    steps:
+      - uses: actions/checkout@v1
+        with:
+          submodules: 'recursive'
+      - name: env check
+        run: |
+          /opt/python/cp38-cp38/bin/python -V
+      - name: build wheel
+        run: |
+          /opt/python/cp38-cp38/bin/python -m pip install -U build
+          /opt/python/cp38-cp38/bin/python -m build
+
+  build_inference:
+    runs-on: ubuntu-latest
+    container: taka23/lightseq:build-linux
+
+    steps:
+      - uses: actions/checkout@v1
+        with:
+          submodules: 'recursive'
+      - name: env check
+        run: |
+          /opt/python/cp38-cp38/bin/python -V
+      - name: build ext
+        run: |
+          /opt/python/cp38-cp38/bin/python setup.py build_ext

setup.py

@@ -132,11 +132,6 @@ def build_extension(self, ext):
     logger.warning(e)
     logger.warning("The inference extension could not be compiled")
 
-    # Retry to install the module without C extensions :
-    # Remove any previously defined build_ext command class.
-    if "build_ext" in setup_kwargs["cmdclass"]:
-        del setup_kwargs["cmdclass"]["build_ext"]
-
     # If this new 'setup' call don't fail, the module
     # will be successfully installed, without the C extension :
     setup(**setup_kwargs)