Re: Review Request 125050: Fixed wrong path in cursor theme KCM

2015-09-04 Thread Ovidiu-Florin BOGDAN
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125050/ --- (Updated Sept. 4, 2015, 9:54 p.m.) Status -- This change has been ma

Re: Review Request 125050: Fixed wrong path in cursor theme KCM

2015-09-04 Thread Ovidiu-Florin BOGDAN
> On Sept. 4, 2015, 9:44 p.m., David Edmundson wrote: > > patch looks good, but why are you in the else branch of > > if(X11_Xcursor_FOUND) > > > > ? > > > > You want to fix that on the packaging side rather than having this patch > > Ovidiu-Florin BOGDAN wrote: > I'm working on the baloo

Re: Review Request 125050: Fixed wrong path in cursor theme KCM

2015-09-04 Thread David Edmundson
> On Sept. 4, 2015, 6:44 p.m., David Edmundson wrote: > > patch looks good, but why are you in the else branch of > > if(X11_Xcursor_FOUND) > > > > ? > > > > You want to fix that on the packaging side rather than having this patch > > Ovidiu-Florin BOGDAN wrote: > I'm working on the baloo

Re: Review Request 125050: Fixed wrong path in cursor theme KCM

2015-09-04 Thread Ovidiu-Florin BOGDAN
> On Sept. 4, 2015, 9:44 p.m., David Edmundson wrote: > > patch looks good, but why are you in the else branch of > > if(X11_Xcursor_FOUND) > > > > ? > > > > You want to fix that on the packaging side rather than having this patch I'm working on the baloo KCM, and this stopped me from compilin

Re: Review Request 125050: Fixed wrong path in cursor theme KCM

2015-09-04 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125050/#review84846 --- Ship it! Ship It! - David Edmundson On Sept. 4, 2015, 6:42

Re: Review Request 125050: Fixed wrong path in cursor theme KCM

2015-09-04 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125050/#review84845 --- patch looks good, but why are you in the else branch of if(X11