[CLI] Won't compile if path contains spaces. #498

aisrael · 2018-01-10T01:17:14Z

Description

If the full path to the current project directory contains any spaces, then amber can't build the project. Originally reported by @elorest

Steps to Reproduce

Spaces in parent path(s)

mkdir "stupid path"
cd "stupid path"
amber new test3 -d sqlite
cd test3
shards
amber w

Spaces in project path itself

amber new test3 -d sqlite
mv test3 "test three"
cd "test three"
shards
amber w

Expected behavior: amber w succeeds

Actual behavior:

09:00:10 Watcher	| Building project Test3...
Error in src/test3.cr:1: while requiring "../config/*": can't specify arguments in both, command and args without including "${@}" into your command

require "../config/*"
^

09:00:12 Watcher	| Starting Test3...
execvp: No such file or directory (Errno)
  from Process::exec_internal<String, Array(Pointer(UInt8)), Nil, Bool, (IO::FileDescriptor | Process::Redirect), (IO::FileDescriptor | Process::Redirect), (IO::FileDescriptor | Process::Redirect), Nil>:Nil
  from Process#initialize:output:error<String, Process::Redirect, Process::Redirect>:Nil
  from Process::new:output:error<String, Process::Redirect, Process::Redirect>:Process
  from Sentry::ProcessRunner#create_watch_process:Process
  from Sentry::ProcessRunner#create_all_processes:(IO+ | Nil)
  from Sentry::ProcessRunner#start_all_processes:(IO+ | Nil)
  from Sentry::ProcessRunner#start_app:(IO+ | Nil)
  from Sentry::ProcessRunner#scan_files:(IO+ | Nil)
  from Sentry::ProcessRunner#run:NoReturn
  from Amber::CLI::MainCommand::Watch@Sentry::SentryCommand#run:NoReturn
  from Amber::CLI::MainCommand::Watch#run:NoReturn
  from Cli::CommandClass+@Cli::CommandClass#amber_cli_main_command_watch__run<Cli::CommandBase+, Array(String)>:Cli::Exit
  from ~procProc(Cli::CommandBase, Array(String), Nil)@src/amber/cli/commands/watch.cr:8
  from Cli::CommandClass+@Cli::CommandClass#run<Amber::CLI::MainCommand, Array(String)>:Nil
  from Cli::CommandClass::Alias#run<Amber::CLI::MainCommand, Array(String)>:Nil
  from Amber::CLI::MainCommand@Cli::Supercommand#run:Nil
  from Cli::CommandClass+@Cli::CommandClass#amber_cli_main_command__run<Nil, Array(String)>:(Cli::Exit | Nil)
  from Cli::CommandClass+@Cli::CommandClass#amber_cli_main_command__run<Array(String)>:(Cli::Exit | Nil)
  from Amber::CLI::MainCommand::run<Array(String)>:(Cli::Exit | Nil)
  from __crystal_main
  from _crystal_main<Int32, Pointer(Pointer(UInt8))>:Nil
  from Crystal::main_user_code<Int32, Pointer(Pointer(UInt8))>:Nil
  from Crystal::main<Int32, Pointer(Pointer(UInt8))>:Int32
  from main

Reproduces how often: Always

Versions

$ amber --version
Amber CLI (amberframework.org) - v0.6.1

$ crystal --version
Crystal 0.24.1 (2017-12-26)

LLVM: 5.0.1
Default target: x86_64-apple-macosx

Additional Information

Even just crystal build src/test3.cr fails.

I was able to narrow it down line 25 in config/routes.cr:

24  routes :web do
25    get "/", HomeController, :index
26  end

When the get "/" ... line is commented out then the project builds properly.

The text was updated successfully, but these errors were encountered:

aisrael · 2018-01-10T03:08:50Z

Upon further diving, I've found that if you add require "./application.cr" to the top of config/routes.cr then issue crystal build config/routes.cr you get:

