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

Arctic space fox/feature/apl #39

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ArcticSpaceFox
Copy link

APL lang integration - #37

Changes

  • Dockerfile.apl sets up aplc to compile APL code
  • apl.rs language configuration
  • mod.rs added apl mod
  • lang_manager.rs added commands for apl [apl, dyalog]

Notes

  • everything but the Dockerfile has not been tested yet

@Kryod
Copy link
Owner

Kryod commented Jan 19, 2021

It's not failing because of syntax error in your APL test (maybe there are but it's not the problem), it's because Travis can't build the image rustacean-apl.

It seems you are trying to copy a rootfs folder that doesn't exist in this repository (so only on your local machine I assume). Is this folder really necessary?

@ArcticSpaceFox
Copy link
Author

No its from the ubuntu docker image, just copying it back. You can test it, it works. The fix was because there would have been a syntax error, now it should work. Just didnt want any more errors then needed

@Kryod
Copy link
Owner

Kryod commented Jan 19, 2021

I tested it doesn't work, same error as the one on the Travis CI .
image

@Kryod
Copy link
Owner

Kryod commented Jan 19, 2021

And for further information COPY can take files from the build context, not what's on the image.

@ArcticSpaceFox
Copy link
Author

Ok changes made, it compiles and its tested with the tests on my linux system

@Kryod
Copy link
Owner

Kryod commented Jan 19, 2021

Now your test doesn't pass because of a different reason:
image

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

Successfully merging this pull request may close these issues.

2 participants