Review Request 115049: Do not require ktranscript to be installed for the test to run

[Aurélien Gâteau]

> On Jan. 16, 2014, 12:11 p.m., Aleix Pol Gonzalez wrote:
> > autotests/ktranscripttest.cpp, line 52
> > <https://git.reviewboard.kde.org/r/115049/diff/1/?file=234121#file234121line52>
> >
> >     This limits the cwd. Maybe you can pass it as a preprocess definition?
> >     aka with add_definitions(-DKTRANSCRIPT_PATH=${blah}) ).

What do you mean with "this limits the cwd"?


- Aurélien


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115049/#review47490
-----------------------------------------------------------


On Jan. 16, 2014, 12:07 p.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115049/
> -----------------------------------------------------------
> 
> (Updated Jan. 16, 2014, 12:07 p.m.)
> 
> 
> Review request for KDE Frameworks and Chusslove Illich.
> 
> 
> Bugs: 329994
>     https://bugs.kde.org/show_bug.cgi?id=329994
> 
> 
> Repository: ki18n
> 
> 
> Description
> -------
> 
> Kubuntu packagers had problems running "make test" because the ktranscript test relied on the ktranscript plugin being installed to run. This RR avoids this by loading the plugin from the build dir.
> 
> 
> Diffs
> -----
> 
>   autotests/ktranscripttest.cpp db8139a 
> 
> Diff: https://git.reviewboard.kde.org/r/115049/diff/
> 
> 
> Testing
> -------
> 
> Reverted install dir to a "before KF5" state, installed KJS, built ki18n without installing, ran the test successfully.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140116/f049e94f/attachment.html>