pcaudiolib icon indicating copy to clipboard operation
pcaudiolib copied to clipboard

GitHub Action to build the code

Open cclauss opened this issue 11 months ago • 7 comments

@alex19EP Can you please approve the workflow to see if this build succeeds? Thanks.

  • Test results: https://github.com/cclauss/pcaudiolib/actions

cclauss avatar Jan 05 '25 21:01 cclauss

hmm. I can't do that. there no workflows to approve.

alex19EP avatar Jan 10 '25 11:01 alex19EP

Test results: https://github.com/cclauss/pcaudiolib/actions

macOS and Windows both fail with ./autogen.sh: line 13: aclocal: command not found

  • Add https://www.gnu.org/software/automake

cclauss avatar Jan 11 '25 10:01 cclauss

Wow! Ubuntu and macOS in Intel pass but macOS on ARM (Apple Silicon) fails!

Windows fails because I do not know how to install https://www.gnu.org/software/automake

cclauss avatar Jan 11 '25 10:01 cclauss

Let's land it like this so that at least the build passes on macOS on Intel and Ubuntu.

  • https://github.com/cclauss/pcaudiolib/actions

Someone with more C++ experience can propose fixes for the macOS on ARM (Apple Silicon) and Windows builds.

cclauss avatar Jan 11 '25 20:01 cclauss

How about changing to use CMake instead? will be much easier to build it cross platform

thewh1teagle avatar Jan 23 '25 20:01 thewh1teagle

@thewh1teagle Let's get the three passing tests merged before we make further changes.

If you are up for creating a pull request for a CMake build, please proceed.

cclauss avatar Jan 23 '25 21:01 cclauss

If you are up for creating a pull request for a CMake build, please proceed.

I started working on CMake support in thewh1teagle/pcaudiolib/feat/build-with-cmake. Will be nice if people join me (I enabled issues for discussion) It will take few days, I'll create new PR when it's ready

By the way, the CI part will be easy once CMake is added as well.

thewh1teagle avatar Jan 23 '25 22:01 thewh1teagle