42shpimanmls / 42sh

Shell project at 42
3 stars 0 forks source link

fix - execute_file ignored returned status #103

Closed n0izn0iz closed 7 years ago

n0izn0iz commented 7 years ago

-> #98 before

[n0iz@arch-vm ~/Projects/42sh]$ false && echo bug
bug
[n0iz@arch-vm ~/Projects/42sh]$ 

after

[n0iz@arch-vm ~/Projects/42sh]$ false && echo bug
[n0iz@arch-vm ~/Projects/42sh]$ 
lsimonne commented 7 years ago
sh-3.2$ true && echo PASS && false && echo FAIL
PASS
[ChezLouise@ ~/Documents/42/42sh]$ true && echo PASS && false && echo FAIL
PASS
FAIL
n0izn0iz commented 7 years ago

c'est le problème réglé par #104 quand il y a plus d'un && ou ||

lsimonne commented 7 years ago

ah ok, pardon, je l'ai testé dans les deux et ça faisait la même chose mais j'pensais que t'étais parti de cette branche pour faire l'autre - mal réveillée ˆˆ

n0izn0iz commented 7 years ago

J'ai fait les deux en même temps mais je me suis dit que c'était mieux de faire 2 branches vu que c'est 2 pbm différent mais apparemment non haha ^^