Skip to content

Cannot compile file via absolute path #969

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
awstlaur opened this issue Jan 13, 2017 · 3 comments
Open

Cannot compile file via absolute path #969

awstlaur opened this issue Jan 13, 2017 · 3 comments

Comments

@awstlaur
Copy link
Collaborator

Command

node build/phaseA/pyret.jarr 
  --build-runnable /home/alex/pyret-examples/example.arr
  --outfile /home/alex/pyret-examples/example.jarr
  --builtin-js-dir src/js/trove/
  --builtin-arr-dir src/arr/trove
  --require-config src/scripts/standalone-configA.json

Output

Gathering dependencies...The run ended in error:
Cannot find import dependency("file", [list: "/home/alex/pyret-examples/example.arr"])

Pyret stack:
  file:///home/alex/repos/pyret-lang/src/arr/compiler/cli-module-loader.arr: line 254, column 10
  file:///home/alex/repos/pyret-lang/src/arr/compiler/cli-module-loader.arr: line 245, column 4
  file:///home/alex/repos/pyret-lang/src/arr/compiler/cli-module-loader.arr: line 337, column 9
  file:///home/alex/repos/pyret-lang/src/arr/compiler/cli-module-loader.arr: line 384, column 18
  file:///home/alex/repos/pyret-lang/src/arr/compiler/pyret.arr: line 100, column 10
  file:///home/alex/repos/pyret-lang/src/arr/compiler/pyret.arr: line 69, column 4
  file:///home/alex/repos/pyret-lang/src/arr/compiler/pyret.arr: line 175, column 4

I am able to do this if I instead specify the same file with a relative path.

@SethPoulsen
Copy link
Contributor

I have the same problem.

SethPoulsen pushed a commit to SethPoulsen/pyret-lang that referenced this issue Mar 19, 2018
jpolitz added a commit that referenced this issue Aug 30, 2018
Handle absolute as well as relative paths (issue #969)
@schanzer
Copy link

@SethPoulsen @jpolitz did this eventually get merged into master?

@SethPoulsen
Copy link
Contributor

@schanzer looks like it did:

seth@seth:~/git/pyret-lang$ git co master
Branch 'master' set up to track remote branch 'master' from 'origin'.
Switched to a new branch 'master'
seth@seth:~/git/pyret-lang$ git log | grep "Handle absolute" -B 5
Author: Joe Politz <joe@cs.brown.edu>
Date:   Thu Aug 30 07:50:06 2018 -0700

    Merge pull request #1317 from SethPoulsen/absolute-paths
    
    Handle absolute as well as relative paths (issue #969)
--

commit a7dcd40d6d2ffd5ea6463272ae76b75ade25ff94
Author: Seth Poulsen <poulsenseth@gmail.com>
Date:   Mon Mar 19 16:59:38 2018 -0600

    Handle absolute as well as relative paths (issue #969)
seth@seth-t480s:~/git/pyret-lang$ 

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants