Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

Resolve circular dependency between org.eclipse.mylyn.monitor.* and org.eclipse.mylyn.context.* #16

Open
ruspl-afed opened this issue Nov 27, 2022 · 6 comments

Comments

@ruspl-afed
Copy link
Contributor

Currently org.eclipse.mylyn.commons repository requires org.eclipse.mylyn.ide_feature feature to be built. We need to introduce an interface to resolve this.

@BeckerFrank
Copy link
Contributor

BeckerFrank commented Nov 27, 2022

Please se my comment in #14 (comment)

The following classes in org.eclipse.commons need some parts from org.eclipse.mylyn.context

  • ActivityContextManager
  • AbstractUserInteractionMonitor

Should I try to solve this issue?

@ruspl-afed
Copy link
Contributor Author

Of cause you are welcome to suggest your solution.

BeckerFrank added a commit that referenced this issue Nov 28, 2022
BeckerFrank added a commit to eclipse-mylyn/org.eclipse.mylyn.context that referenced this issue Nov 28, 2022
BeckerFrank added a commit to eclipse-mylyn/org.eclipse.mylyn.context that referenced this issue Nov 29, 2022
BeckerFrank added a commit to eclipse-mylyn/org.eclipse.mylyn.context that referenced this issue Nov 29, 2022
BeckerFrank added a commit to eclipse-mylyn/org.eclipse.mylyn.context that referenced this issue Nov 30, 2022
ruspl-afed added a commit that referenced this issue Dec 7, 2022
Use the latest nightly snapshot for Mylyn Context

Signed-off-by: Alexander Fedorov <alexander.fedorov@arsysop.ru>
ruspl-afed added a commit that referenced this issue Dec 7, 2022
Try to update to Tycho 3.0.1 to resolve java.lang.NoSuchMethodError:
'void org.eclipse.equinox.internal.p2.repository.helpers.ChecksumProducer

Signed-off-by: Alexander Fedorov <alexander.fedorov@arsysop.ru>
ruspl-afed added a commit that referenced this issue Dec 7, 2022
Remove circular dependency #16 : use nightly snapshot
@ruspl-afed
Copy link
Contributor Author

I'm not sure that this is the only circular dependency, let's postpone the implementation a bit until we can see the whole picture.

@BeckerFrank
Copy link
Contributor

Sure!

@BeckerFrank
Copy link
Contributor

I think that we no longer need this because we want to have a new big Repository with all plugins (eclipse-mylyn/.github#15)
@ruspl-afed, @wimjongman: Should we close this Issue?
Thoughts?

@ruspl-afed
Copy link
Contributor Author

Yes, we can postpone this issue . With one repository it will be easier to fix.
Sorry for postponing the review of your changes @BeckerFrank - currently I'm focused to establish healthy releng.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants