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

환경변수 파싱 #5

Closed
S0YKIM opened this issue Apr 4, 2022 · 1 comment · Fixed by #8
Closed

환경변수 파싱 #5

S0YKIM opened this issue Apr 4, 2022 · 1 comment · Fixed by #8
Assignees
@hhkim0729
Labels
feature ⭐ New feature added

Comments

@S0YKIM
Copy link
Owner

S0YKIM commented Apr 4, 2022

main 함수의 세 번째 인자로 받는 char **envp 를 key-value 형태로 파싱
@S0YKIM S0YKIM added the feature ⭐ New feature added label Apr 4, 2022
hhkim0729 added a commit that referenced this issue Apr 5, 2022
@hhkim0729
Add codes for env parsing (#5)
a721618
@hhkim0729 hhkim0729 linked a pull request Apr 5, 2022 that will close this issue
환경변수 파싱 #8
Merged
@S0YKIM
Copy link
Owner Author

S0YKIM commented Apr 10, 2022

환경변수 리스트 저장하실 때에 마지막에 $?=0 도 추가해주세요! bash 쉘을 켜자마자 echo $? 명령어를 실행하면 0 이 출력되는데, 이를 위해서 필요합니다. env 폴더 내의 update_env() 함수를 이용해 update_env(data, "?", 0); 만 끝에 한 줄 추가해주시면 됩니다~

@S0YKIM S0YKIM mentioned this issue Apr 10, 2022
Closed
hhkim0729 added a commit that referenced this issue Apr 11, 2022
@hhkim0729
Add free_env_list & error handling code (#5)
b1c3fbf
hhkim0729 added a commit that referenced this issue Apr 11, 2022
@hhkim0729
Merge pull request #31 from S0YKIM/heehkim
0fcb537 
Add free_env_list & error handling code (#5)
hhkim0729 added a commit that referenced this issue Apr 13, 2022
@hhkim0729
Remove add_env_node & replace with update_env (#5)
7585e1f
hhkim0729 added a commit that referenced this issue Apr 13, 2022
@hhkim0729
Add add_prev_exit_env(0) (#5 #30)
6c60a5f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hhkim0729 hhkim0729

Labels
feature ⭐ New feature added
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

환경변수 파싱 S0YKIM/42-MINISHELL
2 participants
@hhkim0729 @S0YKIM