fixed exit issue
This commit is contained in:
23
srcs/exec.c
23
srcs/exec.c
@ -6,7 +6,7 @@
|
||||
/* By: marde-vr <marde-vr@42angouleme.fr> +#+ +:+ +#+ */
|
||||
/* +#+#+#+#+#+ +#+ */
|
||||
/* Created: 2024/02/07 14:12:49 by tomoron #+# #+# */
|
||||
/* Updated: 2024/02/22 17:58:49 by marde-vr ### ########.fr */
|
||||
/* Updated: 2024/02/23 14:12:21 by marde-vr ### ########.fr */
|
||||
/* */
|
||||
/* ************************************************************************** */
|
||||
|
||||
@ -31,6 +31,11 @@ int cmd_is_builtin(t_msh *msh, char *cmd_token)
|
||||
alias(msh);
|
||||
return (1);
|
||||
}
|
||||
else if (!ft_strcmp(cmd_token, "exit"))
|
||||
{
|
||||
exit_bt(msh);
|
||||
return (1);
|
||||
}
|
||||
else if (!ft_strcmp(cmd_token, "echo") || !ft_strcmp(cmd_token, "ret")
|
||||
|| !ft_strcmp(cmd_token, "env") || !ft_strcmp(cmd_token, "exit")
|
||||
|| !ft_strcmp(cmd_token, "pwd") || !ft_strcmp(cmd_token, "export"))
|
||||
@ -130,6 +135,7 @@ void pipe_child(t_msh *msh, char **cmd_args)
|
||||
if (msh->cmds->token && (!ft_strcmp(msh->cmds->token, "cd")
|
||||
|| !ft_strcmp(msh->cmds->token, "alias")
|
||||
|| !ft_strcmp(msh->cmds->token, "unalias")
|
||||
|| !ft_strcmp(msh->cmds->token, "exit")
|
||||
|| exec_builtin(msh)))
|
||||
ft_exit(msh, 1);
|
||||
if (msh->cmds->token)
|
||||
@ -143,10 +149,6 @@ void pipe_child(t_msh *msh, char **cmd_args)
|
||||
void pipe_parent(t_msh *msh)
|
||||
{
|
||||
(void)msh;
|
||||
if (msh->type_in != ARG)
|
||||
close(msh->fds[0]);
|
||||
if (msh->type_out != ARG)
|
||||
close(msh->fds[1]);
|
||||
}
|
||||
|
||||
int exec(t_msh *msh, t_cmd *cmd, char **cmd_args)
|
||||
@ -159,6 +161,7 @@ int exec(t_msh *msh, t_cmd *cmd, char **cmd_args)
|
||||
if (pipe(msh->fds) == -1)
|
||||
{
|
||||
perror("pipe");
|
||||
ft_printf_fd(2, "exiting\n");
|
||||
ft_exit(msh, 1);
|
||||
}
|
||||
}
|
||||
@ -166,6 +169,7 @@ int exec(t_msh *msh, t_cmd *cmd, char **cmd_args)
|
||||
if (pid == -1)
|
||||
{
|
||||
perror("fork");
|
||||
ft_printf_fd(2, "exiting\n");
|
||||
ft_exit(msh, 1);
|
||||
}
|
||||
if (pid == 0)
|
||||
@ -173,9 +177,8 @@ int exec(t_msh *msh, t_cmd *cmd, char **cmd_args)
|
||||
else
|
||||
{
|
||||
pipe_parent(msh);
|
||||
rl_redisplay();
|
||||
//if (waitpid(pid, 0, 0) < 0)
|
||||
// ft_exit(msh, 1);
|
||||
//rl_redisplay();
|
||||
//waitpid(pid, 0, 0);
|
||||
i = 0;
|
||||
while (msh->pids[i])
|
||||
i++;
|
||||
@ -255,7 +258,6 @@ void remove_command_from_msh(t_msh *msh)
|
||||
void exec_command(t_msh *msh)
|
||||
{
|
||||
int cmd_count;
|
||||
enum e_token_type type;
|
||||
int i;
|
||||
|
||||
i = 0;
|
||||
@ -282,7 +284,10 @@ void exec_command(t_msh *msh)
|
||||
while (i > 0)
|
||||
{
|
||||
if (waitpid(msh->pids[i], 0, 0) < 0)
|
||||
{
|
||||
ft_printf_fd(2, "exiting\n");
|
||||
ft_exit(msh, 1);
|
||||
}
|
||||
i--;
|
||||
}
|
||||
}
|
||||
|
@ -6,7 +6,7 @@
|
||||
/* By: marde-vr <marde-vr@42angouleme.fr> +#+ +:+ +#+ */
|
||||
/* +#+#+#+#+#+ +#+ */
|
||||
/* Created: 2024/02/02 21:59:20 by tomoron #+# #+# */
|
||||
/* Updated: 2024/02/22 17:53:20 by marde-vr ### ########.fr */
|
||||
/* Updated: 2024/02/23 14:06:24 by marde-vr ### ########.fr */
|
||||
/* */
|
||||
/* ************************************************************************** */
|
||||
|
||||
@ -88,7 +88,7 @@ int main(int argc, char **argv, char **envp)
|
||||
|
||||
command = (char *)1;
|
||||
init_minishell(&msh, argc, argv, envp);
|
||||
handle_minishellrc(msh);
|
||||
//handle_minishellrc(msh);
|
||||
while (msh->env && command)
|
||||
{
|
||||
prompt = get_prompt(msh->env);
|
||||
@ -98,9 +98,9 @@ int main(int argc, char **argv, char **envp)
|
||||
free(prompt);
|
||||
add_history(command);
|
||||
msh->cmds = parse_command(command, msh->env);
|
||||
free(command);
|
||||
//print_parsed_cmd(parsed_cmd);//debug
|
||||
msh->cmds = handle_alias(msh);
|
||||
free(command);
|
||||
//print_parsed_cmd(parsed_cmd);//debug
|
||||
exec_command(msh);
|
||||
free_cmd(msh->cmds);
|
||||
|
Reference in New Issue
Block a user