Re: Review Request 124647: Cmake fail to parse proper taglib version

2015-08-13 Thread Luca Sartorelli
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124647/ --- (Updated Aug. 13, 2015, 12:38 p.m.) Status -- This change has been m

Re: Review Request 124647: Cmake fail to parse proper taglib version

2015-08-13 Thread Bhushan Shah
> On Aug. 13, 2015, 12:38 p.m., Bhushan Shah wrote: > > Do you have commit rights or you want me to commit it for you? > > Luca Sartorelli wrote: > Can you do it please, I don't have commit rigths. Thanks Submitted with http://commits.kde.org/plasma-mediacenter/4c242b2e04dbe6aaba5bd0766f2f

Re: Review Request 124647: Cmake fail to parse proper taglib version

2015-08-13 Thread Luca Sartorelli
> On Aug. 13, 2015, 7:08 a.m., Bhushan Shah wrote: > > Do you have commit rights or you want me to commit it for you? Can you do it please, I don't have commit rigths. Thanks - Luca --- This is an automatically generated e-mail. To repl

Re: Review Request 124647: Cmake fail to parse proper taglib version

2015-08-13 Thread Bhushan Shah
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124647/#review83759 --- Do you have commit rights or you want me to commit it for you?

Re: Review Request 124647: Cmake fail to parse proper taglib version

2015-08-08 Thread Bhushan Shah
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124647/#review83590 --- Ship it! FindTaglib.cmake is also used in various other repos

Re: Review Request 124647: Cmake fail to parse proper taglib version

2015-08-07 Thread Luca Sartorelli
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124647/ --- (Updated Aug. 7, 2015, 9:01 a.m.) Review request for Plasma. Changes --

Re: Review Request 124647: Cmake fail to parse proper taglib version

2015-08-07 Thread Luca Sartorelli
> On Aug. 6, 2015, 8:29 p.m., Rex Dieter wrote: > > Wouldn't it be better to use VERSION_GREATER > > (http://www.cmake.org/cmake/help/v3.0/command/if.html) rather than a string > > compare? That is great!, thank you for the smart suggestion, I didn't know about it - Luca --

Re: Review Request 124647: Cmake fail to parse proper taglib version

2015-08-06 Thread Rex Dieter
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124647/#review83511 --- Wouldn't it be better to use VERSION_GREATER (http://www.cmak

Review Request 124647: Cmake fail to parse proper taglib version

2015-08-06 Thread Luca Sartorelli
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124647/ --- Review request for Plasma. Repository: plasma-mediacenter Description -