> On Oct. 23, 2013, 2:40 p.m., Chusslove Illich wrote: > > I can only say that whatever is the proper fix here, it is fine with me. > > Since non-installed headers are being used, maybe ki18n is using KJS in a > > way which became deprecated somewhere along the way? If so, I've nothing > > against fixing that instead. > > > > Aleix Pol Gonzalez wrote: > Well the thing is that ki18n was using private API (since it was in the > same module, kdelibs). > > Aurélien: what about installing these headers in a private/ directory? > > Chusslove Illich wrote: > If I recall, when implementing that I mimicked what khtml was doing. > And apparently is still doing. >
khtml is in kdelibs as well, nobody has tried to build it standalone just yet, I guess. - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113402/#review42222 ----------------------------------------------------------- On Oct. 23, 2013, 2:30 p.m., Aurélien Gâteau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113402/ > ----------------------------------------------------------- > > (Updated Oct. 23, 2013, 2:30 p.m.) > > > Review request for KDE Frameworks. > > > Repository: kdelibs > > > Description > ------- > > KI18n depends on KJS to build the ktranscript plugin. This plugin requires an > internal KJS Perl script as well as headers which were not installed. > > This is a 3-commit patches, which does the following: > > 1. Install kjs headers > > 2. Install wtf headers, using wtf/ and kjs/ in include directives (because > some kjs headers includes wtf headers) > > 3. Fix KI18n: > - Format top-level CMakeLists.txt according to CMake template > - Duplicate KJS create_hash_table script > - Add call to find_package(KJS) > > Individual patches available from > http://agateau.com/tmp/ki18n-standalone.patch > > > Diffs > ----- > > superbuild/CMakeLists.txt 5cdec94 > tier1/kjs/src/CMakeLists.txt 8629716 > tier1/kjs/src/kjs/CMakeLists.txt 9523e89 > tier1/kjs/src/wtf/CMakeLists.txt 83b4417 > tier1/kjs/src/wtf/FastMalloc.h 29a72a5 > tier1/kjs/src/wtf/HashCountedSet.h be3c387 > tier1/kjs/src/wtf/HashFunctions.h f665447 > tier1/kjs/src/wtf/HashMap.h ba2971c > tier1/kjs/src/wtf/HashSet.h e84b349 > tier1/kjs/src/wtf/HashTable.h 0b2c22c > tier1/kjs/src/wtf/HashTable.cpp e08d09a > tier1/kjs/src/wtf/HashTraits.h 4d01726 > tier1/kjs/src/wtf/ListRefPtr.h 0a704d8 > tier1/kjs/src/wtf/OwnArrayPtr.h 3b77871 > tier1/kjs/src/wtf/OwnPtr.h 188a1dc > tier1/kjs/src/wtf/PassRefPtr.h 25b9906 > tier1/kjs/src/wtf/RefPtr.h 493ab05 > tier1/kjs/src/wtf/Vector.h 9b0f38a > tier1/kjs/src/wtf/VectorTraits.h 31ae028 > tier2/ki18n/CMakeLists.txt 4cc8e30 > tier2/ki18n/src/CMakeLists.txt 7f8259b4 > tier2/ki18n/src/create_hash_table PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/113402/diff/ > > > Testing > ------- > > Builds within kdelibs and standalone. > > > Thanks, > > Aurélien Gâteau > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel