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

bin/omero-importer #25

Closed
jburel opened this issue May 6, 2019 · 9 comments
Closed

bin/omero-importer #25

jburel opened this issue May 6, 2019 · 9 comments

Comments

@jburel
Copy link
Member

jburel commented May 6, 2019

bin/omero-importer does not work
Could not find or load main class org.openmicroscopy.shoola.Main

@mtbc
Copy link
Member

mtbc commented Jan 14, 2020

Still true for 5.5.8.

@PaulVanSchayck
Copy link
Contributor

Still true for 5.5.9.

In case the cause was unclear. The startup scripts misses the contents of the CLASSPATH variable. After unzipping. Look in file bin/omero-importer file. The CLASSPATH= is empty.

When starting with empty CLASSPATH, you get the following error

$ ./bin/omero-importer 
Error: Could not find or load main class org.openmicroscopy.shoola.Main

Copying over the contents of the variable from the insight startup script fixes the issue.

@jburel
Copy link
Member Author

jburel commented Apr 9, 2020

@PaulVanSchayck are you using the importer? The reason I am asking is that we are considering not building the artifacts anymore

@PaulVanSchayck
Copy link
Contributor

Haha.. you've been considering that for years, haven't you? Every time I report an issue with the importer I get told exactly this :P.

We like the simple GUI. Especially because we tell users to use the webclient mostly, and hardly anyone uses the insight client. So we have only the importer installed on our microscopes and workstations.

@jburel
Copy link
Member Author

jburel commented Apr 9, 2020

See https://forum.image.sc/t/omero-desktop-client-omero-insight/35234
The Desktop client will stay e.g. insight.dmg, we were just considering not bundling the importer as a standalone app.
If it is useful, we will keep bundling it.

@PaulVanSchayck
Copy link
Contributor

Hmm I see..

Would it be an option to include the omero-importer startup script in the insight artifact? Or as command line option to the insight startup script? Or do you think that's confusing?

@jburel
Copy link
Member Author

jburel commented Apr 10, 2020

I have fixed the problem.
We did not hear that people still needed and it seems that you still do so we will keep building the artifact
I have also automated the generation of the artifacts via travis so it should not be a problem.

@jburel jburel mentioned this issue Apr 10, 2020
@jburel
Copy link
Member Author

jburel commented Apr 10, 2020

see #127

@jburel jburel closed this as completed Apr 10, 2020
@jburel
Copy link
Member Author

jburel commented Apr 16, 2020

Fixed in 5.5.10

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

3 participants