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.

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

Fix spawn_cmd_background for windows. #1

Merged
merged 2 commits into from
Jul 3, 2024

Conversation

unta1337
Copy link
Contributor

@unta1337 unta1337 commented Jul 2, 2024

spawn_cmd_background가 cmd를 통해 명령을 실행하려고 할 때 아래와 같은 문제가 발생함을 확인했습니다.
image

cmd를 정의하는 부분이 문제가 되는 거 같은데,

let cmd = format!("START /B \"\" {}", cmd);

위 코드에서 \""로 변환되어야 하지만 어째선지 역슬래시가 cmd에 그대로 들어가는 거 같습니다.

image
start /B \"\" geckodriver를 cmd에 입력했을 때 동일한 오류 메시지 창을 확인할 수 있었습니다.

백그라운드 실행과 관련해서 ""가 있는 거 같은데 ""없이 /B만 있어도 백그라운드에서 동작하게 할 수 있는 것 같습니다.

@unta1337 unta1337 marked this pull request as draft July 2, 2024 12:55
@unta1337 unta1337 marked this pull request as ready for review July 2, 2024 13:27
@unta1337
Copy link
Contributor Author

unta1337 commented Jul 2, 2024

백그라운드에서 실행된 geckodriver가 가보자 종료 시 적절히 종료되지 않아 관련 코드를 추가했습니다.

@Bubbler-4 Bubbler-4 merged commit 7591f79 into boj-rs:main Jul 3, 2024
@Bubbler-4
Copy link
Collaborator

기여 감사합니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants