Re: [tracecompass-dev] Moving plugins to component sub-directories

[Alexandre Montplaisir <alexmonthy@xxxxxxxxxxxx>]

On 2015-05-05 12:51 PM, Matthew Khouzam wrote:
> Hi,
>
> This is super interesting and exciting.
>
> I would suggest we fix package tangle between the components before
> going forward though. If not it will look like spaghetti dangling from
> many pots instead of one.

It's completely orthogonal to packaging tangling. The package names or 
contents do not change.

Splitting tmf.core/ui in separate plugins though, that will reduce 
package/plugin tangle. But doing this will be less messy to do if "TMF" 
is in its own sub-directory already.

Cheers,
Alex

> Br
> Matthew
>
> On 15-05-04 07:02 PM, Alexandre Montplaisir wrote:
> > Hi all,
> >
> > I'm currently looking at moving all plugin directories in the source
> > tree to separate "component" directories. Like was done for doc/
> > already, so that eventually top-level would only contain:
> >
> > btf/
> > ctf/
> > doc/
> > lttng/
> > tmf/
> > ...
> >
> > Some are not immediately clear where they should go (examples,
> > releng-site,...) but we'll see once we get there. ;)
> >
> > I started with the BTF component, since it's quite simple and standalone:
> > https://git.eclipse.org/r/#/c/47104/
> > https://git.eclipse.org/r/#/c/47105/
> >
> > As explained in the first patch, I've renamed the feature plugins to
> > "*.feature". We've talked about adding the ".feature" to feature
> > plugin names previously, this is a good occasion to do so, since it
> > will free the o.e.t.{btf|ctf|...} names for the directories
> > themselves. I've added the previous names to the p2.inf files, so that
> > upgrades should follow the rename.
> >
> > Is there anything else that could have slipped my mind?
> >
> > I'm quite certain that this won't be all done in time for the feature
> > freeze, as more complex plugins with test traces and such will take
> > more time. Are we fine with just btf/ and pcap/ in 1.0, and the rest
> > later once stable-1.0 is branched?
> >
> >
> > Cheers,
> > Alexandre
> > _______________________________________________
> > tracecompass-dev mailing list
> > tracecompass-dev@xxxxxxxxxxx
> > To change your delivery options, retrieve your password, or
> > unsubscribe from this list, visit
> > https://dev.eclipse.org/mailman/listinfo/tracecompass-dev
> _______________________________________________
> tracecompass-dev mailing list
> tracecompass-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/tracecompass-dev