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

neurodocker generates files with unprintable characters [when used with docker] #180

Closed
satra opened this issue Jun 4, 2018 · 7 comments

Comments

@satra
Copy link
Contributor

satra commented Jun 4, 2018

@kaczmarj - we need to ensure that none of the generators contain non-printable characters.

+ '[' -n $'docker\r' -a -z '' ']'
+ '[' -x $'/usr/local/libexec/singularity/bootstrap-scripts/deffile-driver-docker\r.sh' ']'

on a unix system cat -v filename will show you the unprintable pieces. one can use dos2unix to remove them. otherwise, they go along even with copy and paste!

@kaczmarj
Copy link
Collaborator

kaczmarj commented Jun 4, 2018

@satra - i'm really sorry about all of the headaches that this bug caused. i can't see the unprintable characters on my debian machine with cat -v. the neurodocker command i tried is

neurodocker generate singularity \
  --base neurodebian:stretch \
  --pkg-manager apt \
  --install git emacs vim nano gedit

@satra
Copy link
Contributor Author

satra commented Jun 4, 2018

@kaczmarj - try this:

docker run -it --rm kaczmarj/neurodocker:master generate singularity   \
 --base neurodebian:stretch-non-free   --pkg-manager apt   \
 --install eatmydata fsl-5.0-core fsl-mni152-templates  \
 --run "eatmydata apt-get -y install --no-install-recommends file"   \
 --add-to-entrypoint "source /etc/fsl/5.0/fsl.sh"

@kaczmarj
Copy link
Collaborator

kaczmarj commented Jun 4, 2018

@satra - yes, i can reproduce it now. thanks. building a new neurodocker image to see if that fixes the issue

@kaczmarj
Copy link
Collaborator

kaczmarj commented Jun 4, 2018

interestingly, running the neurodocker docker image produces the non-printable characters, but running a pip-installed neurdocker does not produce those characters (at least on my linux machine)

@satra
Copy link
Contributor Author

satra commented Jun 4, 2018

ah - here you go:

moby/moby#8513 (comment)

@satra
Copy link
Contributor Author

satra commented Jun 5, 2018

@kaczmarj - perhaps we can update the readme. no fix is needed.

@satra satra changed the title [BUG] neurodocker generates files with unprintable characters neurodocker generates files with unprintable characters [when used with docker] Jun 5, 2018
@yarikoptic
Copy link
Member

FWIW. oh, just ran into it myself, I should remember to not use -t with a docker run of neurodocker!!!

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