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

Add GitHub Actions Workflow & Basic Window Support #18

Open
wants to merge 10 commits into
base: cpp_standalone
Choose a base branch
from

Conversation

sanssecours
Copy link

Hi,

this pull requests adds a GitHub Actions workflow that compiles the library on

  • Linux,
  • macOS, and
  • Windows.

The PR also modifies the CMake code slightly to add support for Windows (Visual Studio). The updates are basically a result of a question of a co-worker, if I could help him to compile this library on Windows.

I hope the code updates do not break anything and might be helpful to others. Please be advised that I did not test the updates (yet) – I never used a laser scanner (neither from SICK nor from any other company). If I should change anything in the pull request, please let me know below. Thank you.

@puck-fzi
Copy link
Collaborator

puck-fzi commented Apr 4, 2023

Hi, thanks for opening this pull request. This might be helpful for others.
I will have a look through it and try to test it. Thank you

@puck-fzi
Copy link
Collaborator

Its currently not planned to include this PR, however I will leave it open, so others can benefit from your work

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.

2 participants