$ crystal build config/routes.cr
can't specify arguments in both, command and args without including "${@}" into your command
Process::prepare_argv<String, Array(String), Bool>:Tuple(String, Array(Pointer(UInt8)))
Crystal::Program#macro_run<String, Array(String)>:Crystal::Program::MacroRunResult
Crystal::MacroInterpreter#interpret_top_level_call?<Crystal::Call>:(Crystal::ASTNode+ | Nil)
Crystal::MacroInterpreter#interpret_top_level_call<Crystal::Call>:Crystal::ASTNode+
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MacroInterpreter>:Nil
Crystal::MacroInterpreter#visit<Crystal::MacroExpression>:Bool
Crystal::Program#expand_macro<Crystal::MacroExpression, Crystal::Type+, (Crystal::Type+ | Nil), (Hash(String, Crystal::ASTNode | Crystal::Type)+ | Nil), (Crystal::Def+ | Nil)>:String
Crystal::MainVisitor@Crystal::SemanticVisitor#expand_inline_macro<Crystal::MacroExpression, Nil>:Crystal::ASTNode+
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor@Crystal::SemanticVisitor#expand_macro<Crystal::Call, Bool, Bool>:Bool
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor@Crystal::SemanticVisitor#expand_macro<Crystal::Call, Bool, Bool>:Bool
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor#visit<Crystal::Block>:(Bool | Nil)
Crystal::MainVisitor#visit<Crystal::Yield>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor#visit<Crystal::Block>:(Bool | Nil)
Crystal::MainVisitor#visit<Crystal::Yield>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::Call#instantiate<Crystal::Matches, Crystal::Type+, Nil>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in_type<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil), Nil, String, Bool, Bool>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil)>:Array(Crystal::Def+)
Crystal::Call#recalculate:Nil
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::Call#instantiate<Crystal::Matches, Crystal::Type+, Nil>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in_type<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil), Nil, String, Bool, Bool>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil)>:Array(Crystal::Def+)
Crystal::Call#recalculate:Nil
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor@Crystal::SemanticVisitor#expand_macro<Crystal::Call, Bool, Bool>:Bool
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor@Crystal::SemanticVisitor#expand_macro<Crystal::Call, Bool, Bool>:Bool
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor#visit<Crystal::Assign>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor@Crystal::SemanticVisitor#expand_macro<Crystal::Call, Bool, Bool>:Bool
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::Call#instantiate<Crystal::Matches, Crystal::Type+, Nil>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in_type<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil), Nil, String, Bool, Bool>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil)>:Array(Crystal::Def+)
Crystal::Call#recalculate:Nil
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor#visit<Crystal::If>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor#visit<Crystal::Assign>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor@Crystal::SemanticVisitor#expand_macro<Crystal::Call, Bool, Bool>:Bool
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor@Crystal::SemanticVisitor#expand_macro<Crystal::Call, Bool, Bool>:Bool
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor#visit<Crystal::Block>:(Bool | Nil)
Crystal::MainVisitor#visit<Crystal::Yield>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::Call#instantiate<Crystal::Matches, Crystal::Type+, Nil>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in_type<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil), Nil, String, Bool, Bool>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil)>:Array(Crystal::Def+)
Crystal::Call#recalculate:Nil
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor@Crystal::SemanticVisitor#expand_macro<Crystal::Call, Bool, Bool>:Bool
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::MainVisitor#visit<Crystal::Block>:(Bool | Nil)
Crystal::MainVisitor#visit<Crystal::Yield>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::Call#instantiate<Crystal::Matches, Crystal::Type+, Nil>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in_type<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil), Nil, String, Bool, Bool>:Array(Crystal::Def+)
Crystal::Call#lookup_matches_in<Crystal::Type+, Array(Crystal::Type+), (Array(Crystal::NamedArgumentType) | Nil)>:Array(Crystal::Def+)
Crystal::Call#recalculate:Nil
Crystal::MainVisitor#visit<Crystal::Call>:Bool
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::ASTNode+@Crystal::ASTNode#accept<Crystal::MainVisitor>:Nil
Crystal::Program#visit_main<Crystal::ASTNode+, Crystal::MainVisitor, Bool, Bool>:Crystal::ASTNode+
Crystal::Program#semantic<Crystal::ASTNode+, Bool>:Crystal::ASTNode+
Crystal::Compiler#compile<Array(Crystal::Compiler::Source), String>:Crystal::Compiler::Result
Crystal::Command#run:(Bool | Crystal::Compiler::Result | IO::FileDescriptor | Nil)
main

Error: you've found a bug in the Crystal compiler. Please open an issue, including source code that will allow us to reproduce the bug: https://github.com/crystal-lang/crystal/issues

paulcsmith · 2018-01-10T04:00:00Z

We had a similar issue on Lucky: luckyframework/lucky_cli#147

