[kdevplatform] a5caf03: Make it possible to add pre-configured launch conf

[Aleix Pol]

I agree that it doesn't scale, I disagree on that we want a wizard.

Reasoning: we don't want a wizard to do the same as the dialog, the only
problem of the dialog is that people is too lazy to use it, so just adding
actions to create lauchers by default is both useful and simple.

Plan: I'll expose QMenu's instead of a list of QAction to provide project
granularity and maybe FS if necessary.

If we ever revive veritas test framework probably, we could remove tests
from this dialog, which would be good too.

Aleix

On Sun, Oct 24, 2010 at 12:51 AM, Milian Wolff <mail at milianw.de> wrote:

> On Friday 22 October 2010 09:03:05 Andreas Pakulat wrote:
> > On 21.10.10 21:51:02, Aleix Pol wrote:
> > > diff --git a/plugins/execute/nativeappconfig.cpp
> > > b/plugins/execute/nativeappconfig.cpp index 5edb957..7aadbfc 100644
> > > --- a/plugins/execute/nativeappconfig.cpp
> > > +++ b/plugins/execute/nativeappconfig.cpp
> > > @@ -395,5 +395,64 @@ void
> > > NativeAppConfigType::configureLaunchFromCmdLineArguments ( KConfigGroup
> > > cfg
> > >
> > >      cfg.sync();
> > >
> > >  }
> > >
> > > +QList<KDevelop::ProjectTargetItem*>
> > > targetsInFolder(KDevelop::ProjectFolderItem* folder) +{
> > > +    QList<KDevelop::ProjectTargetItem*> ret;
> > > +    foreach(KDevelop::ProjectFolderItem* f, folder->folderList())
> > > +        ret += targetsInFolder(f);
> > > +
> > > +    ret += folder->targetList();
> > > +    return ret;
> > > +}
> > > +
> > > +QList<QAction*> NativeAppConfigType::launcherSuggestions()
> > > +{
> > > +    static QList<QAction*> ret;
> > > +    if(!ret.isEmpty())
> > > +        return ret;
> > > +
>
> Hijacking thread:
>
> Sorry to say Aleix, but what you implemented there does not scale. Open
> more
> projects and the dropdown will easily fill your whole screen.
>
> I also dislike the place where this is implemented.
>
> Imo we should make the launch config dialog a real wizard and keep the
> current
> thingy to edit exiting configs.
>
> I imagine a big fat easy to understand button "add new config" that then
> asks
> me:
>
> a) what project
> b) what executable
> c) dependencies
> d) optional debugger settings
>
> bye
> --
> Milian Wolff
> mail at milianw.de
> http://milianw.de
>
> --
> KDevelop-devel mailing list
> KDevelop-devel at kdevelop.org
> https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20101025/cdace022/attachment-0001.html>