Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update tycho to 4.0.7 #1733

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update tycho to 4.0.7 #1733

wants to merge 2 commits into from

Conversation

akurtakov
Copy link
Contributor

No description provided.

@akurtakov akurtakov self-assigned this Apr 9, 2024
Copy link

github-actions bot commented Apr 9, 2024

Test Results

  107 files  ±  0  107 suites  ±0   1m 51s ⏱️ - 7m 25s
  665 tests ±  0  151 ✅  - 503  10 💤 ±0  0 ❌  - 1    504 🔥 +  504 
1 169 runs  +504  151 ✅  - 503  10 💤 ±0  0 ❌  - 1  1 008 🔥 +1 008 

For more details on these errors, see this check.

Results for commit b784e23. ± Comparison against base commit e76a84d.

♻️ This comment has been updated with latest results.

@akurtakov akurtakov force-pushed the master branch 2 times, most recently from c971827 to e8049f4 Compare April 9, 2024 08:43
@HannesWell
Copy link
Contributor

All these test-failures happen also in the attempt to update to Tycho 4.0.6: #1688

I haven't investigated it yet, maybe @laeubi can tell?

@laeubi
Copy link
Member

laeubi commented May 18, 2024

From the buildfailure it is a problem due to classfile changes (ECJ update?)

@HannesWell
Copy link
Contributor

From the buildfailure it is a problem due to classfile changes (ECJ update?)

The update to Tycho 4.0.6 already failed in #1688 and IIRC the comparator errors were not the problem and the fixed ECJ version was removed in an attempt to resolve that failure, but it seems like it didn't fix it.
But lets fix the comparator errors first.

@HannesWell HannesWell force-pushed the master branch 3 times, most recently from e150616 to 8074444 Compare May 18, 2024 10:51
@HannesWell
Copy link
Contributor

From the buildfailure it is a problem due to classfile changes (ECJ update?)

The comparator errors are resolved and now we (again) have the 500 test-failures.

@laeubi
Copy link
Member

laeubi commented May 18, 2024

As they all fail the same way maybe some change in P2 causing this, I would try to add a direct dependency in the test plugin to make sure its not caused by some indirect dependency missing now.

@HannesWell
Copy link
Contributor

As they all fail the same way maybe some change in P2 causing this, I would try to add a direct dependency in the test plugin to make sure its not caused by some indirect dependency missing now.

The plugin org.eclipse.m2e.tests already requires the bundle org.eclipse.m2e.maven.runtime.

@laeubi
Copy link
Member

laeubi commented May 18, 2024

As they all fail the same way maybe some change in P2 causing this, I would try to add a direct dependency in the test plugin to make sure its not caused by some indirect dependency missing now.

The plugin org.eclipse.m2e.tests already requires the bundle org.eclipse.m2e.maven.runtime.

The error says that the bundle org.eclipse.m2e.tests.common can not find the class...

@HannesWell
Copy link
Contributor

As they all fail the same way maybe some change in P2 causing this, I would try to add a direct dependency in the test plugin to make sure its not caused by some indirect dependency missing now.

The plugin org.eclipse.m2e.tests already requires the bundle org.eclipse.m2e.maven.runtime.

The error says that the bundle org.eclipse.m2e.tests.common can not find the class...

Right (I just read the first line and checked the test class). But even org.eclipse.m2e.tests.common already requires org.eclipse.m2e.maven.runtime.

@akurtakov akurtakov force-pushed the master branch 5 times, most recently from 31186b1 to 86d7ee3 Compare May 21, 2024 08:16
@akurtakov akurtakov force-pushed the master branch 2 times, most recently from 7bac292 to 08020fa Compare May 21, 2024 08:33
@laeubi
Copy link
Member

laeubi commented Jun 1, 2024

Interestingly I get these errors locally as well with Tycho 4.0.4

@laeubi
Copy link
Member

laeubi commented Jun 3, 2024

As I can "reproduce" it here with Tycho 4.0.4 I suspect it must be some kind of classpath ordering problem, e.g that for consistency we order the dependencies differently (either Tycho or P2)...

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

Successfully merging this pull request may close these issues.

None yet

3 participants