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

Dockerized image on Docker Hub #93

Open
symgryph opened this issue Jun 27, 2018 · 5 comments
Open

Dockerized image on Docker Hub #93

symgryph opened this issue Jun 27, 2018 · 5 comments

Comments

@symgryph
Copy link

https://hub.docker.com/r/symgryph/rperl/

@wbraswell
Copy link
Owner

@symgryph
Great work, thanks for posting this! Here are a few typos I found in your Docker page, please let me know when you have updated it:

"This is a Centos version of the RPERL compiler."
should be
"This is a CentOS version of the RPerl compiler."

"Centos version 7.5.1804"
should be
"CentOS version 7.5.1804"

"assuming you want to mount /home/myuser/perl/ as your directory to conver (mnt in the docker container)t:"
should be
"assuming you want to mount /home/myuser/perl/ as your directory to convert (mnt in the docker container):"

"Learning Rperl Document"
should be
"Learning RPerl Document"

@wbraswell
Copy link
Owner

@Util
Can you please do a brief check to see if this RPerl-in-CentOS Docker image works on your computer, and can successfully run the rperl -? command (at least)?

@Util
Copy link
Contributor

Util commented Jul 2, 2018 via email

@wbraswell
Copy link
Owner

@symgryph
Please provide us with the Dockerfile as described above.

@Util
RPerl is not meant to parse normal Perl code in the way you are attempting to use it, the RPerl grammar is very restrictive. If you want to use normal Perl syntax freely intermixed with RPerl syntax, then you can just run an RPerl program through the normal perl interpreter instead of the rperl compiler command. Your mixed-syntax code will be interpreted only, so the restrictive RPerl grammar will not come into play. Also, when run in compiled mode RPerl must compile all dependencies, so you can expect further RPerl grammar errors when including non-RPerl modules such as "File::Find" or basically any other Perl module which is not purposefully written using the restricted RPerl syntax. Again, this is easily side-stepped by simply running such code in interpreted mode only using perl instead of rperl.
...
That being said, I believe you are correct about there being some odd error with @inc or other dependency directory mechanisms, because 'sr/share/perl5/vendor_perl/Carp.pm' sure seems like it should be '/usr/share/perl5/vendor_perl/Carp.pm'...

@wbraswell
Copy link
Owner

@Util @symgryph
I am now circling back to this issue, Bruce I need you to please make a pull request from your forked repo of RPerl back into this main repo, so that we can inherit your Dockerfiles as they are now.

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