zafraedu / minishell Goto Github PK

El codigo ahora mismo tiene un problema al asignar valor del split de libft, al ejecutar msh->cmd_args = ft_split(parser->cmd, 32) nos da un problema con los espacios por ejemplo al ejecutar:

echo "hola                adios"

nos imprimira los argumentos sin tener todos los espacios en cuenta:

$> "hola adios"

habrá que crear un split() propio para shell o algo equivalente que nos solucione el problema

Nececitamos implementar los pipes y su redireccionamiento correspondiente;

Esto puede conllevar a que hagamos un cambio bruto en el codigo,
tanto en la parte del parser, como la redirecion creada anteriormente para los pipes.

podemos tener una referencia de clean code aqui

Idea para la remodelacion del parser

t_ps	*p_fill_ps(t_lx *lex, t_ps *par)
{
	t_ps	*curr;
	int		fd[2];

	curr = p_new_node(NULL, lex);
	par = curr;
	while (lex)
	{
		p_fill_content(curr, lex);
		if (lex && lex->token == PIPE)
		{
			lex = lex->next;
			pipe(fd);
			if (curr->outfile == 1)
				curr->outfile = fd[1];
			curr = p_new_node(curr, lex);
			curr->infile = fd[0];
		}
		else if (lex)
			lex = lex->next;
		else
			break ;
	}
	return (par);
}

Idea para la remodelacion del ejecutor y implementacion del pipe

void	ft_execute(t_info *info)
{
	pid_t	pid;

	while (info->par)
	{
		if (check_builtin(info->par->cmd) && info->n_cmds == 1)
			ft_builtins(info);
		else
		{
			g_signal_detector = MID_CMD;
			pid = fork();
			if (pid == 0)
			{
				if (check_builtin(info->par->cmd))
					ft_builtins(info);
				else
					exec_cmd(info);
			}
			else
				waitpid(-1, &info->status, 0);
			g_signal_detector = BASE;
			ft_error_msg(info, 0);
		}
		ft_next_cmd(info);
	}
}

La idea es hacer que todas las redirecciones sean ejecutadas por un proceso hijo, para asi no preucuparnos mucho por dup2();

Hacer que las redirecciones de infile y outfile funcione con la builts creadas

• esto no funciona por ejemplo:

echo hola > out

Hay que implementar el here_doc

Hay que arreglar las comillas que se almacenan en msh->cmd_args, por ejemplo usando el comando echo.

asi nos sale:

$> echo "hola          adios"
"hola          adios"

asi nos deberia de salir:

$> echo "hola          adios"
hola          adios

hay que modificar la funcion ft_split_shell()