fix env restored with modifications

[lsimonne]

j'avais cassé setenv/unsetenv ˆˆ bon du coup ça aurait été pas mal de passer l'env aux builtins, là c'est un peu moche...

[n0izn0iz]

P'tite clarification sur ce que c'est sensé faire ? :P

[lsimonne]

restaurer l'env (les variables modifiées par les assignments) en gardant les éventuelles modifications de setenv/unsetenv après l'exécution des builtins

[n0izn0iz]

OK, faut faire gaffe avec l'assignement, quand un builtin est "special" (voir posix) les assignements doivent être fait sur l’environnement courant (rester après)

[n0izn0iz]

Mais donc pas pour setenv unsetenv

[lsimonne]

y'a quoi comme builtins spéciaux? ah tiens j'ai oublié de supprimer les variables assignées non-préexistantes je crois

[n0izn0iz]

http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_14

[n0izn0iz]

break, colon (:), continue, dot (.), eval, exec, exit, export, readonly, return, set, shift, times, trap, unset.

Donc ceux qui nous concernent: exit, export et unset.

[n0izn0iz]

235

[lsimonne]

j'ai fait les modifs ici pour #231 parce que sinon y'aurait eu des conflits chiants, et je pars demain

[pilespin]

les double quote sa passe pas dans les test et accepter #233 sinon les test il sont foireux

[n0izn0iz]

ya aucun problème de double quote ^^

[n0izn0iz]

Voila :D Pls review me !

[pilespin]

faut pas teste env aussi

[pilespin]

42sh: error signal -139 -- pourquoi il y a pas l'erreur segfault

[pilespin]

la il faut -11 pas -139

[n0izn0iz]

• pour env je viens de fix en ajoutant env -i devant le test
• j'ai rien changé a la partie gestion des signaux (que gestion des exit status) dans cette pr

[pilespin]

il y a pas l'erreur segfault