-
Notifications
You must be signed in to change notification settings - Fork 16
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
remove codebox as it does not work with recent nodejs versions
- Loading branch information
Showing
1 changed file
with
6 additions
and
12 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bdf22bb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need codebox though 😦 it is so good for Docker and cloud images.
bdf22bb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry @jmchilton I just tried to get this image running and this seems to be a known problem: CodeboxIDE/codebox#517 and many more.
Codebox does not seem to be maintained anymore. Last release was one year ago and this one is not working :(
We can try to install a very old nodejs version, I will see if I can find time for this - 😢
On the bride side with these patches: https://github.com/galaxyproject/ansible-galaxy-os/pulls
This images can be build, planemo is running, just a few tweaks are needed ... and we have a new solid VM.
bdf22bb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jmchilton @bgruening
bdf22bb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bgruening It should not be your job to get an older version of node working... lets merge this as is and work toward fixing or replacing this. Che looks fascinating but it requires Docker - so it isn't really a very good solution from within Docker... maybe we should have a Docker compose recipe for configuring this and an IDE at the same time - would that work with the current Windows / Mac GUIs I wonder.
bdf22bb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @martenson for the info.
@jmchilton sounds good. This is probably for after GCC.
bdf22bb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does anyone remember if we used codebox during the conference? Pretty much no right - people with VMs were using Atom I think and I didn't see anyone using Docker with Codebox?
bdf22bb
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we did advertise it and I saw few people having it up, whether they actually used it for tool dev I dunno