51874 2024-10-29 17:09:01 +0300 Ошибка запаковки debuginfo если в исходниках есть квадратные скобки 2025-09-12 00:06:27 +0300 1 1 4 Development Sisyphus rpm-build unstable all Linux NEW https://bugzilla.altlinux.org/show_bug.cgi?id=43049 P5 major --- 1 iv placeholder arseny glebfm imz lav ldv placeholder vt qa-sisyphus oldest_to_newest 253600 0 17077 iv 2024-10-29 17:09:01 +0300 Created attachment 17077 соберите этот спек чтобы воспроизвести проблему При сборке одного проекта на go наткнулся на интересную проблему: часть сгенерированных исходников эта штука называет something[generated].go, а наш rpmbuild позже воспринимает квадратные скобки как glob. Проще всего реализовать воспроизвести эту проблему с помощью приложенного spec-файла, который на моей системе не собирается со следующей ошибкой: [...] Creating bracket-test-debuginfo package Processing files: bracket-test-debuginfo-0.1-alt1 error: File not found by glob: /tmp/.private/iv/RPM/TMP/bracket-test-buildroot/usr/src/debug/test[generated].c RPM build errors: File not found by glob: /tmp/.private/iv/RPM/TMP/bracket-test-buildroot/usr/src/debug/test[generated].c 253601 1 iv 2024-10-29 17:10:00 +0300 > часть сгенерированных исходников эта штука называет something[generated].go Оказывается эти исходники довольно сложно переименовать, наверное придётся отключить debuginfo пока это не исправленно. 253620 2 vt 2024-10-30 02:16:05 +0300 Похоже это баг в логике build/files.c:processBinaryFile. Сначала определяется что fileURL это glob, по наличию ? * [ ], если glob, то он обрабатывается как glob (с экранированием символов через \, а если не glob, то обрабатывается как путь (без поддержки экранирования). То есть если добавить экранирование, то путь перестает быть glob и символы экранирования испортят путь. И это должно относиться не только к debuginfo, а к любым файлам. 253621 3 vt 2024-10-30 03:10:47 +0300 Я думаю правильным было бы изначально считать тот путь glob'ом. Но, просто отключить это уже нельзя из-за возможных неочевидных последствий. Апстрим поменял логику в c10e2310e4c41a626b524ae71b3c4f87a29134b2, это огромный коммит. [Ссылающийся на коммит 4030062f2bf16d7a4540c78c1e61706f3810fe9b, которого нет (видимо rebase его потер).] Другой вариант - придумать что-то ALT-specific. 253622 4 vt 2024-10-30 04:30:08 +0300 > Другой вариант - придумать что-то ALT-specific. RFC https://git.altlinux.org/tasks/361121/ 253661 5 glebfm 2024-10-30 15:26:30 +0300 (In reply to Vitaly Chikunov from comment #4) > RFC https://git.altlinux.org/tasks/361121/ В теории, наверное, было бы ещё удобнее если бы это делалось через %attr или что-то такое, чтобы можно было glob выключить и потом включить обратно. Но пока что это, вроде, не нужно. LGTM 253666 6 arseny 2024-10-30 15:40:03 +0300 (In reply to Vitaly Chikunov from comment #4) > > Другой вариант - придумать что-то ALT-specific. > RFC https://git.altlinux.org/tasks/361121/ Вдогонку: есть ли у нас (стоит ли добавить) тест, покрывающий файлы с пробелами — записи в %files, видимо, такого вида: %files --noglob "%_datadir/project/2001 a space odyssey" "%_datadir/project/2001[a space odyssey]" ? А то rpm-s-m, исправляя у себя, столкнулись с поломкой таких файлов. 253668 7 arseny 2024-10-30 15:43:12 +0300 (In reply to Vitaly Chikunov from comment #4) > > Другой вариант - придумать что-то ALT-specific. > > RFC https://git.altlinux.org/tasks/361121/ Для генератных пакетов — LGTM, за исключением соображения выше. (In reply to Gleb F-Malinovskiy from comment #5) > (In reply to Vitaly Chikunov from comment #4) > > RFC https://git.altlinux.org/tasks/361121/ > > В теории, наверное, было бы ещё удобнее если бы это делалось через %attr или > что-то такое, чтобы можно было glob выключить и потом включить обратно. Но > пока что это, вроде, не нужно. Когда секцию пишет человек — да. А роботу-генератору, как я imho себе их представляю, так будет менее удобно: у него получился список готовых путей, и ему нужно их в неизменном виде донести до rpm-build. 253685 8 glebfm 2024-10-30 17:17:22 +0300 (In reply to Vitaly Chikunov from comment #2) > Похоже это баг в логике build/files.c:processBinaryFile. Сначала > определяется что fileURL это glob, по наличию ? * [ ], если glob, то он > обрабатывается как glob (с экранированием символов через \, а если не glob, > то обрабатывается как путь (без поддержки экранирования). То есть если > добавить экранирование, то путь перестает быть glob и символы экранирования > испортят путь. Кстати, в этом случае в качестве объезда можно сделать, чтобы это всегда был glob, а сам путь при этом остался однозначным. Я вспомнил, что мы делаем похожее в сборочнице с git-ом, там вот такое есть: old_tag_id="$(git rev-parse --tags="[${tag_name:0:1}]${tag_name:1}")" 256711 9 vt 2024-12-27 00:19:47 +0300 (In reply to Vitaly Chikunov from comment #4) > > Другой вариант - придумать что-то ALT-specific. > > RFC https://git.altlinux.org/tasks/361121/ Случайно удалил тестовое задание (скриптом). ﷐[U+1F937]﷑ 17077 2024-10-29 17:09:01 +0300 2024-10-29 17:09:01 +0300 соберите этот спек чтобы воспроизвести проблему test.spec text/x-rpm-spec 504 iv Ck5hbWU6ICAgIGJyYWNrZXQtdGVzdApWZXJzaW9uOiAwLjEKUmVsZWFzZTogYWx0MQoKU3VtbWFy eTogVGVzdCBmb3Igc291cmNlIHdpdGggYnJhY2tldHMKTGljZW5zZTogTUlUCkdyb3VwOiAgIE90 aGVyCgpQYWNrYWdlcjogSXZhbiBBLiBNZWxuaWtvdiA8aXZAYWx0bGludXgub3JnPgoKJWRlc2Ny aXB0aW9uCiVzdW1tYXJ5CgolcHJlcAoKY2F0ID4gJ3Rlc3RbZ2VuZXJhdGVkXS5jJyA8PCAnRU9G JwppbnQgbWFpbigpIHsKICByZXR1cm4gMDsKfQpFT0YKCgolYnVpbGQKCmdjYyAlb3B0ZmxhZ3Mg J3Rlc3RbZ2VuZXJhdGVkXS5jJyAtbyB0ZXN0LWdlbmVyYXRlZAoKCiVpbnN0YWxsCmluc3RhbGwg LURtNzU1IHRlc3QtZ2VuZXJhdGVkICVidWlsZHJvb3QlX2JpbmRpci90ZXN0LWdlbmVyYXRlZAoK JWZpbGVzCiVfYmluZGlyLyoKCiVjaGFuZ2Vsb2cKKiBUdWUgT2N0IDI5IDIwMjQgSXZhbiBBLiBN ZWxuaWtvdiA8aXZAYWx0bGludXgub3JnPiAwLjEtYWx0MQotIHRlc3Qgc3R1ZmYK