Bug 34057 - Исправление сборки с lcc (проблема обнаружения rpmlib)
Summary: Исправление сборки с lcc (проблема обнаружения rpmlib)
Status: CLOSED WORKSFORME
Alias: None
Product: Sisyphus
Classification: Development
Component: synaptic (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Anton Farygin
QA Contact: qa-sisyphus
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-10-24 01:37 MSK by Andrew Savchenko
Modified: 2017-10-24 21:40 MSK (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Savchenko 2017-10-24 01:37:22 MSK
synaptic-0.58-alt16 не собирается с lcc: хотя проблема с makepair и исправлена в alt16, осталась ещё одна:

lcc: "rpackagelister.cc", line 63: catastrophic error: cannot open source file
          "apt-pkg/debfile.h"
  #include <apt-pkg/debfile.h>

Это происходит из-за неопределения rpmlib (rpackagelister.cc):

#ifndef HAVE_RPM
#include <apt-pkg/debfile.h>
#endif

которое происходит из-за фейла автосгенерированного теста configure:

configure:8353: gcc -o conftest -pipe -Wall -g -O3 -fno-exceptions -std=gnu11  -I/usr/include/rpm  conftest.c -lrpm   -lrpm -lrpmio >&5
/usr/bin/ld: /usr/src/tmp/lcc_n6Bhzd.o: undefined reference to symbol 'rpmdbInitIterator'
/usr/lib64/librpmdb-4.0.4.so: error adding symbols: DSO missing from command line

Суть проблемы в том, что autotools генерирует примерно следующий код для тестирования наличия символа:

#include <rpm/rpmlib.h>

int| main ()
{
    return rpmdbInitIterator ();
}

В то время, как у rpmdbInitIterator 4 обязательных аргумента:

rpmdbMatchIterator rpmdbInitIterator(rpmdb db, rpmDbiTagVal rpmtag,
            const void * keyp, size_t keylen);

Так вот gcc это проглатывает, а lcc давится. Дабы глубоко не перелопачивать внутренности autotools, проблема решилась заменой rpmdbMatchIterator(...) на функцию, которой на самом деле не нужны аргументы: rpmFreeRpmrc().

Патч:
http://git.altlinux.org/people/bircoph/packages/?p=synaptic.git;a=blob;f=synaptic-0.58-alt-rpmlib.patch;h=33f2e74e1c9d5628e1d0aa67d099e07ba0104cf4;hb=refs/heads/sisyphus

Git дерево с новым патч релизом:
http://git.altlinux.org/people/bircoph/packages/synaptic.git
Comment 1 Michael Shigorin 2017-10-24 11:53:56 MSK
Эээ... забыл отметить, что сизифный synaptic переехал на API rpm-4.1x (см. коммит  1469e78afdff387fd3f882896b9a9b7368eeaadb), а на e2k пока ещё rpm-4.0.4, сопоставимый с версией из p8.  При этом перетаскивать e2k на rpm-4.13+ тоже планируется (Глеб его уже собрал).

Ergo: проверим сборку/функционирование собранного с твоим патчем и на sisyphus/x86_64, и на ~sisyphus/e2k; если всё в порядке, значит, и в сизиф.
Кстати, можешь взять эту задачку как join job.
Comment 2 Michael Shigorin 2017-10-24 12:20:44 MSK
На e2k отлично работает.
Comment 3 Michael Shigorin 2017-10-24 12:24:30 MSK
Гм, а вот на sisyphus/x86_64 каким-то образом собрался unmet:

  synaptic: Требует: libapt-pkg-libc6.9-6.so.6()(64bit) (>= set:og81aJo7DqKG8MDeK03yoZ2WOoZ5s4TREEvrfrVYY1WLkn7eja2MCt4R7LhQsV4KTPZ2SE1DNm0j3ZnhZ7c7izr613VBO5XSJa2QRgZinWrMr1u0YQVXZ8B4HLKlfC5jfgUcHsLutI2ciNihsTjZyme4OmGEHI7seyGU1fJG8cL2azWqaJCbdHzXd9Kbs6jcU4xDsdej4zahI437ZLG2GjCM6EXQ0plFYmMQck63d1C6DPHSgYmAH6m7VXhMMurmZaMxgWH7MZng132D5pz40ZqTukPMjN1WajSn9zcAIPUaKSmW2XC6u9iw2BXJJ6qSaz0CLmg3qL1O5AjK9DGtctZb9qFQPh2qNKZyxrBEiKH6uFeabHUpo4zOxpPsDi8d5ZA2mheZBtX7Ej5Ad8vSohFVmZgg)
Comment 4 Andrew Savchenko 2017-10-24 19:27:31 MSK
Исходное исправление было не совсем правильным, переделал, теперь собирается и устанавливается успешно как на x86_64 (мой десктоп), так и на e2k (проверил в hasher):http://git.altlinux.org/people/bircoph/packages/synaptic.git
Comment 5 Gleb F-Malinovskiy 2017-10-24 21:40:11 MSK
(In reply to comment #4)
> Исходное исправление было не совсем правильным, переделал, теперь собирается и
> устанавливается успешно как на x86_64 (мой десктоп), так и на e2k (проверил в
> hasher):http://git.altlinux.org/people/bircoph/packages/synaptic.git

В таком виде, мне кажется, класть в Сизиф это не нужно, потому что в Сизифе новый rpm, а этот патч вообще не является свойством самой архитектуры e2k.