On Mon, 13 Sep 2010 09:09:27 +0200
Jaroslav Reznik <jreznik(a)redhat.com> wrote:
On Sunday, September 12, 2010 10:51:11 pm Kevin Fenzi wrote:
> On Sun, 12 Sep 2010 22:41:39 +0200
>
> Martin Sourada <martin.sourada(a)gmail.com> wrote:
> > On Sun, 2010-09-12 at 14:26 -0600, Kevin Fenzi wrote:
> > > Greetings.
> > >
> > > There's a reasonable request in:
> > >
https://bugzilla.redhat.com/show_bug.cgi?id=566744
> > > about moving the webkit inspector into the main package from the
> > > -devel subpackage.
> > >
> > > The inspector is useful for many people who may not want to
> > > install the needed 31(!) devel packages to have it.
> > >
> > > I looked at just moving it into the main package and it seems to
> > > work just fine in there in midori at least.
> > >
> > > There's a f14 scratch build here:
> > >
http://koji.fedoraproject.org/koji/taskinfo?taskID=2462354
> > >
> > > Thoughts? Any objections to moving it?
> >
> > I'm all for it. You might consider moving it to -webinspector
> > subpackage (or something like that) instead if anyone thinks it's
> > unnecessary clutter in the main package. IMHO having it -devel,
> > as it is now, is wrong. It's used for debugging/developing web
> > pages, not to build applications on top of webkitgtk..
>
> Yeah.
>
> It's 2.3MB of javascript and images. It seems fine to have in the
> main package from all I can see.
I agree with Martin - it does not belong to -devel package - it's
completely wrong but I tend more to put it to subpackage. It's
webdevelopers tool, most users are not interested in. And as for
example Midora is user as lightweight browser on smaller devices, I'm
not sure shipping bigger package makes sense + does any WebKit
browser reflect Inspector installed in UI?
well, 2.3MB seems hardly worth a subpackage to me. ;)
In midori at least there is a 'inspect page' in the right click menu.
If you don't currently have webkitgtk-devel installed, it just does
nothing at all. (yes, it should probibly error on the inspector not
being available). Even if it did say "Inspector not available", most
people wouldn't know to look for a subpackage to enable it, IMHO.
kevin