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

Indigo release items #5

Open
2 tasks
shaun-edwards opened this issue Aug 21, 2015 · 1 comment
Open
2 tasks

Indigo release items #5

shaun-edwards opened this issue Aug 21, 2015 · 1 comment

Comments

@shaun-edwards
Copy link
Contributor

The following is a quick list of the items that must be addressed before transitioning to Indigo:

  • Update yaml parsing to new yaml library API
  • CP 1616 driver does not compile under Ubuntu 14.04
@gavanderhoorn
Copy link
Member

Regarding the yaml-cpp changes: if there are no significant functional differences in the rest of the package between the Hydro and the Indigo version, it would perhaps make sense to localise the impact of the changes (ie: move yaml parsing to some helper functions) and use something like:

if(NOT ${YAML_CPP_VERSION} VERSION_LESS "0.5")
  add_definitions(-DHAVE_NEW_YAMLCPP)
endif()

in the CMakeLists.txt together with some conditional inclusion to keep the package compatible with both Hydro and Indigo.

gavanderhoorn pushed a commit to gavanderhoorn/siemens_experimental that referenced this issue Jan 9, 2016
…refixes

cp1616: remove prefixes from binaries
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

No branches or pull requests

2 participants