esy / esy/esy-bash

Occasional syntax errors in the temp sh file

Open
#89 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
11
Forks
3
PR merge metrics
No merged PRs in 30d

Description

Sometime `__esy-bash__824592099__1__` file has syntax errors like these

```
mount -c /cygdrive -o binary,noacl,posix=0,user;
cd "C:/Users/user/b-runner/gl/builds/0/xxxxxxxx/ligolang/ligo/_esy/default/store/b/opam__s__tezos_base-842c849a";
"C:/Users/user/AppData/Roaming/npm/node_modules/esy/node_modules/esy-bash/.cygwin/bin/rm.exe" "-rf" "vendors";s";
```

This is likely happening because the random number collisions in the filename, causing file overwrites. A good solution could be clearing these files after use. They end up getting accumulated in the current state of things.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating where __esy-bash__ temporary files are generated and cleaned up, then reproduce the filename-collision case in the Cygwin environment described. Done means temporary files cannot overwrite one another and are removed after use; check for existing tests covering temporary shell-file handling and extend them if present.

Written by the indexing model from the issue text.

Assessment

Tech stack
bash, javascript
Domain
tooling
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.