36382 2019-03-25 20:15:46 +0300 extlinux-6.04.pre1-alt1.5e426532 builds its own copy of zlib 2019-03-25 23:05:36 +0300 1 1 4 Development Sisyphus extlinux unstable all Linux CLOSED NOTABUG P3 normal --- 1 ldv legion legion qa-sisyphus oldest_to_newest 179925 0 ldv 2019-03-25 20:15:46 +0300 $ grep -Fc /com32/lib/zlib/ beehive/logs/Sisyphus-x86_64/latest/success/extlinux-6.04.pre1-alt1.5e426532 53 $ comm -12 <(/usr/lib/rpm/provided_symbols /lib64/libz.so.1 |sort) \ <(/usr/lib/rpm/provided_symbols /boot/extlinux/libcom32.c32 |sort) adler32 adler32_combine adler32_combine64 compress compress2 compressBound crc32 crc32_combine crc32_combine64 deflate deflateBound deflateCopy deflateEnd deflateInit_ deflateInit2_ deflateParams deflatePrime deflateReset deflateSetDictionary deflateSetHeader deflateTune get_crc_table inflate inflateBack inflateBackEnd inflateBackInit_ inflateCopy inflateEnd inflateGetHeader inflateInit_ inflateInit2_ inflateMark inflatePrime inflateReset inflateReset2 inflateSetDictionary inflateSync inflateSyncPoint inflateUndermine uncompress zError zlibCompileFlags zlibVersion 179928 1 legion 2019-03-25 20:37:21 +0300 Yes. This code is executed before the kernel is loaded. It cannot be linked with dynamic libraries. 179929 2 ldv 2019-03-25 20:40:08 +0300 (In reply to comment #1) > Yes. This code is executed before the kernel is loaded. It cannot be linked > with dynamic libraries. What about linking with zlib.a? 179933 3 legion 2019-03-25 21:09:32 +0300 (In reply to comment #2) > What about linking with zlib.a? They have not only zlib: https://repo.or.cz/syslinux.git/tree/05ac953c23f90b2328d393f7eecde96e41aed067:/com32/lib/zlib https://repo.or.cz/syslinux.git/tree/05ac953c23f90b2328d393f7eecde96e41aed067:/com32/lib/libpng BTW I can see that they changed this code: https://repo.or.cz/syslinux.git/history/HEAD:/com32/lib/zlib I see no point in linking this code with static zlib and libpng. It runs on another level than kernel or everything else in the system. 179936 4 ldv 2019-03-25 21:47:44 +0300 (In reply to comment #3) > (In reply to comment #2) > > What about linking with zlib.a? > > They have not only zlib: > > https://repo.or.cz/syslinux.git/tree/05ac953c23f90b2328d393f7eecde96e41aed067:/com32/lib/zlib > https://repo.or.cz/syslinux.git/tree/05ac953c23f90b2328d393f7eecde96e41aed067:/com32/lib/libpng > > BTW I can see that they changed this code: > > https://repo.or.cz/syslinux.git/history/HEAD:/com32/lib/zlib It's the same code as zlib v1.2.11 with some files removed. > I see no point in linking this code with static zlib and libpng. It runs on > another level than kernel or everything else in the system. Could you at least make these zlib symbols hidden? 179938 5 legion 2019-03-25 22:21:22 +0300 (In reply to comment #4) > > I see no point in linking this code with static zlib and libpng. It runs on > > another level than kernel or everything else in the system. > > Could you at least make these zlib symbols hidden? Why ? All that is below the /boot/extlinux can't be verified for unresolved symbols as well. We agreed that these files will be ignored by any distro checks. 179940 6 ldv 2019-03-25 22:32:57 +0300 (In reply to comment #5) > (In reply to comment #4) > > > I see no point in linking this code with static zlib and libpng. It runs on > > > another level than kernel or everything else in the system. > > > > Could you at least make these zlib symbols hidden? > > Why ? Because you can. I wish I could change the default in gcc to generate all symbols as hidden. > All that is below the /boot/extlinux can't be verified for unresolved symbols > as well. We agreed that these files will be ignored by any distro checks. I don't see any signs of it in the code. 179943 7 legion 2019-03-25 22:48:05 +0300 (In reply to comment #6) > > Why ? > > Because you can. > I wish I could change the default in gcc to generate all symbols as hidden. Я могу попробовать, но не обещаю. Я очень не хочу этот код сломать изменениями ради изменений. Если он тебе мешает, то удаляй его из сизифа. Я буду его использовать вне репозитория. > > All that is below the /boot/extlinux can't be verified for unresolved symbols > > as well. We agreed that these files will be ignored by any distro checks. > > I don't see any signs of it in the code. Дим, да ты чего ? Мы же с тобой отдельно расследовали это, когда я выкладывал extlinux-6 и когда он не мог пройти сборочницу из-за того что апстрим перешёл на ELF. 179944 8 ldv 2019-03-25 22:57:57 +0300 (In reply to comment #7) > (In reply to comment #6) > > > Why ? > > > > Because you can. > > I wish I could change the default in gcc to generate all symbols as hidden. > > Я могу попробовать, но не обещаю. Я очень не хочу этот код сломать изменениями > ради изменений. Если он тебе мешает, то удаляй его из сизифа. Я буду его > использовать вне репозитория. Тебе же будет лучше, если zlib не будет из него светиться. > > > All that is below the /boot/extlinux can't be verified for unresolved symbols > > > as well. We agreed that these files will be ignored by any distro checks. > > > > I don't see any signs of it in the code. > > Дим, да ты чего ? Мы же с тобой отдельно расследовали это, когда я выкладывал > extlinux-6 и когда он не мог пройти сборочницу из-за того что апстрим перешёл > на ELF. Значит, я тогда сделал какое-то временное решение, которое никуда не закоммичено. Плохо. 179946 9 legion 2019-03-25 23:05:36 +0300 (In reply to comment #8) > > Я могу попробовать, но не обещаю. Я очень не хочу этот код сломать изменениями > > ради изменений. Если он тебе мешает, то удаляй его из сизифа. Я буду его > > использовать вне репозитория. > > Тебе же будет лучше, если zlib не будет из него светиться. С чего мне будет лучше ? Это никак не влияет на функционал. Это только заткнёт ещё одну userspace проверку разве что. > > Дим, да ты чего ? Мы же с тобой отдельно расследовали это, когда я выкладывал > > extlinux-6 и когда он не мог пройти сборочницу из-за того что апстрим перешёл > > на ELF. > > Значит, я тогда сделал какое-то временное решение, которое никуда не > закоммичено. Плохо. $ diff -u /usr/lib64/perl5/qa/rpmelfsym.pm{~,} --- /usr/lib64/perl5/qa/rpmelfsym.pm~ +++ /usr/lib64/perl5/qa/rpmelfsym.pm @@ -33,6 +33,7 @@ $filename =~ s#^\./|^/|^#/#; next if $filename =~ m#^/usr/lib/debug/.+\.debug\z#; + next if $filename =~ /^\/boot\//; $ent->read(my $magic, 4) == 4 or die "$rpm: $filename: cpio read failed"; Смотри наш тред "Re: [#197489] FAILED extlinux.git=6.03-alt1" от 26 декабря 2017.