----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129122/#review99871 -----------------------------------------------------------
cmake/FindUTEMPTER.cmake (line 45) <https://git.reviewboard.kde.org/r/129122/#comment67101> Why search in `KDE_INSTALL` folders? `ulog-helper` comes with base and should always be in `/usr/libexec`, no? - Gleb Popov On Oct. 8, 2016, 9:37 p.m., Tobias Berner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129122/ > ----------------------------------------------------------- > > (Updated Oct. 8, 2016, 9:37 p.m.) > > > Review request for KDE Frameworks, Adriaan de Groot, Gleb Popov, and Martin > Tobias Holmedahl Sandsmark. > > > Repository: kpty > > > Description > ------- > > FreeBSD does not have `/usr/libexec/*/utempter`. It does however have > `/usr/libexec/ulog-helper` [1]. > > It uses `login` instead of `add` and `logout` instead of `del`. > > > [1] > https://svnweb.freebsd.org/base/head/libexec/ulog-helper/ulog-helper.c?revision=234469&view=markup > > > Diffs > ----- > > cmake/FindUTEMPTER.cmake 9773963 > src/kpty.cpp 7bf31c3 > > Diff: https://git.reviewboard.kde.org/r/129122/diff/ > > > Testing > ------- > > Builds fine. Still need to test if it is actually working. > > > Thanks, > > Tobias Berner > >
