search

SNU-ARC / 2024_spring_sysprog_Lab4

8 stars 0 forks source link

README에 등장하는 `kill` built-in command #15

Open m-joon-ixix opened 4 months ago

m-joon-ixix commented 4 months ago

안녕하세요, readme를 보면 built-in command로 kill 이라는 것이 등장하는데요, https://github.com/SNU-ARC/2024_spring_sysprog_Lab4/blob/03e02fe213f3398992de91f5de1df244770b5ebe/README.md?plain=1#L152 이것도 구현해야 할 대상에 포함되는지 궁금합니다.

다른 부분을 보면 https://github.com/SNU-ARC/2024_spring_sysprog_Lab4/blob/03e02fe213f3398992de91f5de1df244770b5ebe/README.md?plain=1#L187-L191 built-in command의 목록에 kill이 누락된 것으로 확인되는데, 그래서 이게 구현해야 하는 범위인지 모르겠습니다.

reference csapsh에서 bg로 돌고 있는 Job에 kill 하니까 동작은 하네요. 다만 설명에 누락된 부분이 있다보니 double check을 위해 문의드립니다.

Jong25 commented 4 months ago

해당 내용은 unix shell에 대한 기본적인 배경을 설명하는 부분으로 csapsh의 구현과는 무관합니다. 예시 command의 prompt가 csapsh로 되어있어 혼동의 여지가 있었던 것 같습니다. 구현은 아래 csapsh specification 부분을 참고하여 진행하시면 됩니다. +) reference에서 kill이 동작한 것은 built-in command가 아닌 kill binary가 실행되었기 때문입니다. which kill command를 확인해보시기 바랍니다.

Those commands are to explain the general unix shell, not our csapsh. You do not have to implement kill command. +) The kill command would be the kill binary, not the built-in command. Check out the which kill.