42shpimanmls / 42sh

Shell project at 42
3 stars 0 forks source link

Command substitution #222

Closed lsimonne closed 7 years ago

lsimonne commented 7 years ago
[ChezLouise@iMac-de-Louise ~/Documents/42/42sh]$ echo `ls\``
>42sh: ls`: no such command.
[ChezLouise@iMac-de-Louise ~/Documents/42/42sh]$ echo $?
0
[ChezLouise@iMac-de-Louise ~/Documents/42/42sh]$ `toto`
>42sh: toto: no such command.
[ChezLouise@iMac-de-Louise ~/Documents/42/42sh]$ echo $?
127
lsimonne commented 7 years ago

my bad, problème de communication - du coup on peut quand même faire une issue pour que la substitution se fasse avec les builtins aussi - whoever is assigned will do it this time :)

n0izn0iz commented 7 years ago

nonon je me suis trompé, ce execute() appele juste42sh -c "la substitution" donc pas de builtin a gérer, je suis parti sur mars -_-

lsimonne commented 7 years ago

aha effectivement

n0izn0iz commented 7 years ago

J'ai peur d'avoir fait une connerie avec ce merge :/

lsimonne commented 7 years ago

ça m'a l'air ok, il faut juste rechanger split_substitutions()

n0izn0iz commented 7 years ago

Je te laisse fix et merge la pr, t'as l'air de savoir bien mieux que moi ce que tu fait sur ces fichiers haha