mirror of
https://github.com/nektos/act.git
synced 2026-04-26 09:25:54 +03:00
[GH-ISSUE #145] Seemingly invalid path separator on Windows using actions/setup-node #95
Labels
No labels
area/action
area/cli
area/docs
area/image
area/runner
area/workflow
backlog
confirmed/not-planned
kind/bug
kind/discussion
kind/external
kind/feature-request
kind/question
meta/duplicate
meta/invalid
meta/need-more-info
meta/resolved
meta/wontfix
meta/workaround
needs-work
pull-request
review/not-planned
size/M
size/XL
size/XXL
stale
stale-exempt
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/act#95
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Swiftwork on GitHub (Mar 11, 2020).
Original GitHub issue: https://github.com/nektos/act/issues/145
Running command
on
Microsoft Windows 10 Pro Build 10.0.18363usingrelease v0.2.6results in
This seems to be an issue with Windows and POSIX not using the same path separators. However I don't know where this issue lies, if its with nektos/act or actions/setup-node. If you can't reproduce the issue it occurs currently with trutoo/event-bus. It does however work in in my WSL.
@cplee commented on GitHub (Mar 11, 2020):
This looks like a bug with
act. In an effort to support windows, i overdid the use offilepath.Join()andfilepath.Separator. These create paths that are windows specific and are then fed in to the linux container 🤦♂️All the places that use those need to be reviewed to determine if the path is used on the host or within the container. The ones in the container should all use
/explicitly.@aslafy-z commented on GitHub (Mar 11, 2020):
This behavior has to be specific to Linux containers. In windows containers, path separator has to match the windows path separator (
\). I've seen that windows runners aren't supported in act from now so it may not be a priority but it guess it can be implemented in a adaptable way for the future.@ArwynFr commented on GitHub (Apr 4, 2020):
Workspace volume mount is impacted as well.
Instead of creating a hierarchic directory, all files are flattened in the same directory :
@github-actions[bot] commented on GitHub (Jun 4, 2020):
Issue is stale and will be closed in 7 days unless there is new activity
@Pomax commented on GitHub (Jun 7, 2020):
I'm running into this today, Windows 10 x64 with Latest Docker, github action running against ubuntu-latest,
actversion 0.2.9:Result:
@Pomax commented on GitHub (Jun 7, 2020):
So one thing to bear in mind: windows has had support for the
/path delimiter since literally before it was Windows; the only consideration is that you need to use double-quote paths with special characters. So sanitizing by using\/is actually a really bad idea and will more often than not outright break things.@fenilli commented on GitHub (Jun 7, 2020):
Did you find a workaround? having the same problem right now.
@Pomax commented on GitHub (Jun 8, 2020):
no I literally gave up after trying to get
actto actually work properly for several hours today. I'll just spent GA minutes on iterating on a script in a live repo instead. Good idea for a tool, but just nowhere near where it needs to be in order to make life easier, rather than harder.@fenilli commented on GitHub (Jun 8, 2020):
I will try running on wsl and see how it works, but this bug is really an annoying one.
@fenilli commented on GitHub (Jun 9, 2020):
As a workaround to this bug, you can use WSL2.
By installing windows 10 update 2004, and enabling wsl, updating the kernel to wsl 2, installing docker and enabling to use wsl2, then running the command act will work properly without invalid path separator.
@DUG-nick commented on GitHub (Jul 16, 2020):
Hi,
I would like to point out, that I could not replicate the workaround.
I am running Docker Version
2.3.0.3 (45519)with WSL2 on a freshly updated Windows 10 Pro.Sadly, I have nothing to contribute. Just issues reproducing the workaround :(
@Splode commented on GitHub (Jul 25, 2020):
I'm receiving this error as well running Windows 10 with Docker and WSL2. I'm trying to run act with the github-actions-demo project.
@satyaki1 commented on GitHub (Jul 29, 2020):
Windows users, please make sure you're on "Linux Container".
To check, right-click on Docker Desktop from task bar and choose "Switch to Linux Container". This will solve the problem.
This works, irrespective of your windows version compatible with WSL2 or not.
@Splode commented on GitHub (Jul 30, 2020):
Docker Desktop is running Linux containers in my case.
@Komposten commented on GitHub (Jul 30, 2020):
Ran act successfully using WSL 1 today, with Docker for Windows set to Linux containers.
Running it straight on Windows (using CMD, Git Bash, or similar) does not work. (In my case it was the cedx/setup-dart action rather than actions/setup-node.)
@satyaki1 commented on GitHub (Jul 30, 2020):
You may try adding to your environment path
COMPOSE_CONVERT_WINDOWS_PATHS0 or 1 as per your need, and run again. FYI@terabytesoftw commented on GitHub (Aug 19, 2020):
any solution
@github-actions[bot] commented on GitHub (Oct 19, 2020):
Issue is stale and will be closed in 7 days unless there is new activity