sysrepo / sysrepo-cpp

C++ bindings for the sysrepo library
BSD 3-Clause "New" or "Revised" License
6 stars 5 forks source link

compile error " sr_get_changes" #11

Closed olegzharov closed 2 years ago

olegzharov commented 2 years ago

Hi! I've got an error when start to build this:

_[vmware@srv-01 build]$ make [ 5%] Building CXX object CMakeFiles/sysrepo-cpp.dir/src/Connection.cpp.o [ 11%] Building CXX object CMakeFiles/sysrepo-cpp.dir/src/Enum.cpp.o [ 16%] Building CXX object CMakeFiles/sysrepo-cpp.dir/src/Session.cpp.o /home/vmware/sysrepo-cpp-master/src/Session.cpp: In member function ‘std::optional sysrepo::Session::getPendingChanges() const’: /home/vmware/sysrepo-cpp-master/src/Session.cpp:217:20: error: ‘sr_get_changes’ was not declared in this scope; did you mean ‘sr_has_changes’? 217 | auto changes = sr_get_changes(m_sess.get()); | ^~~~~~ | sr_haschanges /home/vmware/sysrepo-cpp-master/src/Session.cpp: At global scope: /home/vmware/sysrepo-cpp-master/src/Session.cpp:573:2: warning: extra ‘;’ [-Wpedantic] 573 | }; | ^ make[2]: [CMakeFiles/sysrepo-cpp.dir/src/Session.cpp.o] Error 1 make[1]: [CMakeFiles/sysrepo-cpp.dir/all] Error 2 make: *** [all] Error 2

Can you please help with this issue?

jktjkt commented 2 years ago

Hi, you'll need the latest, unreleased sysrepo for this.

michalvasko commented 2 years ago

Minor correction, released today.

olegzharov commented 2 years ago

will it be in Master branch or in Devel?

michalvasko commented 2 years ago

On a release devel is merged into master so in both, currently.