Skip to content

Ci scripts#14

Open
keenanlang wants to merge 8 commits into
masterfrom
ci-scripts
Open

Ci scripts#14
keenanlang wants to merge 8 commits into
masterfrom
ci-scripts

Conversation

@keenanlang

Copy link
Copy Markdown
Member

Adds a standard set of CI scripts to attempt to build on Windows, Mac, and Linux, as well as against Epics Base 3.15 and 7.0, both the most recent stable set of modules from synApps_6_3 and then the master versions of any necessary modules.

@nmarks99

Copy link
Copy Markdown
Collaborator

The Windows build fails because it cannot find boost.

The MacOS build finds boost, but fails to include this header: #include <boost/asio/io_service.hpp>. I encountered this same problem trying to build ur_rtde on arch linux with the latest boost. I think this is due to newer versions of boost having got rid of boost_system. See https://bbs.archlinux.org/viewtopic.php?id=309669 and boostorg/system@7a495bb. This maybe should be fixed upstream in ur_rtde, though for this MacOS build, setting boost to something <1.87 I think should work. I have Boost 1.75 on RHEL9 and it works fine

@keenanlang

Copy link
Copy Markdown
Member Author

Yeah, as of boost v1.87, boost::asio::io_service has been removed, the replacement boost::asio::io_context was added in v1.70. Homebrew has disabled all versions before 1.90, so we can't tag it to an earlier version.

Still working through the windows pathing issues, but the version from chocolatey is 1.87, so it will have the same issue with io_service.

After getting this to the point where windows and macos fail due to boost versioning issues, I can submit a PR with a boost version transition plan.

@keenanlang

Copy link
Copy Markdown
Member Author

Windows has started its actual build, so the ci scripts should be properly set up now, the failures are real. Created issue #15 to work on the deprecation of io_service for people who work on slightly more recent systems than we do.

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