Opened 5 years ago

Closed 5 years ago

#33 closed ожидается проверка (задача сдана)

WW #1

Reported by: chaykova.anastasiya Owned by: Egor Suvorov
Component: WW_make Version: 3.0
Keywords: Cc:

Description


Change History (10)

comment:1 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to chaykova.anastasiya
Type: ожидается проверкаожидаются исправления

Почти успех, надо немного причесать код и поправить Makefile:

  1. Слишком большие изменение между correct и multiple. Можно на них посмотреть, запустив команду diff -r correct multiple из папки lab_01. Там какой-то код меняется, который можно не менять.
  2. Не хватает одной зависимости у util.o.
  3. Сейчас при повторном запуске make всё пересобирается, потому что есть тонкости с зависимостью lab1: bin. Посмотрите в требования в обновлённом задании, там написано, как с этим разбираться.
  4. И от bin должен зависеть не lab1, а каждый из объектных файлов — это им для успешного выполнения рецепта нужна папка, а не lab1.

Стилистическое:

  1. -c в CFLAGS традиционно не включают. Причина: может хотеться скомпилировать и слинковать одной командой, тогда -c не нужен, а все остальные нужны.

comment:2 Changed 5 years ago by Egor Suvorov

Owner: changed from chaykova.anastasiya to Egor Suvorov
Type: ожидаются исправленияожидается проверка

comment:3 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to chaykova.anastasiya
Type: ожидается проверкаожидаются исправления

Вроде бы лишняя папка lab_01(1) всё ещё есть.

comment:4 Changed 5 years ago by chaykova.anastasiya

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:5 Changed 5 years ago by Egor Suvorov

Owner: changed from chaykova.anastasiya to Egor Suvorov

Пожалуйста, обновляйте также поле Owner (обновил инструкцию) в группе Action/reassign to на suvorov

comment:6 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to chaykova.anastasiya
Type: ожидается проверкаожидаются исправления

Почти успех!

  1. Слишком большие изменения между correct и multiple: изменения в algorithm.c и util.c наверняка не нужны.
  2. Аналогично с correct и unresolved: достаточно просто изменить имя doComputation на helloWorld или hello_world, а код менять не надо.

В остальном окей

Стилистическое:

  1. Флаг -c обычно в CFLAGS не включают, а пишут в каждой команде компиляции отдельно. Это нужно, чтобы можно было, если захочется, скомпилировать и слинковать одной командой с теми же флагами, тогда -c как раз оказывается не нужен.

comment:7 Changed 5 years ago by chaykova.anastasiya

Owner: changed from chaykova.anastasiya to Egor Suvorov
Type: ожидаются исправленияожидается проверка
Version: 2.03.0

comment:8 Changed 5 years ago by Egor Suvorov

Owner: changed from Egor Suvorov to chaykova.anastasiya
Type: ожидается проверкаожидаются исправления

Почти успех: всё ещё слишком большие изменения между папками: лишние пробелы/переводы строк. Избавьтесь от них, пожалуйста. Проверьте, что всё чисто, запустив команды diff -r correct multiple и diff -r correct unresolved.

Есть ещё попытки до среды.

comment:9 Changed 5 years ago by chaykova.anastasiya

Owner: changed from chaykova.anastasiya to Egor Suvorov
Type: ожидаются исправленияожидается проверка

comment:10 Changed 5 years ago by Egor Suvorov

Resolution: задача сдана
Status: assignedclosed

Успех!

Note: See TracTickets for help on using tickets.