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

generate bom for the complete eclipse simrel #27

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

Conversation

amergey
Copy link

@amergey amergey commented Nov 1, 2022

Hello,

As a follow-up of issue #25 I reported some weeks ago, here a proposal to generate the bom for complete simrel
Some additional notes:

  • rap 2022-09 depends on eclipse platform 2022-09, rap from a version cannot depends on platform from another version, so upgrading rap to a version requires to upgrade platform as well, saying that, I am not sure modularization would help, as it is simpler to consume the whole module
  • I added a test case to check missing artifacts (wrong mapping or not available in maven central) and update the build to ignore test error, it seems odd to have a test that will always fail for sure but it is interesting to know what we miss
  • Feel free to take whatever you want from my PR, and leave things behind. I am not sure having the full bom is something you want so you can get rid of this part,but I have added some test case and the update of mapping file is probably interesting to keep.

Best

@jmini
Copy link
Owner

jmini commented Nov 8, 2022

Thank you very much for this PR.
It is a lot to process for me right now, but I will have a look

@amergey amergey force-pushed the master branch 2 times, most recently from 5768d7f to 0d7b506 Compare September 23, 2023 17:31
@amergey amergey force-pushed the master branch 2 times, most recently from 0c04af6 to b5f4818 Compare December 21, 2023 19:27
+ update gradle wrapper
[Gradle Release Plugin] - new version commit:  '2.0.2-SNAPSHOT'.
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.

2 participants