> On Nov. 7, 2014, 3:34 p.m., Thomas Lübking wrote: > > kcms/autostart/autostart.cpp, line 174 > > <https://git.reviewboard.kde.org/r/121058/diff/2/?file=327177#file327177line174> > > > > I don't want to be annoyin, but "isAutostart" in "Autostart::load()" > > and below "autostartdir" is probably suboptimal ;-) > > > > "requiresServiceFile" or similar?
how about "inScriptSupportedDir" ? - Jeremy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121058/#review70020 ----------------------------------------------------------- On Nov. 7, 2014, 3:31 p.m., Jeremy Whiting wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121058/ > ----------------------------------------------------------- > > (Updated Nov. 7, 2014, 3:31 p.m.) > > > Review request for Plasma and Luca Beltrame. > > > Repository: plasma-desktop > > > Description > ------- > > KLauncher doesn't start these, so we shouldn't list them. > > > Diffs > ----- > > kcms/autostart/autostart.cpp 8fe81ad7fc752936cd8ba1c799ad06bafd2c4006 > > Diff: https://git.reviewboard.kde.org/r/121058/diff/ > > > Testing > ------- > > an executable script (ssh-add.sh) in ~/.config/autostart/ doesn't appear in > the kcm here anymore. > > > Thanks, > > Jeremy Whiting > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel