Bug 37407
| Summary: | [3.5] join zah@ | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Team Accounts | Reporter: | zah <zah> | ||||||
| Component: | join | Assignee: | Gleb F-Malinovskiy <glebfm> | ||||||
| Status: | ASSIGNED --- | QA Contact: | Andrey Cherepanov <cas> | ||||||
| Severity: | normal | ||||||||
| Priority: | P3 | CC: | glebfm, grenka, ldv, mike | ||||||
| Version: | unspecified | ||||||||
| Hardware: | all | ||||||||
| OS: | Linux | ||||||||
| URL: | https://altlinux.org/Team/Join | ||||||||
| Attachments: |
|
||||||||
Created attachment 8355 [details]
gpg pub key
Подтверждаю менторство. Прошу завести учётку в git.alt для продолжения обучения. Прошу завести учётку на git.alt. (В ответ на комментарий №0)
> Почта - zah@altlinux.org
А куда пересылать письма?
(В ответ на комментарий №5) > (В ответ на комментарий №0) > > Почта - zah@altlinux.org > > А куда пересылать письма? zah@ivk.ru (В ответ на комментарий №0) > Created an attachment (id=8354) (В ответ на комментарий №1) > Created an attachment (id=8355) (В ответ на комментарий №6) Ok. ssh ключ на gitery.alt зарегистрирован. ssh ключ на gyle.alt зарегистрирован. Адрес для пересылки создан. T/J/S -> 3.0. (Ответ для Gleb F-Malinovskiy на комментарий #8) > ssh ключ на gitery.alt зарегистрирован. > ssh ключ на gyle.alt зарегистрирован. > Адрес для пересылки создан. > > T/J/S -> 3.0. zah@ утверждает, что не может собирать в сборочницу. Выложите, пожалуйста, повторно. (Ответ для Andrey Cherepanov на комментарий #9) > (Ответ для Gleb F-Malinovskiy на комментарий #8) > > ssh ключ на gitery.alt зарегистрирован. > > ssh ключ на gyle.alt зарегистрирован. > > Адрес для пересылки создан. > > > > T/J/S -> 3.0. > > zah@ утверждает, что не может собирать в сборочницу. Это ожидаемый эффект на стадии 3.0. Сборочница начинает принимать подписи кандидата после выполнения пункта 3.3. Подтверждаю, что кандидат научился писать спеки и готовить пакеты. Прошу перейти к следующему этапу. Подтверждаю, что кандидат научился писать спеки и готовить пакеты. Прошу перейти к следующему этапу. Пакет alt-gpgkeys обновлён. T/J/S -> 4.0. (Ответ для Gleb F-Malinovskiy на комментарий #13) > Пакет alt-gpgkeys обновлён. > > T/J/S -> 4.0. Кандидат показал, что готов собирать пакеты уже в репозиторий. Прошу перейти на следующий этап. (In reply to Andrey Cherepanov from comment #14) > (Ответ для Gleb F-Malinovskiy на комментарий #13) > > Пакет alt-gpgkeys обновлён. > > > > T/J/S -> 4.0. > > Кандидат показал, что готов собирать пакеты уже в репозиторий. Прошу перейти > на следующий этап. Готов собирать пакеты в репозиторий - это хорошо, но хотелось бы прояснить несколько вопросов по тестовому заданию. spec-файл назван wavemon-0.9.2.spec, это странно, обычно версию пакета в имени spec-файла не указывают. Несмотря на то, что в wavemon/README.md написано, что "wavemon is distributed under the [GPLv3](http://www.gnu.org/licenses/gpl-3.0.en.html), refer to the file `COPYING`.", в самих исходниках все copyright headers говорят про GPLv2+. > %configure LDFLAGS="-Wl,--copy-dt-needed-entries" Зачем понадобился этот костыль, интересно? > %find_lang --all %name Зачем это делать, если результат не используется? > spec-файл назван wavemon-0.9.2.spec, это странно, обычно версию пакета в > имени spec-файла не указывают. Исправил. > Несмотря на то, что в wavemon/README.md написано, что > "wavemon is distributed under the > [GPLv3](http://www.gnu.org/licenses/gpl-3.0.en.html), refer to the file > `COPYING`.", > в самих исходниках все copyright headers говорят про GPLv2+. Поле License мне заполнил скрипт github2spec. > > %configure LDFLAGS="-Wl,--copy-dt-needed-entries" > Зачем понадобился этот костыль, интересно? Возникает ошибка при компоновке: /usr/bin/ld.default: info_scr.o: undefined reference to symbol 'pthread_sigmask@@GLIBC_2.2.5' /usr/bin/ld.default: /lib64/libpthread.so.0: error adding symbols: DSO missing from command line collect2: error: ld returned 1 exit status make: *** [<builtin>: wavemon] Error 1 > > %find_lang --all %name > Зачем это делать, если результат не используется? Убрал (In reply to zah from comment #16) > > spec-файл назван wavemon-0.9.2.spec, это странно, обычно версию пакета в > > имени spec-файла не указывают. > Исправил. Спасибо. > > Несмотря на то, что в wavemon/README.md написано, что > > "wavemon is distributed under the > > [GPLv3](http://www.gnu.org/licenses/gpl-3.0.en.html), refer to the file > > `COPYING`.", > > в самих исходниках все copyright headers говорят про GPLv2+. > Поле License мне заполнил скрипт github2spec. На мой взгляд, в wavemon/README.md ошибка, поскольку информация в самих исходниках приоритетнее. Если этот проект ведётся на гитхабе, не хотите открыть там issue на эту тему? > > > %configure LDFLAGS="-Wl,--copy-dt-needed-entries" > > Зачем понадобился этот костыль, интересно? > Возникает ошибка при компоновке: > > /usr/bin/ld.default: info_scr.o: undefined reference to symbol > 'pthread_sigmask@@GLIBC_2.2.5' > /usr/bin/ld.default: /lib64/libpthread.so.0: error adding symbols: DSO > missing from command line > collect2: error: ld returned 1 exit status > make: *** [<builtin>: wavemon] Error 1 Это верный признак недолинковки: приложение использует pthread, но не слинковано с -pthread. Вот цитата из лога сборки: cc -pipe -frecord-gcc-switches -Wall -g -O2 -I /usr/include/libnl3 -Wl,--copy-dt-needed-entries wavemon.c about_scr.o conf.o conf_scr.o error.o help_scr.o info_scr.o iw_if.o iw_nl80211.o iw_scan.o lhist_scr.o llist.o scan_scr.o ui.o utils.o -lcap -lncursesw -lm -lnl-genl-3 -lnl-3 -lnl-3 -o wavemon Причина - неправильный тест в configure.ac: $ git grep FLAGS wavemon/configure.ac wavemon/configure.ac:CFLAGS="-O2 -Wall" wavemon/configure.ac:AC_CHECK_LIB([pthread], [pthread_create], [CFLAGS="$CFLAGS -pthread"], wavemon/configure.ac: [CFLAGS="$CFLAGS $LIBNL3_CFLAGS"], Менять CFLAGS в configure.ac - это распространённая ошибка, которую следует исправить. Вот пример исправления из другого пакета: https://kernel.googlesource.com/pub/scm/utils/quota/quota-tools/+/f2eb78c3833a123101c8c89c45f71dd4c7cd4787%5E!/ > > > %find_lang --all %name > > Зачем это делать, если результат не используется? > Убрал Спасибо. (In reply to Andrey Cherepanov from comment #2) > Подтверждаю менторство. Андрей, если ты не возражаешь, я тоже немножко поменторю тут. (Ответ для Dmitry V. Levin на комментарий #18) > (In reply to Andrey Cherepanov from comment #2) > > Подтверждаю менторство. > > Андрей, если ты не возражаешь, я тоже немножко поменторю тут. Да, конечно! > На мой взгляд, в wavemon/README.md ошибка, поскольку информация в самих > исходниках приоритетнее. Если этот проект ведётся на гитхабе, не хотите > открыть там issue на эту тему? Issue открыл https://github.com/uoaerg/wavemon/issues/81 > > /usr/bin/ld.default: info_scr.o: undefined reference to symbol > > 'pthread_sigmask@@GLIBC_2.2.5' > > /usr/bin/ld.default: /lib64/libpthread.so.0: error adding symbols: DSO > > missing from command line > > collect2: error: ld returned 1 exit status > > make: *** [<builtin>: wavemon] Error 1 > > Это верный признак недолинковки: приложение использует pthread, но не > слинковано с -pthread. Вот цитата из лога сборки: > cc -pipe -frecord-gcc-switches -Wall -g -O2 -I /usr/include/libnl3 > -Wl,--copy-dt-needed-entries wavemon.c about_scr.o conf.o conf_scr.o > error.o help_scr.o info_scr.o iw_if.o iw_nl80211.o iw_scan.o lhist_scr.o > llist.o scan_scr.o ui.o utils.o -lcap -lncursesw -lm -lnl-genl-3 -lnl-3 > -lnl-3 -o wavemon > > Причина - неправильный тест в configure.ac: > $ git grep FLAGS wavemon/configure.ac > wavemon/configure.ac:CFLAGS="-O2 -Wall" > wavemon/configure.ac:AC_CHECK_LIB([pthread], [pthread_create], > [CFLAGS="$CFLAGS -pthread"], > wavemon/configure.ac: [CFLAGS="$CFLAGS $LIBNL3_CFLAGS"], У меня получилось вот так вот: 11c11 < #CFLAGS="-O2 -Wall" --- > CFLAGS="-O2 -Wall" 58c58 < AC_CHECK_LIB([pthread], [pthread_create], [PTHREAD_CFLAGS="$CFLAGS -pthread"], --- > AC_CHECK_LIB([pthread], [pthread_create], [CFLAGS="$CFLAGS -pthread"], 77c77 < [LIBNL3_CFLAGS="$PTHREAD_CFLAGS"], --- > [CFLAGS="$CFLAGS $LIBNL3_CFLAGS"], НО без изменений. В целом я понял что при компановке не хватает -pthread. Мне нужно ещё что-то добавить или правильно переменные определить? (In reply to zah from comment #20) > > На мой взгляд, в wavemon/README.md ошибка, поскольку информация в самих > > исходниках приоритетнее. Если этот проект ведётся на гитхабе, не хотите > > открыть там issue на эту тему? > Issue открыл > https://github.com/uoaerg/wavemon/issues/81 Спасибо. > > > /usr/bin/ld.default: info_scr.o: undefined reference to symbol > > > 'pthread_sigmask@@GLIBC_2.2.5' > > > /usr/bin/ld.default: /lib64/libpthread.so.0: error adding symbols: DSO > > > missing from command line > > > collect2: error: ld returned 1 exit status > > > make: *** [<builtin>: wavemon] Error 1 > > > > Это верный признак недолинковки: приложение использует pthread, но не > > слинковано с -pthread. Вот цитата из лога сборки: > > cc -pipe -frecord-gcc-switches -Wall -g -O2 -I /usr/include/libnl3 > > -Wl,--copy-dt-needed-entries wavemon.c about_scr.o conf.o conf_scr.o > > error.o help_scr.o info_scr.o iw_if.o iw_nl80211.o iw_scan.o lhist_scr.o > > llist.o scan_scr.o ui.o utils.o -lcap -lncursesw -lm -lnl-genl-3 -lnl-3 > > -lnl-3 -o wavemon > > > > Причина - неправильный тест в configure.ac: > > $ git grep FLAGS wavemon/configure.ac > > wavemon/configure.ac:CFLAGS="-O2 -Wall" > > wavemon/configure.ac:AC_CHECK_LIB([pthread], [pthread_create], > > [CFLAGS="$CFLAGS -pthread"], > > wavemon/configure.ac: [CFLAGS="$CFLAGS $LIBNL3_CFLAGS"], > У меня получилось вот так вот: На будущее: всегда и везде присылайте патчи только в формате unified diff (diff -u). Кстати, git так делает по-умолчанию, так что git diff вам в помощь. > 11c11 > < #CFLAGS="-O2 -Wall" > --- > > CFLAGS="-O2 -Wall" > 58c58 > < AC_CHECK_LIB([pthread], [pthread_create], [PTHREAD_CFLAGS="$CFLAGS > -pthread"], > --- > > AC_CHECK_LIB([pthread], [pthread_create], [CFLAGS="$CFLAGS -pthread"], Недостаточно поменять переменные, их ещё надо AC_SUBST'ить и потом использовать в Makefile.in; LIBNL3_CFLAGS не надо AC_SUBST'ить вручную, это уже сделал PKG_CHECK_MODULES, а вот PTHREAD_CFLAGS надо. Когда Makefile.in будет исправлен на тему использования LIBNL3_CFLAGS и PTHREAD_CFLAGS, другой костыль (%add_optflags -I %_includedir/libnl3) тоже больше не понадобится. Кстати, -pthread нужен не cтолько для компиляции, сколько для линковки, он должен попасть туда, куда попадает LIBNL3_LIBS. > На будущее: всегда и везде присылайте патчи только в формате unified diff > (diff -u). Кстати, git так делает по-умолчанию, так что git diff вам в > помощь. Понял > Когда Makefile.in будет исправлен на тему использования LIBNL3_CFLAGS и > PTHREAD_CFLAGS, другой костыль (%add_optflags -I %_includedir/libnl3) тоже > больше не понадобится. Сделал AC_SUBST(PTHREAD_CFLAGS) в configure.ac и использовал в Makefile.in, LIBNL3_CFLAGS уже был в Makefile.in. Теперь сборка и линковка проходит успешно. Но при сборке в hsh без костыля (%add_optflags -I %_includedir/libnl3) выпадает ошибка: + make -j8 make: Entering directory '/usr/src/RPM/BUILD/wavemon-0.9.2' CC conf.c conf.c:23:10: fatal error: netlink/version.h: No such file or directory #include <netlink/version.h> (In reply to zah from comment #22) > Сделал AC_SUBST(PTHREAD_CFLAGS) в configure.ac и использовал в Makefile.in, > LIBNL3_CFLAGS уже был в Makefile.in. Теперь сборка и линковка проходит > успешно. > > Но при сборке в hsh без костыля (%add_optflags -I %_includedir/libnl3) > выпадает ошибка: > + make -j8 > make: Entering directory '/usr/src/RPM/BUILD/wavemon-0.9.2' > CC conf.c > conf.c:23:10: fatal error: netlink/version.h: No such file or directory > #include <netlink/version.h> В апстримном Makefile.in написано: CFLAGS ?= @CFLAGS@ @LIBNL3_CFLAGS@ ... %.o: %.c $(HEADERS) @echo ' CC ' $< @$(CC) $(CFLAGS) $(CPPFLAGS) $(LDFLAGS) $(DEFS) -c -o $@ $< В результате получается, что CFLAGS, неявно определённый в спек-файле в результате использования %configure, перекрывает апстримное определение CFLAGS. Не хотел бы повторяться, но менять CFLAGS в configure.ac/Makefile.in - это распространённая ошибка, которую следует исправить. (Ответ для Dmitry V. Levin на комментарий #23) > В результате получается, что CFLAGS, неявно определённый в спек-файле в > результате использования %configure, перекрывает апстримное определение > CFLAGS. > Не хотел бы повторяться, но менять CFLAGS в configure.ac/Makefile.in - это > распространённая ошибка, которую следует исправить. Исправил CFLAGS и оформил изменения в patch. p.s. Пока ковырялся в мэйнстриме поправили и закрыли issue на тему лицензии. Подтверждаю готовность кандидата к сборке в репозиторий. (Ответ для Andrey Cherepanov на комментарий #25) > Подтверждаю готовность кандидата к сборке в репозиторий. Повторяю: Подтверждаю готовность кандидата к сборке в репозиторий. Прошу перейти к следующему этапу. (In reply to Andrey Cherepanov from comment #26) > (Ответ для Andrey Cherepanov на комментарий #25) > > Подтверждаю готовность кандидата к сборке в репозиторий. > > Повторяю: Подтверждаю готовность кандидата к сборке в репозиторий. > > Прошу перейти к следующему этапу. С тех пор правила поменялись. К сожалению, я выпал из контекста того единственного тестового пакета, о котором шла речь. Возможно, я ещё к нему вернусь, а пока предлагаю дать какое-нибудь ещё тестовое задание. Преветствую всех! Прошу поменять почтовый адрес для пересылке писем на zakharchenko.sn@gmail.com Спасибо! (In reply to zah from comment #28) > zakharchenko.sn@gmail.com Так, это было сделано, больше ничего не помню. В целом актуально вообще? (In reply to Gleb F-Malinovskiy from comment #29) > (In reply to zah from comment #28) > > zakharchenko.sn@gmail.com > Так, это было сделано, больше ничего не помню. > > В целом актуально вообще? Актуально (In reply to zah from comment #30) > Актуально Тогда нам нужно понять, в каком состоянии мы находимся и каковы планы. Мне кажется, что мы находимся примерно на стадии T/J/S 3.5, если что поправьте. Андрей? ping? (In reply to Gleb F-Malinovskiy from comment #31) > (In reply to zah from comment #30) > > Актуально > Тогда нам нужно понять, в каком состоянии мы находимся и каковы планы. Мне > кажется, что мы находимся примерно на стадии T/J/S 3.5, если что поправьте. > > Андрей? Готов продолжить с любого пункта, с которого посчитаете необходимым. Могу выступить в роли ментора, если у cas@ нет возможности продолжать джойн. (In reply to Grigory Ustinov from comment #34) > Могу выступить в роли ментора, если у cas@ нет возможности продолжать джойн. Григорий, предложение ещё в силе? Я бы хотел пройти джойн с нуля, поскольку давно не практиковал. Да. (In reply to Grigory Ustinov from comment #36) > Да. Где будем общение вести? Если удобно ответь на почту. |
Description
zah 2019-11-01 12:32:48 MSK