<div dir="ltr">Hi,<div><br></div><div>I found a way to better manage individual review requests separated by using different local branches.</div><div>I recreated the review request for the additional 2 fields (boardgame).</div><div>I'm sorry for all the mails being generated on the mailing list because of this.</div><div><br></div><div>> <span style="font-size:12.8000001907349px">Thanks, Sven. At first glance, those looks great. I'm on travel right now, but I'll get them pulled shortly.</span></div><div><span style="font-size:12.8000001907349px">Sure. No problem. Let me know if any of my contribution should be changed.</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">Btw... I love tellico. Thanks for creating it and maintaining it all these years!</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><i><span style="font-size:12.8000001907349px">Note: I also found the problem with </span><span style="font-size:12.8000001907349px">tellicoui.rc and </span><span style="font-size:12.8000001907349px">/usr/share/tellico not beeing found.</span></i></div><div><span style="font-size:12.8000001907349px"><i>I had to change the cmake install path to /usr (default is /usr/share).</i></span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">Sven</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px"> </span></div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-01-08 6:13 GMT-05:00 Robby Stephenson <span dir="ltr"><<a href="mailto:robby@periapsis.org" target="_blank">robby@periapsis.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 01/06/2016 10:00 PM, Sven Werlen wrote:<br>
</span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
Hi Robby,<br>
<br>
I filed 2 review board requests:<br>
<a href="https://git.reviewboard.kde.org/r/126656/" rel="noreferrer" target="_blank">https://git.reviewboard.kde.org/r/126656/</a><br>
</span><a href="https://git.reviewboard.kde.org/r/126657/" rel="noreferrer" target="_blank">https://git.reviewboard.kde.org/r/126657/</a><<a href="https://git.reviewboard.kde.org/r/126656/" rel="noreferrer" target="_blank">https://git.reviewboard.kde.org/r/126656/</a>><br>
/(in the second one, I only wanted to include the changes for the 2<br>
additional fields but it automatically add the bug fix as well)/<br>
</blockquote>
<br>
Thanks, Sven. At first glance, those looks great. I'm on travel right now, but I'll get them pulled shortly.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 > Are you developing againsht master or the 2.3 branch?<span class=""><br>
I used "master" branch for developping and testing. Should I rather use<br>
the 2.3 branch?<br>
</span></blockquote>
The 2.3 branch is for KDE 4. I just wanted to be sure you were intentionally working with the KDE Frameworks 5 branch. I'll probably have one more 2.3.x release, I think, but that's it.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
In order to make Tellico work from my build folder, I had to apply 2<br>
"hacks":<br>
<br></span>
 1. Add #define UIFILE QString::fromUtf8("PATH TO tellicoui.rc file") in<br>
    src/mainwindow.h<br>
 2. Create a symbolic link /usr/share/tellico to my tellico folder<span class=""><br>
<br>
Without 1), Tellico was not able to find tellicoui.rc file and some menu<br>
items were missing. I tried to change the "working dir" in KDevelop<br>
without succes.<br>
Without 2), Tellico was complaining about not finding Fancy.xsl at the<br>
startup. I couldn't find any other way to make it work.<br>
<br>
Is there any other way to configure KDevelop and make it work?<br>
</span></blockquote>
<br>
I'll have to check. Not having Tellico installed otherwise, you mean?<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Last question: I think that "./src/fetch/scripts/boardgamegeek.rb" is<br>
not used anymore. Can you confirm?<br>
</blockquote>
<br></span>
I think so, probably just need to remove it from that branch.<br>
<br>
Thanks!<span class="HOEnZb"><font color="#888888"><br>
Robby<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Sven</div></div>
</div>