Skip to content
This repository has been archived by the owner on Jul 8, 2022. It is now read-only.

TANGO-678 Migrate to cmake #1

Merged
merged 49 commits into from
Sep 16, 2016
Merged

Conversation

Ingvord
Copy link
Member

@Ingvord Ingvord commented Sep 13, 2016

In progress

@taurel
Copy link
Collaborator

taurel commented Sep 14, 2016

Igor,

Some comments:

  • At ESRF, we are still using a lot of Debian 7 host where the default cmake release is 2.8. May be it's better to have a minimum cmake version required to 2.8
  • I haven't seen the compiler option for c++11
  • I think we need a way to specify where is omniORB or ZMQ

@Ingvord
Copy link
Member Author

Ingvord commented Sep 14, 2016

Hi Emmanuel,

Sure we can set minimum version to 2.8;

C++11 will be added;

you mean some external variable, like --omiorb-lib-path ?

Copy link
Member

@bourtemb bourtemb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,
we just tried it and got some errors while trying it on our debian7 development machine. Could you please test your changes on our development machines to ensure we can compile the project there?
It would be good to give instructions explaining how to compile for people not familiar with cmake (in the README).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants