* [devel] I: brp-verify-unit: "bad permissions on ..."
@ 2024-02-10 9:55 Arseny Maslennikov
2024-02-10 10:01 ` Anton Farygin
2024-02-10 10:12 ` Dmitry V. Levin
0 siblings, 2 replies; 8+ messages in thread
From: Arseny Maslennikov @ 2024-02-10 9:55 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 9986 bytes --]
Hi!
В опубликованный сегодня Sisyphus вошёл новый rpm-build:
> rpm-build - Scripts and executable programs used to build packages
> * Thu Jan 11 2024 Arseny Maslennikov <arseny@altlinux> 4.0.4.195-alt1
> - debuginfo: Changed compression format (--lzma2=dict=2MiB ->
> --check=crc32 --lzma2=dict=1MiB) of xz-compressed modules for compatibility
> with kmod >= 31 (thx asheplyakov@).
> - Introduced brp-verify-unit to check sanity of systemd units included
> in built packages.
Новый brp-модуль проверяет юниты systemd на вшивость. Пока он содержит
две проверки:
* на файле с systemd-юнитом не должно быть x-бита;
* файл с systemd-юнитом, предусматривающим порождение процесса, не
должен запускать что-либо под nobody.
В результате сегодняшней тестовой пересборки обнаружилось[1] 14 исходных
пакетов, куда-то кладущих юнит с правами rwxr-xr-x, и 1 пакет,
содержащий юнит с правами rwxr-x---.
[1] https://lore.altlinux.org/sisyphus-cybertalk/Zcb1ezIHJkgVff21@beehive.mskdc.altlinux.org/T/#u
Пакеты, перечисленные ниже, нужно исправить, сняв x-биты с юнитов
systemd под %buildroot.
Под каждой цитатой из лога пересборки размещён acl на пакет.
bonito-open-5.58.1-alt1
+ mv misc/bonito_clear_cache.cron
/usr/src/tmp/bonito-open-buildroot//etc/cron.d/bonito_clear_cache
+ sed 's|/usr/bin/bonito_clear_cache|/usr/bin/bonito_clear_cache|' bonito_clear_cache
+ chmod a+x /usr/src/tmp/bonito-open-buildroot//usr/bin/bonito_clear_cache
+ /usr/lib/rpm/brp-alt
Cleaning files in /usr/src/tmp/bonito-open-buildroot (auto)
Verifying and fixing files in /usr/src/tmp/bonito-open-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/bonito-open-buildroot/ (default)
Compressing files in /usr/src/tmp/bonito-open-buildroot (auto)
Verifying systemd units in /usr/src/tmp/bonito-open-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/skejobserver.service":
-rwxr-xr-x
bonito-open kirill @everybody
bozohttpd-20220517-alt1
044-verify-unit.brp: bad permissions on "/lib/systemd/system/bozohttpd@.service":
-rwxr-xr-x
044-verify-unit.brp: ERROR: "/lib/systemd/system/bozohttpd@.service" assumes overflowugid
credentials
bozohttpd george @everybody
cpufreqd-2.4.3-alt3
<...>
Verifying and fixing files in /usr/src/tmp/cpufreqd-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/cpufreqd-buildroot/ (default)
Compressing files in /usr/src/tmp/cpufreqd-buildroot (auto)
Verifying systemd units in /usr/src/tmp/cpufreqd-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/cpufreqd.service": -rwxr-xr-x
cpufreqd shaba
ctwm-1:4.1.0-alt1
Verifying and fixing files in /usr/src/tmp/ctwm-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/ctwm-buildroot/ (default)
Compressing files in /usr/src/tmp/ctwm-buildroot (auto)
mode of '/usr/src/tmp/ctwm-buildroot/usr/share/man/man1/ctwm.1' changed from 0755
(rwxr-xr-x) to 0644 (rw-r--r--)
gunzip: /usr/src/tmp/ctwm-buildroot/usr/share/man/man1/ctwm.1 already exists; not
overwritten
Verifying systemd units in /usr/src/tmp/ctwm-buildroot
044-verify-unit.brp: bad permissions on "/usr/lib/systemd/user/ctwm.target": -rwxr-xr-x
044-verify-unit.brp: bad permissions on "/usr/lib/systemd/user/ctwm-session.target":
-rwxr-xr-x
044-verify-unit.brp: bad permissions on "/usr/lib/systemd/user/ctwm.service": -rwxr-xr-x
ctwm george @qa
dictd-1:1.13.1-alt1
<...>
Checking contents of files in /usr/src/tmp/dictd-buildroot/ (default)
Compressing files in /usr/src/tmp/dictd-buildroot (auto)
Verifying systemd units in /usr/src/tmp/dictd-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/dictd.service": -rwxr-xr-x
dictd lav cheusov @qa @everybody
foreman-3.5.1-alt8
+ /usr/lib/rpm/brp-alt
Cleaning files in /usr/src/tmp/foreman-buildroot (auto)
removed './usr/lib/foreman/Gemfile.orig'
removed './usr/lib/foreman/app/models/setting.rb.orig'
removed './usr/lib/foreman/app/models/role.rb.orig'
Verifying and fixing files in /usr/src/tmp/foreman-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/foreman-buildroot/ (default)
Compressing files in /usr/src/tmp/foreman-buildroot (auto)
Verifying systemd units in /usr/src/tmp/foreman-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/foreman.service": -rwxr-xr-x
foreman majioa @everybody
ima-integrity-check-0.5.1-alt1
+ /usr/lib/rpm/brp-alt
Cleaning files in /usr/src/tmp/ima-integrity-check-buildroot (auto)
Verifying and fixing files in /usr/src/tmp/ima-integrity-check-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/ima-integrity-check-buildroot/ (default)
Compressing files in /usr/src/tmp/ima-integrity-check-buildroot (auto)
Verifying systemd units in /usr/src/tmp/ima-integrity-check-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/signing.service": -rwxr-x---
ima-integrity-check nbr @everybody
matterbridge-1.22.3-alt1
+ /usr/lib/rpm/brp-alt
Cleaning files in /usr/src/tmp/matterbridge-buildroot (auto)
Verifying and fixing files in /usr/src/tmp/matterbridge-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/matterbridge-buildroot/ (default)
Compressing files in /usr/src/tmp/matterbridge-buildroot (auto)
Verifying systemd units in /usr/src/tmp/matterbridge-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/matterbridge.service":
-rwxr-xr-x
matterbridge @nobody
nbd-3.25-alt1
+ install -pD -m644 /usr/src/RPM/SOURCES/nbd.sysconfig
/usr/src/tmp/nbd-buildroot/etc/sysconfig/nbd-server
+ mkdir -p /usr/src/tmp/nbd-buildroot/usr/share/doc/nbd-3.25
+ install -pm644 README.md tests/run/simple_test
/usr/src/tmp/nbd-buildroot/usr/share/doc/nbd-3.25/
+ /usr/lib/rpm/brp-alt
Cleaning files in /usr/src/tmp/nbd-buildroot (auto)
Verifying and fixing files in /usr/src/tmp/nbd-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/nbd-buildroot/ (default)
Compressing files in /usr/src/tmp/nbd-buildroot (auto)
Verifying systemd units in /usr/src/tmp/nbd-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/nbd-server.service":
-rwxr-xr-x
nbd rider @everybody
passivedns-1.2.1-alt3
+ mkdir -p /usr/src/tmp/passivedns-buildroot/etc/logrotate.d
+ cat
+ ln -s /dev/null /usr/src/tmp/passivedns-buildroot/lib/systemd/system/passivedns.service
+ /usr/lib/rpm/brp-alt
Cleaning files in /usr/src/tmp/passivedns-buildroot (auto)
Verifying and fixing files in /usr/src/tmp/passivedns-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/passivedns-buildroot/ (default)
Compressing files in /usr/src/tmp/passivedns-buildroot (auto)
Verifying systemd units in /usr/src/tmp/passivedns-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/passivedns@.service":
-rwxr-xr-x
passivedns rider @everybody
puppetdb-7.12.0-alt1
+ /usr/lib/rpm/brp-alt
Cleaning files in /usr/src/tmp/puppetdb-buildroot (auto)
Verifying and fixing files in /usr/src/tmp/puppetdb-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/puppetdb-buildroot/ (default)
Compressing files in /usr/src/tmp/puppetdb-buildroot (auto)
Verifying systemd units in /usr/src/tmp/puppetdb-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/puppetdb.service": -rwxr-xr-x
error: Bad exit status from /usr/src/tmp/rpm-tmp.52351 (%install)
RPM build errors:
Macro %ubt not found
puppetdb dshein @everybody
virtualbox-7.0.14-alt1
Checking contents of files in /usr/src/tmp/virtualbox-buildroot/ (default)
Compressing files in /usr/src/tmp/virtualbox-buildroot (auto)
Verifying systemd units in /usr/src/tmp/virtualbox-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/virtualbox-vmsvga.service":
-rwxr-xr-x
error: Bad exit status from /usr/src/tmp/rpm-tmp.25157 (%install)
RPM build errors:
line 181: Deprecated PreReq converted to Requires(pre,postun): PreReq: virtualbox-common
= 7.0.14-alt1
line 314: Deprecated PreReq converted to Requires(pre,postun): PreReq: control >=
0.7.2-alt1
line 315: Deprecated PreReq converted to Requires(pre,postun): PreReq: shadow-utils
line 317: Deprecated PreReq converted to Requires(pre,postun): PreReq: sysvinit-utils
virtualbox sin nbr greh
vnstat-2.11-alt1
+ /usr/lib/rpm/brp-alt
Cleaning files in /usr/src/tmp/vnstat-buildroot (auto)
Verifying and fixing files in /usr/src/tmp/vnstat-buildroot
(binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/vnstat-buildroot/ (default)
Compressing files in /usr/src/tmp/vnstat-buildroot (auto)
Verifying systemd units in /usr/src/tmp/vnstat-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/vnstatd.service": -rwxr-xr-x
error: Bad exit status from /usr/src/tmp/rpm-tmp.43441 (%install)
RPM build errors:
File /usr/src/RPM/SOURCES/vnstat-2.11-alt1.patch is smaller than 8 bytes
vnstat naf
О 5 пакетах, которые brp-verify-unit зарубил из-за overflowugid
credentials, напишу немного позже.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [devel] I: brp-verify-unit: "bad permissions on ..."
2024-02-10 9:55 [devel] I: brp-verify-unit: "bad permissions on ..." Arseny Maslennikov
@ 2024-02-10 10:01 ` Anton Farygin
2024-02-10 10:22 ` Arseny Maslennikov
2024-02-10 10:12 ` Dmitry V. Levin
1 sibling, 1 reply; 8+ messages in thread
From: Anton Farygin @ 2024-02-10 10:01 UTC (permalink / raw)
To: devel
Арсений, у нас есть механизм (работающий) принятия разного рода политик.
Когда вы делаете изменение, блокирующее пересборку какого-то пакета, то
было бы неплохо давать ссылку на принятую политику, на основании которой
было сделано такое изменений.
И делать анонс в devel чуть чуть раньше, чем факт падения пересборки.
Изменения хорошие, но для меня, как и для всех собирающим под Альт (не
только участникам Team) было бы намного удобнее прочитать про правила
упаковки systemd юнитов на www.altlinux.org а не в архивах рассылки devel.
On 10.02.2024 12:55, Arseny Maslennikov wrote:
> Hi!
>
> В опубликованный сегодня Sisyphus вошёл новый rpm-build:
>> rpm-build - Scripts and executable programs used to build packages
>> * Thu Jan 11 2024 Arseny Maslennikov <arseny@altlinux> 4.0.4.195-alt1
>> - debuginfo: Changed compression format (--lzma2=dict=2MiB ->
>> --check=crc32 --lzma2=dict=1MiB) of xz-compressed modules for compatibility
>> with kmod >= 31 (thx asheplyakov@).
>> - Introduced brp-verify-unit to check sanity of systemd units included
>> in built packages.
> Новый brp-модуль проверяет юниты systemd на вшивость. Пока он содержит
> две проверки:
> * на файле с systemd-юнитом не должно быть x-бита;
> * файл с systemd-юнитом, предусматривающим порождение процесса, не
> должен запускать что-либо под nobody.
>
> В результате сегодняшней тестовой пересборки обнаружилось[1] 14 исходных
> пакетов, куда-то кладущих юнит с правами rwxr-xr-x, и 1 пакет,
> содержащий юнит с правами rwxr-x---.
>
> [1] https://lore.altlinux.org/sisyphus-cybertalk/Zcb1ezIHJkgVff21@beehive.mskdc.altlinux.org/T/#u
>
> Пакеты, перечисленные ниже, нужно исправить, сняв x-биты с юнитов
> systemd под %buildroot.
> Под каждой цитатой из лога пересборки размещён acl на пакет.
>
> bonito-open-5.58.1-alt1
> + mv misc/bonito_clear_cache.cron
> /usr/src/tmp/bonito-open-buildroot//etc/cron.d/bonito_clear_cache
> + sed 's|/usr/bin/bonito_clear_cache|/usr/bin/bonito_clear_cache|' bonito_clear_cache
> + chmod a+x /usr/src/tmp/bonito-open-buildroot//usr/bin/bonito_clear_cache
> + /usr/lib/rpm/brp-alt
> Cleaning files in /usr/src/tmp/bonito-open-buildroot (auto)
> Verifying and fixing files in /usr/src/tmp/bonito-open-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/bonito-open-buildroot/ (default)
> Compressing files in /usr/src/tmp/bonito-open-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/bonito-open-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/skejobserver.service":
> -rwxr-xr-x
> bonito-open kirill @everybody
>
> bozohttpd-20220517-alt1
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/bozohttpd@.service":
> -rwxr-xr-x
> 044-verify-unit.brp: ERROR: "/lib/systemd/system/bozohttpd@.service" assumes overflowugid
> credentials
> bozohttpd george @everybody
>
> cpufreqd-2.4.3-alt3
> <...>
> Verifying and fixing files in /usr/src/tmp/cpufreqd-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/cpufreqd-buildroot/ (default)
> Compressing files in /usr/src/tmp/cpufreqd-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/cpufreqd-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/cpufreqd.service": -rwxr-xr-x
> cpufreqd shaba
>
> ctwm-1:4.1.0-alt1
> Verifying and fixing files in /usr/src/tmp/ctwm-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/ctwm-buildroot/ (default)
> Compressing files in /usr/src/tmp/ctwm-buildroot (auto)
> mode of '/usr/src/tmp/ctwm-buildroot/usr/share/man/man1/ctwm.1' changed from 0755
> (rwxr-xr-x) to 0644 (rw-r--r--)
> gunzip: /usr/src/tmp/ctwm-buildroot/usr/share/man/man1/ctwm.1 already exists; not
> overwritten
> Verifying systemd units in /usr/src/tmp/ctwm-buildroot
> 044-verify-unit.brp: bad permissions on "/usr/lib/systemd/user/ctwm.target": -rwxr-xr-x
> 044-verify-unit.brp: bad permissions on "/usr/lib/systemd/user/ctwm-session.target":
> -rwxr-xr-x
> 044-verify-unit.brp: bad permissions on "/usr/lib/systemd/user/ctwm.service": -rwxr-xr-x
> ctwm george @qa
>
> dictd-1:1.13.1-alt1
> <...>
> Checking contents of files in /usr/src/tmp/dictd-buildroot/ (default)
> Compressing files in /usr/src/tmp/dictd-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/dictd-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/dictd.service": -rwxr-xr-x
> dictd lav cheusov @qa @everybody
>
> foreman-3.5.1-alt8
> + /usr/lib/rpm/brp-alt
> Cleaning files in /usr/src/tmp/foreman-buildroot (auto)
> removed './usr/lib/foreman/Gemfile.orig'
> removed './usr/lib/foreman/app/models/setting.rb.orig'
> removed './usr/lib/foreman/app/models/role.rb.orig'
> Verifying and fixing files in /usr/src/tmp/foreman-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/foreman-buildroot/ (default)
> Compressing files in /usr/src/tmp/foreman-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/foreman-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/foreman.service": -rwxr-xr-x
> foreman majioa @everybody
>
> ima-integrity-check-0.5.1-alt1
> + /usr/lib/rpm/brp-alt
> Cleaning files in /usr/src/tmp/ima-integrity-check-buildroot (auto)
> Verifying and fixing files in /usr/src/tmp/ima-integrity-check-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/ima-integrity-check-buildroot/ (default)
> Compressing files in /usr/src/tmp/ima-integrity-check-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/ima-integrity-check-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/signing.service": -rwxr-x---
> ima-integrity-check nbr @everybody
>
> matterbridge-1.22.3-alt1
> + /usr/lib/rpm/brp-alt
> Cleaning files in /usr/src/tmp/matterbridge-buildroot (auto)
> Verifying and fixing files in /usr/src/tmp/matterbridge-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/matterbridge-buildroot/ (default)
> Compressing files in /usr/src/tmp/matterbridge-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/matterbridge-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/matterbridge.service":
> -rwxr-xr-x
> matterbridge @nobody
>
> nbd-3.25-alt1
> + install -pD -m644 /usr/src/RPM/SOURCES/nbd.sysconfig
> /usr/src/tmp/nbd-buildroot/etc/sysconfig/nbd-server
> + mkdir -p /usr/src/tmp/nbd-buildroot/usr/share/doc/nbd-3.25
> + install -pm644 README.md tests/run/simple_test
> /usr/src/tmp/nbd-buildroot/usr/share/doc/nbd-3.25/
> + /usr/lib/rpm/brp-alt
> Cleaning files in /usr/src/tmp/nbd-buildroot (auto)
> Verifying and fixing files in /usr/src/tmp/nbd-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/nbd-buildroot/ (default)
> Compressing files in /usr/src/tmp/nbd-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/nbd-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/nbd-server.service":
> -rwxr-xr-x
> nbd rider @everybody
>
> passivedns-1.2.1-alt3
> + mkdir -p /usr/src/tmp/passivedns-buildroot/etc/logrotate.d
> + cat
> + ln -s /dev/null /usr/src/tmp/passivedns-buildroot/lib/systemd/system/passivedns.service
> + /usr/lib/rpm/brp-alt
> Cleaning files in /usr/src/tmp/passivedns-buildroot (auto)
> Verifying and fixing files in /usr/src/tmp/passivedns-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/passivedns-buildroot/ (default)
> Compressing files in /usr/src/tmp/passivedns-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/passivedns-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/passivedns@.service":
> -rwxr-xr-x
> passivedns rider @everybody
>
> puppetdb-7.12.0-alt1
> + /usr/lib/rpm/brp-alt
> Cleaning files in /usr/src/tmp/puppetdb-buildroot (auto)
> Verifying and fixing files in /usr/src/tmp/puppetdb-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/puppetdb-buildroot/ (default)
> Compressing files in /usr/src/tmp/puppetdb-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/puppetdb-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/puppetdb.service": -rwxr-xr-x
> error: Bad exit status from /usr/src/tmp/rpm-tmp.52351 (%install)
> RPM build errors:
> Macro %ubt not found
> puppetdb dshein @everybody
>
> virtualbox-7.0.14-alt1
> Checking contents of files in /usr/src/tmp/virtualbox-buildroot/ (default)
> Compressing files in /usr/src/tmp/virtualbox-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/virtualbox-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/virtualbox-vmsvga.service":
> -rwxr-xr-x
> error: Bad exit status from /usr/src/tmp/rpm-tmp.25157 (%install)
> RPM build errors:
> line 181: Deprecated PreReq converted to Requires(pre,postun): PreReq: virtualbox-common
> = 7.0.14-alt1
> line 314: Deprecated PreReq converted to Requires(pre,postun): PreReq: control >=
> 0.7.2-alt1
> line 315: Deprecated PreReq converted to Requires(pre,postun): PreReq: shadow-utils
> line 317: Deprecated PreReq converted to Requires(pre,postun): PreReq: sysvinit-utils
> virtualbox sin nbr greh
>
> vnstat-2.11-alt1
> + /usr/lib/rpm/brp-alt
> Cleaning files in /usr/src/tmp/vnstat-buildroot (auto)
> Verifying and fixing files in /usr/src/tmp/vnstat-buildroot
> (binconfig,pkgconfig,libtool,desktop,gnuconfig)
> Checking contents of files in /usr/src/tmp/vnstat-buildroot/ (default)
> Compressing files in /usr/src/tmp/vnstat-buildroot (auto)
> Verifying systemd units in /usr/src/tmp/vnstat-buildroot
> 044-verify-unit.brp: bad permissions on "/lib/systemd/system/vnstatd.service": -rwxr-xr-x
> error: Bad exit status from /usr/src/tmp/rpm-tmp.43441 (%install)
> RPM build errors:
> File /usr/src/RPM/SOURCES/vnstat-2.11-alt1.patch is smaller than 8 bytes
> vnstat naf
>
> О 5 пакетах, которые brp-verify-unit зарубил из-за overflowugid
> credentials, напишу немного позже.
>
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [devel] I: brp-verify-unit: "bad permissions on ..."
2024-02-10 10:01 ` Anton Farygin
@ 2024-02-10 10:22 ` Arseny Maslennikov
0 siblings, 0 replies; 8+ messages in thread
From: Arseny Maslennikov @ 2024-02-10 10:22 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- Attachment #1: Type: text/plain, Size: 2194 bytes --]
On Sat, Feb 10, 2024 at 01:01:20PM +0300, Anton Farygin wrote:
> Арсений, у нас есть механизм (работающий) принятия разного рода политик.
> Когда вы делаете изменение, блокирующее пересборку какого-то пакета, то было
> бы неплохо давать ссылку на принятую политику, на основании которой было
> сделано такое изменений.
Видимо, нам будет полезна политика упаковки деймонов/сервисов вообще. В
том числе там написать и конкретно о юнитах.
> И делать анонс в devel чуть чуть раньше, чем факт падения пересборки.
Анонс можно делать и до пересборки, но тогда заранее не сообщишь, какие
пакеты окажутся сломаны.
Чтобы это было целесообразно, стоит интегрировать пересборочницу в
основную сборочницу сначала. Сейчас такая возможность есть только у
администраторов сборочницы, остальным она доступна после длительного
согласования.
> Изменения хорошие, но для меня, как и для всех собирающим под Альт (не
> только участникам Team) было бы намного удобнее прочитать про правила
> упаковки systemd юнитов на www.altlinux.org а не в архивах рассылки devel.
А вот это хорошая идея. Сейчас даже странно, что такой страницы нет;
конкретно про x-бит на юнитах при мне shaba@ и другие участники
талдычили множество раз, может быть, даже на ресурсах сообщества.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [devel] I: brp-verify-unit: "bad permissions on ..."
2024-02-10 9:55 [devel] I: brp-verify-unit: "bad permissions on ..." Arseny Maslennikov
2024-02-10 10:01 ` Anton Farygin
@ 2024-02-10 10:12 ` Dmitry V. Levin
2024-02-10 10:36 ` Arseny Maslennikov
2024-02-10 10:42 ` Arseny Maslennikov
1 sibling, 2 replies; 8+ messages in thread
From: Dmitry V. Levin @ 2024-02-10 10:12 UTC (permalink / raw)
To: devel
On Sat, Feb 10, 2024 at 12:55:26PM +0300, Arseny Maslennikov wrote:
> Hi!
>
> В опубликованный сегодня Sisyphus вошёл новый rpm-build:
[...]
> > - Introduced brp-verify-unit to check sanity of systemd units included
> > in built packages.
>
> Новый brp-модуль проверяет юниты systemd на вшивость. Пока он содержит
> две проверки:
> * на файле с systemd-юнитом не должно быть x-бита;
Проверять права доступа на упакованные файлы лучше в sisyphus_check.
На мой взгляд, brp-скриптам лучше бы сразу исправлять эту ошибку,
по аналогии с тем, как уже более 20 лет делают scripts/brp-fix-perms
и scripts/fixup-libraries.
Вот цитата из лога сборки пакета, в которой это хорошо видно:
+ /usr/lib/rpm/brp-alt
Cleaning files in /usr/src/tmp/cpufreqd-buildroot (auto)
mode of './usr/lib64/cpufreqd/cpufreqd_tau.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
mode of './usr/lib64/cpufreqd/cpufreqd_governor_parameters.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
mode of './usr/lib64/cpufreqd/cpufreqd_exec.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
mode of './usr/lib64/cpufreqd/cpufreqd_sensors.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
mode of './usr/lib64/cpufreqd/cpufreqd_pmu.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
mode of './usr/lib64/cpufreqd/cpufreqd_apm.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
mode of './usr/lib64/cpufreqd/cpufreqd_nforce2.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
mode of './usr/lib64/cpufreqd/cpufreqd_acpi.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
mode of './usr/lib64/cpufreqd/cpufreqd_cpu.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
mode of './usr/lib64/cpufreqd/cpufreqd_programs.so' changed from 0755 (rwxr-xr-x) to 0644 (rw-r--r--)
Verifying and fixing files in /usr/src/tmp/cpufreqd-buildroot (binconfig,pkgconfig,libtool,desktop,gnuconfig)
Checking contents of files in /usr/src/tmp/cpufreqd-buildroot/ (default)
Compressing files in /usr/src/tmp/cpufreqd-buildroot (auto)
Verifying systemd units in /usr/src/tmp/cpufreqd-buildroot
044-verify-unit.brp: bad permissions on "/lib/systemd/system/cpufreqd.service": -rwxr-xr-x
--
ldv
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [devel] I: brp-verify-unit: "bad permissions on ..."
2024-02-10 10:12 ` Dmitry V. Levin
@ 2024-02-10 10:36 ` Arseny Maslennikov
2024-02-10 10:47 ` Dmitry V. Levin
2024-02-10 10:42 ` Arseny Maslennikov
1 sibling, 1 reply; 8+ messages in thread
From: Arseny Maslennikov @ 2024-02-10 10:36 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- Attachment #1: Type: text/plain, Size: 1501 bytes --]
On Sat, Feb 10, 2024 at 12:12:53PM +0200, Dmitry V. Levin wrote:
> On Sat, Feb 10, 2024 at 12:55:26PM +0300, Arseny Maslennikov wrote:
> > Hi!
> >
> > В опубликованный сегодня Sisyphus вошёл новый rpm-build:
> [...]
> > > - Introduced brp-verify-unit to check sanity of systemd units included
> > > in built packages.
> >
> > Новый brp-модуль проверяет юниты systemd на вшивость. Пока он содержит
> > две проверки:
> > * на файле с systemd-юнитом не должно быть x-бита;
>
> Проверять права доступа на упакованные файлы лучше в sisyphus_check.
>
> На мой взгляд, brp-скриптам лучше бы сразу исправлять эту ошибку,
> по аналогии с тем, как уже более 20 лет делают scripts/brp-fix-perms
> и scripts/fixup-libraries.
Есть ли какой-либо критерий, определяющий, когда такое стоит автоисправлять,
а когда сообщать об ошибке?
А то вдруг файлы с библиотеками — по какой-то причине особый случай: у
них, например, и entry point может быть, это иногда используется.
В остальном — не возражаю, можно и так. :)
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [devel] I: brp-verify-unit: "bad permissions on ..."
2024-02-10 10:36 ` Arseny Maslennikov
@ 2024-02-10 10:47 ` Dmitry V. Levin
0 siblings, 0 replies; 8+ messages in thread
From: Dmitry V. Levin @ 2024-02-10 10:47 UTC (permalink / raw)
To: devel
On Sat, Feb 10, 2024 at 01:36:39PM +0300, Arseny Maslennikov wrote:
> On Sat, Feb 10, 2024 at 12:12:53PM +0200, Dmitry V. Levin wrote:
> > On Sat, Feb 10, 2024 at 12:55:26PM +0300, Arseny Maslennikov wrote:
> > > Hi!
> > >
> > > В опубликованный сегодня Sisyphus вошёл новый rpm-build:
> > [...]
> > > > - Introduced brp-verify-unit to check sanity of systemd units included
> > > > in built packages.
> > >
> > > Новый brp-модуль проверяет юниты systemd на вшивость. Пока он содержит
> > > две проверки:
> > > * на файле с systemd-юнитом не должно быть x-бита;
> >
> > Проверять права доступа на упакованные файлы лучше в sisyphus_check.
> >
> > На мой взгляд, brp-скриптам лучше бы сразу исправлять эту ошибку,
> > по аналогии с тем, как уже более 20 лет делают scripts/brp-fix-perms
> > и scripts/fixup-libraries.
>
> Есть ли какой-либо критерий, определяющий, когда такое стоит автоисправлять,
> а когда сообщать об ошибке?
Автоисправлять лучше, когда выполнены следующие условия:
- в диагностике ошибки нет ложных срабатываний,
- исправление не может ничего испортить,
- ошибка является массовой.
> А то вдруг файлы с библиотеками — по какой-то причине особый случай: у
> них, например, и entry point может быть, это иногда используется.
Там обрабатывается этот особый случай.
--
ldv
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [devel] I: brp-verify-unit: "bad permissions on ..."
2024-02-10 10:12 ` Dmitry V. Levin
2024-02-10 10:36 ` Arseny Maslennikov
@ 2024-02-10 10:42 ` Arseny Maslennikov
2024-02-10 10:49 ` Dmitry V. Levin
1 sibling, 1 reply; 8+ messages in thread
From: Arseny Maslennikov @ 2024-02-10 10:42 UTC (permalink / raw)
To: ALT Linux Team development discussions
[-- Attachment #1: Type: text/plain, Size: 1248 bytes --]
On Sat, Feb 10, 2024 at 12:12:53PM +0200, Dmitry V. Levin wrote:
> On Sat, Feb 10, 2024 at 12:55:26PM +0300, Arseny Maslennikov wrote:
> > Hi!
> >
> > В опубликованный сегодня Sisyphus вошёл новый rpm-build:
> [...]
> > > - Introduced brp-verify-unit to check sanity of systemd units included
> > > in built packages.
> >
> > Новый brp-модуль проверяет юниты systemd на вшивость. Пока он содержит
> > две проверки:
> > * на файле с systemd-юнитом не должно быть x-бита;
>
> Проверять права доступа на упакованные файлы лучше в sisyphus_check.
>
> На мой взгляд, brp-скриптам лучше бы сразу исправлять эту ошибку,
> по аналогии с тем, как уже более 20 лет делают scripts/brp-fix-perms
> и scripts/fixup-libraries.
У такого подхода есть и недостаток: это приводит к маскировке ошибок в
реализации install-фазы в апстриме (например, скрипт-рецепт по `make
install`).
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [devel] I: brp-verify-unit: "bad permissions on ..."
2024-02-10 10:42 ` Arseny Maslennikov
@ 2024-02-10 10:49 ` Dmitry V. Levin
0 siblings, 0 replies; 8+ messages in thread
From: Dmitry V. Levin @ 2024-02-10 10:49 UTC (permalink / raw)
To: devel
On Sat, Feb 10, 2024 at 01:42:50PM +0300, Arseny Maslennikov wrote:
> On Sat, Feb 10, 2024 at 12:12:53PM +0200, Dmitry V. Levin wrote:
> > On Sat, Feb 10, 2024 at 12:55:26PM +0300, Arseny Maslennikov wrote:
> > > Hi!
> > >
> > > В опубликованный сегодня Sisyphus вошёл новый rpm-build:
> > [...]
> > > > - Introduced brp-verify-unit to check sanity of systemd units included
> > > > in built packages.
> > >
> > > Новый brp-модуль проверяет юниты systemd на вшивость. Пока он содержит
> > > две проверки:
> > > * на файле с systemd-юнитом не должно быть x-бита;
> >
> > Проверять права доступа на упакованные файлы лучше в sisyphus_check.
> >
> > На мой взгляд, brp-скриптам лучше бы сразу исправлять эту ошибку,
> > по аналогии с тем, как уже более 20 лет делают scripts/brp-fix-perms
> > и scripts/fixup-libraries.
>
> У такого подхода есть и недостаток: это приводит к маскировке ошибок в
> реализации install-фазы в апстриме (например, скрипт-рецепт по `make
> install`).
В том числе и по этой причине автоисправление должно быть видимым в логе сборки.
--
ldv
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2024-02-10 10:49 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-10 9:55 [devel] I: brp-verify-unit: "bad permissions on ..." Arseny Maslennikov
2024-02-10 10:01 ` Anton Farygin
2024-02-10 10:22 ` Arseny Maslennikov
2024-02-10 10:12 ` Dmitry V. Levin
2024-02-10 10:36 ` Arseny Maslennikov
2024-02-10 10:47 ` Dmitry V. Levin
2024-02-10 10:42 ` Arseny Maslennikov
2024-02-10 10:49 ` Dmitry V. Levin
ALT Linux Team development discussions
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
public-inbox-index devel
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git