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

Use SPDX identifier for license #1579

Merged
merged 1 commit into from
Oct 15, 2023
Merged

Conversation

Bananeweizen
Copy link
Contributor

Recommended by Maven documentation and Eclipse EMO.

Recommended by Maven documentation and Eclipse EMO.
@github-actions
Copy link

Test Results

0 files   - 107  0 suites   - 107   0s ⏱️ - 9m 58s
0 tests  - 662  0 ✔️  - 652  0 💤  - 10  0 ±0 
0 runs   - 662  0 ✔️  - 651  0 💤  - 11  0 ±0 

Results for commit 3692686. ± Comparison against base commit cea7f3b.

Copy link
Contributor

@HannesWell HannesWell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.
Thank you.
Could you please add links to the recommendations as comment?

@HannesWell
Copy link
Contributor

The build fails with the following error, but this change is very likely unrelated:

 [INFO] --- tycho-baseline:4.0.3:verify (baseline-check) @ org.eclipse.m2e.model.edit ---
 [INFO] Adding repository https://download.eclipse.org/technology/m2e/releases/latest
 [INFO] Adding repository https://download.eclipse.org/technology/m2e/releases/latest
 [ERROR] ┌──────────────────────────┬──────────────────────────┬──────────────────────────┬─────────────────────────┬─────────────────────────┬─────────────────────────┐
 [ERROR] │Delta                     │Type                      │Name                      │Project Version          │Baseline Version         │Suggested Version        │
 [ERROR] ├──────────────────────────┼──────────────────────────┼──────────────────────────┼─────────────────────────┼─────────────────────────┼─────────────────────────┤
 [ERROR] │MICRO                     │MANIFEST                  │org.eclipse.m2e.model.edit│2.0.300                  │2.0.300                  │2.0.301                  │
 [ERROR] │                          │                          │-2.0.300-SNAPSHOT.jar     │                         │                         │                         │
 [ERROR] ├──────────────────────────┼──────────────────────────┼──────────────────────────┴─────────────────────────┴─────────────────────────┴─────────────────────────┤
 [ERROR] │..CHANGED                 │HEADER                    │Bundle-License                                                                                          │
 [ERROR] └──────────────────────────┴──────────────────────────┴────────────────────────────────────────────────────────────────────────────────────────────────────────┘

I also noticed that the master build is failing since we upgraded to Tycho 4.0.3 while validation the m2e-repository:
https://ci.eclipse.org/m2e/job/m2e/job/master/
@laeubi is this something know?

@HannesWell HannesWell merged commit 248d8fc into eclipse-m2e:master Oct 15, 2023
4 of 7 checks passed
@mickaelistria
Copy link
Contributor

I think the failure here is related: the Bundle-License header is added by BND according to the license described in the pom file. If license changes in pom file, then the bundle changes and thus may need a version bump if there were no other change since last release.

@laeubi
Copy link
Member

laeubi commented Oct 16, 2023

I'm not sure if it is BND or Tycho (some items are read at Tycho level as well) so this seems valid.

But I would have exspected from the output to show the old and the new value (waht would have made this more clear to the user), so it would be good to create a Tycho issue or even a testcase for this.

@Bananeweizen
Copy link
Contributor Author

Bananeweizen commented Oct 16, 2023

Related references:

(As I mentioned in an earlier issue, I see such baseline verification errors since months locally when building m2e)

@Bananeweizen Bananeweizen deleted the spdx branch October 16, 2023 07:40
@laeubi
Copy link
Member

laeubi commented Oct 16, 2023

As I mentioned in an earlier issue, I see such baseline verification errors since months locally when building m2e

If you think there is a bug/problem please report it (at best with a reproucing testcase) so it could be fixed.

@mickaelistria
Copy link
Contributor

The failure here was related. It has caused build to fail constantly for 2 weeks.
Proper solution seems to be version bump of all modified bundles.

@Bananeweizen
Copy link
Contributor Author

thanks @mickaelistria for changing all the bundles now. I wasn't aware this single POM change could have such consequences. Will take note for future changes.

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.

4 participants