----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128916/#review99215 -----------------------------------------------------------
autotests/kconfigtest.cpp (line 1373) <https://git.reviewboard.kde.org/r/128916/#comment66803> getuid() isn't present on Windows. Guard this check with `#ifndef Q_OS_WIN`. - Gleb Popov On Sept. 15, 2016, 4:20 p.m., Evgeniy Sadovnik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128916/ > ----------------------------------------------------------- > > (Updated Sept. 15, 2016, 4:20 p.m.) > > > Review request for KDE Frameworks and Gleb Popov. > > > Repository: kconfig > > > Description > ------- > > The test checks that saving a read-only config file fails. But because root > can write into read-only files, the test is failing when running by root. > Check for uid when running the test and make it XFAIL if we are running as > root. > > > Diffs > ----- > > autotests/kconfigtest.cpp 2b905b5 > > Diff: https://git.reviewboard.kde.org/r/128916/diff/ > > > Testing > ------- > > > Thanks, > > Evgeniy Sadovnik > >
