IppClub / IppClub/YueScript

target not accepting paths with spaces

Open
#136 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Lua
Stars
613
Forks
45
PR merge metrics
No merged PRs in 30d

Description

-t path Specify where to place compiled files

I tried single quotes, double quotes and no quotes. None resulted in the files being compiled in the right place.

Microsoft Windows [Version 10.0.19045.2965]
(c) Microsoft Corporation. All rights reserved.

D:\user\projects\lua\yue\another_project>..\yue --target=5.1 -s -t 'D:\path with spaces\Yue\' yue\

D:\user\projects\lua\yue\another_project>..\yue --target=5.1 -s -t "D:\path with spaces\Yue\" yue\
no input files

D:\user\projects\lua\yue\another_project>..\yue --target=5.1 -s -t D:\user\path with spaces\Yue\ yue\
Built yue\main.yue
Failed to read file: path
Failed to read file: with
Failed to read file: spaces\Yue\

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the command-line handling for the --target/-t option and reproduce the reported Windows commands using a path containing spaces. Done means YueScript accepts the quoted target path and places compiled files there without treating parts of the path as input files.

Written by the indexing model from the issue text.

Assessment

Tech stack
lua
Domain
cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.