Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Esqueleto del TP2 #9

Open
wants to merge 55 commits into
base: tp2
Choose a base branch
from
Open

Esqueleto del TP2 #9

wants to merge 55 commits into from

Conversation

sjdalessandro
Copy link

@sjdalessandro sjdalessandro commented Apr 3, 2020

Si bien la base de este PR es una copia de la rama tp1, hacer merge de tp2-new sobre la solución del TP1 genera muchos conflictos, creo que es mejor dejar los TPs separados.

Lo probé con mi solución y pasa las pruebas.


This change is Reviewable

anishathalye and others added 30 commits October 2, 2019 15:08
The defaults work fine now, since
3cbbeb0.
This is necessary because gcc may generate calls to memcmp, memset,
memcpy, and memmove when compiling with -nostdlib.
its system calls), since we will not ask students to do the mount lab.
joshuafried and others added 25 commits April 3, 2020 02:26
El parámetro terminate_match permite indicar a los tests las condiciones
en las que se debería abortar el test: por ejemplo, una llamada a panic().
Con esto, se consigue no tener que tener que esperar al timeout si ello
sucede.

En el commit se incluye:

  - una expresión regular específica para alloctest

  - una expresión regular específica para usertests, definida en gradelib.py
    para poder ser usada en futuros labs

Nota: la llamada a re.match() en gradelib se cambia a re.search() para evitar
tener que especificar ".*" innecesariamente; y se añade el flag re.MULTILINE
para aquellos matchers con anclaje.
See: 6ff9f94 ("Perhaps a helpful hint when a student
sees a kernel panic").
@sjdalessandro sjdalessandro requested a review from dato April 3, 2020 07:39
@dato dato removed their request for review November 21, 2021 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants