search

YunoHost-Apps / Experimental_helpers

6 stars 12 forks source link

Add java helper as for nodejs #104

Closed gcollin closed 1 year ago

gcollin commented 1 year ago

Seems like some other PR removed it (my fault)

yalh76 commented 1 year ago

Can you rename it to ynh_install_java to be more consistent to others language install

yalh76 commented 1 year ago

ynh_remove_java is missing

yalh76 commented 1 year ago

you should better base this helper from ynh_install_go of ynh_install_ruby

gcollin commented 1 year ago

Ok, I'll check with install_go and updates

gcollin commented 1 year ago

@yalh76 I renamed the scripts and added ynh_remove_java as well. However, I don't need to clean-up versions like for go because Java is installed using the yunohost package dependencies. It's removed when no other apps are using it.