It is actually a problem in Crystal that has likely been fixed by crystal-lang/crystal#5543

So hopefully the next Crystal release will fix this :D

robacarp · 2018-01-10T05:30:30Z

@paulcsmith ow!

elorest · 2018-01-10T05:48:21Z

Thanks @paulcsmith

faustinoaq · 2018-05-05T05:10:20Z

@amberframework/contributors Is this issue already fixed?

robacarp · 2018-05-10T01:07:39Z

Current behavior:

$ mkdir 'test yolo'
$ cd 'test yolo'
$ amber new test -d sqlite

09:04:21 Database   | (ERROR) Path and project name can't contain a space.
09:04:21 Database   | (ERROR) Replace spaces with underscores or dashes.
09:04:21 Database   | (ERROR) /Users/robert/Documents/repositories/amberframework/test yolo/test should be /Users/robert/Documents/repositories/amberframework/test_yolo/test

I guess the error message is more helpful than just breaking, but Amber shouldn't have an opinion on how people name the folders on their machines.

elorest · 2018-05-10T03:35:57Z

I don't think it's fixed yet. I believe there's a fix in the next version of crystal for this as it's not actually something uniq to amber.

eliasjpr · 2018-06-22T16:02:12Z

Any status on this?

faustinoaq · 2018-06-22T21:00:55Z

@eliasjpr still an issue on amber, although crystal looks good 😅

➜  ~ mkdir "foo bar"  
➜  ~ cd foo\ bar 
➜  foo bar amber new baz -d sqlite
03:58:31 Class      | (ERROR) Path and project name can't contain a space.
03:58:31 Class      | (INFO) Replace spaces with underscores or dashes.
03:58:31 Class      | (INFO) /home/main/foo bar/baz should be /home/main/foo_bar/baz
➜  foo bar crystal init app baz
    create  baz/.gitignore
    create  baz/.editorconfig
    create  baz/LICENSE
    create  baz/README.md
    create  baz/.travis.yml
    create  baz/shard.yml
    create  baz/src/baz.cr
    create  baz/src/baz/version.cr
    create  baz/spec/spec_helper.cr
    create  baz/spec/baz_spec.cr
Initialized empty Git repository in /home/main/foo bar/baz/.git/
➜  foo bar cd baz
➜  baz git:(master) ✗ shards build
Dependencies are satisfied
Building: baz

drujensen · 2018-12-24T19:10:30Z

We are providing an error message to let the user know this isn't allowed. If we feel we need to support spaces, we can reopen this.

eliasjpr added the kind:crystal-issue label Jan 11, 2018

eliasjpr added this to the Version TBD milestone Jan 20, 2018

eliasjpr changed the title ~~Won't compile if path contains spaces.~~ [CLI] Won't compile if path contains spaces. Feb 9, 2018

faustinoaq added status:on-hold kind:cli kind:crystal-issue and removed kind:crystal-issue labels Mar 17, 2018

faustinoaq mentioned this issue May 13, 2018

make install fails with spaces in CWD #781

Closed

faustinoaq mentioned this issue Oct 29, 2018

No such file or directory #968

Closed

drujensen closed this as completed Dec 24, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[CLI] Won't compile if path contains spaces. #498

[CLI] Won't compile if path contains spaces. #498

aisrael commented Jan 10, 2018

aisrael commented Jan 10, 2018

paulcsmith commented Jan 10, 2018

robacarp commented Jan 10, 2018

elorest commented Jan 10, 2018

faustinoaq commented May 5, 2018

robacarp commented May 10, 2018

elorest commented May 10, 2018

eliasjpr commented Jun 22, 2018

faustinoaq commented Jun 22, 2018

drujensen commented Dec 24, 2018

[CLI] Won't compile if path contains spaces. #498

[CLI] Won't compile if path contains spaces. #498

Comments

aisrael commented Jan 10, 2018

Description

Steps to Reproduce

Spaces in parent path(s)

Spaces in project path itself

Versions

Additional Information

aisrael commented Jan 10, 2018

paulcsmith commented Jan 10, 2018

robacarp commented Jan 10, 2018

elorest commented Jan 10, 2018

faustinoaq commented May 5, 2018

robacarp commented May 10, 2018

elorest commented May 10, 2018

eliasjpr commented Jun 22, 2018

faustinoaq commented Jun 22, 2018

drujensen commented Dec 24, 2018