53250 2025-02-28 00:07:24 +0300 У brp-debuginfo проблемы с пробелами 2025-06-14 21:15:28 +0300 1 1 4 Development Sisyphus rpm-build unstable x86_64 Linux CLOSED FIXED P5 normal --- 1 iv placeholder arseny glebfm imz ldv placeholder vt qa-sisyphus oldest_to_newest 260066 0 17840 iv 2025-02-28 00:07:24 +0300 Created attachment 17840 соберите этот спек чтобы воспроизвести проблему Так получилось, что мне очень понадобилось собрать библиотеку с пробелом в имени. Так я узнал, что у нас brp-debuginfo вообще не способен справиться с бинарниками с пробельными символами в именах. Для воспроизведения ошибки проще всего попробовать запаковать тривиальную программу (самодостаточный спек в приложении). Содержательная чать buildroot получается такой: $ find /tmp/.private/iv/spaces-buildroot/usr/ /tmp/.private/iv/spaces-buildroot/usr/ /tmp/.private/iv/spaces-buildroot/usr/bin /tmp/.private/iv/spaces-buildroot/usr/bin/space test То есть, пакуется исполняемый файл "/usr/bin/space test". Ошибка выглядит так: date: ./usr/bin/space: No such file or directory date: test: No such file or directory error: Bad exit status from /tmp/.private/iv/rpm-tmp.35317 (%install) 260067 1 iv 2025-02-28 00:10:29 +0300 Пока писал репорт заметил, что у одного xargs там есть `--delimeter='\n'`, а у другого нет. Пойду попробую сделать патч. Конечно, тогда останутся проблемы с переносом строки в имени, но это уже не так актуально. 260068 2 iv 2025-02-28 00:34:51 +0300 Тривиальный фикс достаточно тривиален https://git.altlinux.org/people/iv/packages/rpm.git?a=commitdiff;h=f983dc085c41f3790dee026286a873b5855e7994 но явно не достаточен: [...] Finding debuginfo files (using /usr/lib/rpm/find-debuginfo-files) Executing: /bin/sh -e /tmp/.private/iv/rpm-tmp.lC8hA9 Creating spaces-debuginfo package Processing files: spaces-debuginfo-0.1-alt1 error: Two files on one line: /usr/lib/debug/usr/bin/space error: File must begin with "/": test.debug RPM build errors: Two files on one line: /usr/lib/debug/usr/bin/space File must begin with "/": test.debug 260069 3 iv 2025-02-28 00:39:37 +0300 В качестве workaround'а можно воспользоваться %add_debuginfo_skiplist. 260073 4 vt 2025-02-28 08:49:57 +0300 А разве в rpm можно запаковать файлы с пробелами. 260075 5 iv 2025-02-28 08:57:09 +0300 (In reply to Vitaly Chikunov from comment #4) > А разве в rpm можно запаковать файлы с пробелами. А почему нет? [iv@team ~]$ grep -c ' ' /ALT/sisyphus/{x86_64,noarch}/base/contents_index /ALT/sisyphus/x86_64/base/contents_index:13097 /ALT/sisyphus/noarch/base/contents_index:67962 Если добавить в начало тестового спека `%add_debuginfo_skiplist %_bindir`, он соберётся; с установкой и удалением тоже вроде проблем никаких. 260077 6 vt 2025-02-28 09:18:36 +0300 Возможно, я тестировал как раз на бинарниках и принял эту ошибку за то, что "вообще не работает". Так же, Арсений упоминал https://bugzilla.altlinux.org/show_bug.cgi?id=51874#c6 что есть какая-то проблема с пробелами, но пояснения не дал, я решил тогда, что она относится к предложенному там патчу. 260087 7 vt 2025-02-28 10:01:44 +0300 Ну да, если указать в спеке имя файла с пробелом %files %_bindir/test generated то ошибка rpm-build: "Two files on one line". То что запаковывается - это по glob или по вложенным в диры файлам. 260091 8 vt 2025-02-28 10:09:42 +0300 Может кто знает как это решается в апстримах? В смысле разрешено ли там указывать файлы с пробелами в %files, может для этого требуется \ или " ". 260092 9 iv 2025-02-28 10:17:37 +0300 (In reply to Vitaly Chikunov from comment #7) > То что запаковывается - это по > glob или по вложенным в диры файлам. Внезапно, ещё вот так работает: %files "%_bindir/space test" А экранирование `\` или `[ ]` нет. 260094 10 iv 2025-02-28 10:22:54 +0300 Разбор проблем: https://github.com/rpm-software-management/rpm/issues/1749#issuecomment-1020420616 До конца по ссылке пока не дочитал читал. 260101 11 vt 2025-02-28 11:07:26 +0300 (In reply to Ivan A. Melnikov from comment #10) > Разбор проблем: > > https://github.com/rpm-software-management/rpm/issues/1749#issuecomment- > 1020420616 > > До конца по ссылке пока не дочитал читал. Спасибо. Апстрим захотели как можно больше приблизить формат строки к shell и сделали большие изменения для этого. Мы можем пока просто заключать эти генеренные имена файлов в кавычки. Тестовое задание https://git.altlinux.org/tasks/376351/ (пока еще собирается) 260129 12 iv 2025-02-28 16:50:02 +0300 > Тестовое задание https://git.altlinux.org/tasks/376351/ С rpm-build их этого задания пакеты -- и тестовый спек, и тот, на котором я вчера наткнулся на эту проблему -- успешно собрались. Debuginfo для spaces из тестового спека ставится и работает в gdb. При этом, насколько я понимаю работу и применение strtokWithQuotes, сломать эти изменения ничего не должны. 260131 13 vt 2025-02-28 17:26:26 +0300 Надо подумать может не так сложно отключить поддержку glob внутри кавычек, это решит и баг #51874. 267178 14 vt 2025-06-14 01:40:57 +0300 (In reply to Vitaly Chikunov from comment #11) > Тестовое задание https://git.altlinux.org/tasks/376351/ (пока еще собирается) Раз все ок, то я коммичу эти изменения вместе с еще одним фиксом для debuginfo. 267188 15 repository-robot 2025-06-14 21:15:28 +0300 rpm-build-4.0.4.205-alt1 -> sisyphus: Sat Jun 14 2025 Vitaly Chikunov <vt@altlinux> 4.0.4.205-alt1 - debuginfo: Quote filenames to allow spaces in them (ALT#53250). - debuginfo: Restore .build-id symlink for vmlinux binary. 17840 2025-02-28 00:07:24 +0300 2025-02-28 00:07:24 +0300 соберите этот спек чтобы воспроизвести проблему spaces.spec text/x-rpm-spec 396 iv Ck5hbWU6IHNwYWNlcwpWZXJzaW9uOiAwLjEKUmVsZWFzZTogYWx0MQoKU3VtbWFyeTogU3BhY2Vz IFRlc3QKTGljZW5zZTogTUlUCkdyb3VwOiBPdGhlcgoKJWRlc2NyaXB0aW9uCiVzdW1tYXJ5Cgol cHJlcApjYXQgPiAndGVzdC5jJyA8PCAnRU9GJwppbnQgbWFpbigpIHsKICByZXR1cm4gMDsKfQpF T0YKCiVidWlsZApnY2MgLWcyIC1PMiB0ZXN0LmMgLW8gdGVzdAoKJWluc3RhbGwKbWtkaXIgLXAg JWJ1aWxkcm9vdCVfYmluZGlyCmNwIC4vdGVzdCAgIiVidWlsZHJvb3QlX2JpbmRpci9zcGFjZSB0 ZXN0IgoKJWZpbGVzCiVfYmluZGlyLyoKCiVjaGFuZ2Vsb2cKKiBGcmkgRmViIDI4IDIwMjUgSXZh biBBLiBNZWxuaWtvdiA8aXZAYWx0bGludXgub3JnPiAxLjAtYWx0MQotIHRlc3Qgc3R1ZmYK