> On Nov. 7, 2011, 3:56 p.m., Ralf Jung wrote: > > Any comment on this? The patch looks trivial, but I am still new to all > > this so I'd prefer if someone could confirm that KActivites can and should > > actually require Soprano 2.7.
it builds here with 2.6.51. so 2.7 is not required, but perhaps some patch level version of 2.6. in any case, this CMakeLists.txt file should not be requiring 2.7 as it isn't actually required. - Aaron J. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102893/#review7993 ----------------------------------------------------------- On Oct. 16, 2011, 10:29 a.m., Ralf Jung wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102893/ > ----------------------------------------------------------- > > (Updated Oct. 16, 2011, 10:29 a.m.) > > > Review request for kdelibs. > > > Description > ------- > > I tried compiling KActivities with Soprano 2.6 installed, which resulted in > compile errors. After upgrading to Soprano 2.7, these were fixed. > > I assume this means that KActivities actually requires Soprano 2.7? If that's > the case, IMHO attached patch should be applied to detect that at cmake-time. > (I also hope this is the right list, there is no "kactivities" group in > reviewboard) > > > Diffs > ----- > > service/CMakeLists.txt 5489b6b > > Diff: http://git.reviewboard.kde.org/r/102893/diff/diff > > > Testing > ------- > > compile-tested > > > Thanks, > > Ralf Jung > >
