search

42shpimanmls / 42sh

Shell project at 42
3 stars 0 forks source link

Heredoc #251

Closed M5oul closed 7 years ago

M5oul commented 7 years ago

Functionnal heredoc

Missing features which could be done later opening tickets:

n0izn0iz commented 7 years ago

On peut finir la partie obligatoire avant de faire des extras ?

M5oul commented 7 years ago

Valider le 21sh n’est pas un extra pour valider le 42sh. Bien entendu, je conçois que le 42sh n’est pas fini.

n0izn0iz commented 7 years ago 
[n0iz@nomad ~/Projects/42sh]$ echo << un << deux << trois

enter

>bla

enter

>un

enter

>>
>>>

[n0iz@nomad ~/Projects/42sh]$

VS

sh-4.4$ echo << un << deux << trois

enter

> bli

enter

> un

enter

> bla

enter

> deux

enter

> blou

enter

> trois

enter


sh-4.4$
n0izn0iz commented 7 years ago

Je suis contre cette pr, je veux pas prendre le risque d'introduire des bugs pour un bonus

M5oul commented 7 years ago

On est pas obligé de l’intégrer. On peut patcher le 42sh avec cette PR pour valider le 21sh.

n0izn0iz commented 7 years ago

Mec tu va prendre notre 42 pour ton 21?

n0izn0iz commented 7 years ago

ol:

Vous l'avez dit, les 2 premiers projs sont individuels (en l'état actuel, je mets de coté la remarque sur sa difficulté) et doivent donc être évalués comme tels. Vérifier la triche lors d'une soutenance, c'est normal. Autant le repompage git, que celui qui sait et pense savoir faire alors qu'il sait pas faire. C'est une responsabilité de la communauté entière de jouer ce rôle en soutenance (même si c'est pas toujours "agréable"), et cela dans l'intérêt même de cette communauté entière.

https://forum.intra.42.fr/topics/14804/messages?page=1#50135

M5oul commented 7 years ago

Oui, j’aurais aimé valider mon 21sh. Mais, je n’y arrive pas à cause du fait que des crashs soient découverts par au moins un des cinq correcteurs. Sa conception, qui a été faite sur le tas, est mal pensée. Si je n’avais pas développé le 21sh, je conçois que rendre le 42sh pour un 21sh n’est pas honnête de ma part. Malheureusement, le temps est compté pour que je refactorise mon 21sh étant donné que le trou noir est dans trois semaines. Par conséquent, je suis parti sur l’idée de faire valider notre projet commun pour mon 21sh.

pilespin commented 7 years ago

Ungasebenzisa futhi Valgrind