Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [tracecompass-dev] Moving plugins to component sub-directories


On 15-05-05 02:05 PM, Alexandre Montplaisir wrote:
> 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.
I kinda sorta disagree on that point.

I will use oe.tmf.ctf.core as an example
It points to the internals of CTF.
It would probably go into TMF as it is TMF.ctf
We should cut dependencies on internals first. Would you have a better
term for this? Maybe project tangle?

Matthew
>
> 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
>
> _______________________________________________
> 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



Back to the top