TheRoddyWMS / Roddy

The Roddy workflow development and management system.
http://roddy-documentation.readthedocs.io
MIT License
8 stars 3 forks source link

Substitute 'dist/bin/current/' by 'dist/bin/develop' and additional changes #64

Closed dankwart-de closed 6 years ago

dankwart-de commented 7 years ago

I think we really need to discuss this again. I don't know if it is a wise idea to leave out current. current always stores the latest loader scripts and fixes for these should be available fast. They are independent from the actual processing but are necessary to guarantee a proper startup.

vinjana commented 7 years ago

Naveed thinks this is a wise idea -- to prevent that production data is analysed with development Roddy versions.

If we can ensure the dist/bin/current/roddy.sh can be run independently from all the other versions and the dir layout outside, I think we don't need a current/ handling of the top-level roddy.sh anymore. Obviously, with a github Roddy we cannot prevent people using development Roddy on production data, but the above solution would break the old bad habbit.

dankwart-de commented 7 years ago

Ok, makes sense. How about a switch: --allowcurrent for both Roddy and plugins? Default behaviour would be to disable it.

vinjana commented 7 years ago

Hmm, not sure. Let's hear @naveedishaque says. ...

dankwart-de commented 7 years ago

As discussed...

So =>