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

[Refactor] main함수 리팩토링 #59

Closed
wonyangs opened this issue Jan 15, 2023 · 0 comments · Fixed by #67
Closed

[Refactor] main함수 리팩토링 #59

wonyangs opened this issue Jan 15, 2023 · 0 comments · Fixed by #67
Assignees
Labels
💻 Implement 기능 구현 🛠 Refactor 코드 리팩토링

Comments

@wonyangs
Copy link
Collaborator

🚅 Issue 한 줄 요약

main 함수 리팩토링, minishell 실행 시 기본 설정 추가

🤷 Issue 세부 내용

  • minishell 프로그램 실행 시 인자가 들어오면 에러로 처리
  • 현재 모든 로직이 모여있는 main 함수를 파트에 맞게 분리
  • 이곳 저곳에 배치되어 있는 시그널, 터미널 세팅 정리
  • minishell 실행 시 환경변수 초기화
    • SHLVL 환경변수는 기존 쉘보다 1 증가해야함
      (shell level : 쉘 위에 쉘이 몇 개 실행되었는지 나타내는 환경변수)
    • OLDPWD 환경변수는 기존 환경변수와 상관없이 NULL로 설정되어 있음
    • PWD 환경변수는 기존 환경변수와 상관없이 현재 디렉토리 위치로 설정되어 있음
  • norminette에 맞도록 기존 코드 수정

📸 스크린샷

스크린샷 2023-01-15 오후 12 19 00

@wonyangs wonyangs added 🛠 Refactor 코드 리팩토링 💻 Implement 기능 구현 labels Jan 15, 2023
@wonyangs wonyangs self-assigned this Jan 15, 2023
@mingxoxo mingxoxo moved this to 📋 To-do in Minishell backlog Jan 15, 2023
wonyangs added a commit that referenced this issue Jan 15, 2023
@wonyangs wonyangs linked a pull request Jan 15, 2023 that will close this issue
mingxoxo added a commit that referenced this issue Jan 15, 2023
@github-project-automation github-project-automation bot moved this from 📋 To-do to ✅ Done in Minishell backlog Jan 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 Implement 기능 구현 🛠 Refactor 코드 리팩토링
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant