Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

local backend dont work with "cmd" shell #2612

Closed
6543 opened this issue Oct 19, 2023 · 0 comments · Fixed by #2636
Closed

local backend dont work with "cmd" shell #2612

6543 opened this issue Oct 19, 2023 · 0 comments · Fixed by #2636
Labels
backend/local bug Something isn't working

Comments

@6543
Copy link
Member

6543 commented Oct 19, 2023

if you have this config:

steps:
  # a demo step that say hello and show current work path
  hello:
    image: cmd.exe
    commands:
      - echo hello world
      - echo %CD%

for a local backend on windows you will get:
image
@6543 6543 added bug Something isn't working backend/local labels Oct 19, 2023
@6543 6543 mentioned this issue Oct 24, 2023
Merged
6543 added a commit that referenced this issue Oct 28, 2023
@6543
local backend ignore errors in commands inbetween (#2636)
5b3bba7 
for normal posix shells we have to add the `-e ` option ... but as there
are more shells out there we have to handle this edgecases on base per
base case.

create a switch case statement in woodpecker should be fine as there is
only a finite number of shells, used in production.

also close  #2612

---
*Sponsored by Kithara Software GmbH*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backend/local bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

local backend ignore errors in commands inbetween 6543-forks/woodpecker
1 participant
@6543