From 1a89fee6658179e4f2b13f62fc4451227c3e133d Mon Sep 17 00:00:00 2001
From: vsariola <5684185+vsariola@users.noreply.github.com>
Date: Sat, 8 May 2021 16:57:51 +0300
Subject: [PATCH] CI: don't test oto & remove libasound2-dev dependency

Installing this dependency failed in the cloud and caused tests to fail, so for now, we do not install it and do not test oto package.
The tests were actually about some float / int16 conversions, which should not anyway be in oto package, so future solution will be to refactor those functions somewhere else.
---
 .github/workflows/tests.yml | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index ff62aa6..4561715 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -50,9 +50,6 @@ jobs:
         node-version: '15'
     - uses: actions/checkout@v2
     - uses: ilammy/setup-nasm@v1.2.0
-    - name: Install libasound2-dev                   # sointu-cli has alsa as dependency for playing sound and
-      if: ${{ matrix.config.os == 'ubuntu-latest' }} # ubuntu was complaining about "Package alsa was not found in the pkg-config search path.",
-      run: sudo apt install libasound2-dev           # leading to tests failing. This fixes that.
     - name: Run ctest
       env:
         ASM_NASM: ${{ matrix.config.asmnasm }}
@@ -67,4 +64,4 @@ jobs:
       env:
         CGO_LDFLAGS: ${{ matrix.config.cgo_ldflags }}
       run: |
-        go test ./vm ./vm/compiler/bridge ./vm/compiler ./oto
+        go test ./vm ./vm/compiler/bridge ./vm/compiler