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

First feedback on oomph setup #1

Open
chrisrueger opened this issue Sep 20, 2024 · 0 comments
Open

First feedback on oomph setup #1

chrisrueger opened this issue Sep 20, 2024 · 0 comments

Comments

@chrisrueger
Copy link

@peterkir thanks for the great work with this repo and the Oomph setups.
I did a first test and have some feedback:

FYI I took the liberty to enable Issues in this repo for now (we can disable it later if no longer needed). @pkriens

bndtools oomph notes

"/Applications/Eclipse Installer.app/Contents/MacOS/eclipse-inst" \
    https://raw.githubusercontent.com/bndtools/bndtools.p2.repo/master/setup/config_ecl-latest_bnd-next_on_next.setup

the oomph opened. Great 👍

Feedback

  • can we default to git HTTP readonly checkout ?
    (works for everybody right away... credentials can be setup later. e.g. current ssh default did not work for me because I have not set up SSH github stuff... using a git GUI client and usually HTTP github urls)
  • can we disable the "build" at the end of the installation oomph screen? Takes quite long... not sure what is building... maven / gradle plugins maybe?
  • compile errors after installation. Lots of compile errors in Eclipse Problems view: Most compile errors went away after pressing "Reload Workspace" in the Bndtools explorer. Could this be automated somehow or display some kind of hint?
  • Eclipse / Settings / Java / Compiler / Compiler compliance setting was set to Java 22. Can oomph initialize it with Java 17? Otherwise aQute.lib.collections.SortedList<T> shows a compile error (Duplicate default methods named addLast with the parameters (E) and (E) are inherited from the types List<T> and SortedSet<T>)
    I also had to change my mapping in Eclipse / Settings / Java / Installed JREs / Execution Environments / JavaSE-17 to my already installed Temurin JDK 17 because the one installed by oomph (/Users/me/.p2/pool/plugins/org.eclipse.justj.openjdk.hotspot.jre.full.macosx.x86_64_22.0.2.v20240802-1626/jre) was a Java 22 JRE

2nd approach via p2 repo directly

Also tried installing in an existing Eclipse instance with the 7.0.0 p2 repo url https://bndtools.org/bndtools.p2.repo/7.0.0/ at https://bndtools.org/bndtools.p2.repo/p2.html directly via Eclipse / Help / Install new Software (basically how you usually install stuff)

Worked like a charme.

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

No branches or pull requests

1 participant