42shpimanmls / 42sh

Shell project at 42
3 stars 0 forks source link

Fix redirection parsing #115

Closed n0izn0iz closed 7 years ago

n0izn0iz commented 7 years ago

[EDIT] pas #114 Fix au cas ou il y a un token non valide après un operateur de redirection

pilespin commented 7 years ago

not work. j'ai merge master dans fix_redir_parsing j'ai bien le commit f5702fc8f70a460ada8525893d2873152819b292 j'ai la meme erreur

n0izn0iz commented 7 years ago

tu a 'make re' ? quand je checkout, pull, merge master puis make re j'ai

 n0iz@nomad  ~/Projects/42sh   fix_redir_parsing  ./42sh 
[n0iz@nomad ~/Projects/42sh]$ cat <1<0
42sh: syntax error: near unexpected token
[n0iz@nomad ~/Projects/42sh]$ 
pilespin commented 7 years ago

ta /dev/zero > l j'utilise l pas 1 mais je pense pas que sa change

lsimonne commented 7 years ago

bizarre tout ça, il doit faire une redirection de 0 on dirait, puisque c'est le tcgetattr sur 0 qui fail, avec comme erreur "Not a typewriter" ˆˆ au passage, il me semble qu'on a pas le droit d'utiliser getenv (man 3, a priori pas autorisé dans minishell)

n0izn0iz commented 7 years ago

ça change rien, <1<0 c'est une erreur de syntaxe puisque le '1' est un nombre juste avant une redirection (le 2e <) donc pas un mot valide après une redirection (le 1er <)

WOOPS

lsimonne commented 7 years ago
sh-3.2$ cat < l < 0
sh: 0: No such file or directory
sh-3.2$
n0izn0iz commented 7 years ago

Bon, grosse incomprehension, la je fait un fix au cas ou il y a <1<0 (avec un 'un') mais du coup pas de rapport avec le truc du TERM

n0izn0iz commented 7 years ago

Je crois que le truc du TERM est normal enfait, puisque 0 est écrasé

lsimonne commented 7 years ago

alors en fait il faut juste reset les backup stdin_out s'il y a une erreur!

n0izn0iz commented 7 years ago

arg ça peut etre lié a un truc que j'ai fix dans la branche ou je bosse. execute_file n'exit pas si il y a une erreur avec execvp dans le child, ça peut fouttre un sacré merdier puisque qu'on a 2 shell qui tournent en même temps après

lsimonne commented 7 years ago

119

n0izn0iz commented 7 years ago

Review ? Voir le premier comment @pilespin @lsimonne @M5oul