Re: [sisyphus] Re: XMMS - когда?!

[php-coder <php-coder@ngs.ru>]

--- Michael Shigorin 2005-04-20 11:07:28 +0300
+++ php-coder 2005-04-20 23:35:22 +0700

> http://paq.osdn.org.ua/~mike/xmms-alt4/
> (из src.rpm надо выкинуть patch46 -- пока сам в задумчивости,
> как же с ним собрались бинарники, если не накладывается:
> собирал точно в одном месте)

    Тут и без этого взрывающегося патча хватает проблем :-( Часть патчей
    просто отвалились :-( Hunk'и практически во всех, которые я пока
    успел проверить. Сейчас правлю патчи на предмет соответствия
    версии 1.2.10

    Вот, к примеру, не очень знаю что делать с патчем xmms-1.2.7-automake.patch:

    [c1der@mycomp ~/xmms-1.2.10]$ cat ../RPM/SOURCES/xmms-1.2.7-automake.patch PD
    patching file aclocal/as.m4
    patching file configure.in
    Hunk #1 succeeded at 59 (offset 11 lines).
    can't find file to patch at input line 52
    Perhaps you used the wrong -p or --strip option?
    The text leading up to this was:
    --------------------------
    |diff -urN xmms-1.2.7.orig/configure.in.orig
    xmms-1.2.7/configure.in.orig
    |--- xmms-1.2.7.orig/configure.in.orig  Fri Aug  9 16:07:22 2002
    |+++ xmms-1.2.7/configure.in.orig       Fri Aug  9 16:07:22 2002
    --------------------------
    File to patch: 
    [c130der@mycomp ~/xmms-1.2.10]$

    После этого поправил адрес строки, и удалил данные правящие файл
    configure.in.orig -- его попросту нет. Идем дальше:

    [c130der@mycomp ~/xmms-1.2.10]$ cat ../RPM/SOURCES/xmms-1.2.7-automake.patch PD
    patching file aclocal/as.m4
    patching file configure.in
    patching file xmms/Makefile.am
    Hunk #1 FAILED at 6.
    1 out of 1 hunk FAILED -- saving rejects to file xmms/Makefile.am.rej
    
    Тут уже нужна правка помощнее. Вот что в оригинальном патче:

    [skip]
    -xmms_LDADD += @GTK_LIBS@ @PTHREAD_LIBS@ @SM_LIBS@ @VM_LIBS@ \
    -@POSIX_LIBS@ $(top_builddir)/libxmms/libxmms.la @INTLLIBS@
    +xmms_LDADD = @GTK_LIBS@ @PTHREAD_LIBS@ @SM_LIBS@ @VM_LIBS@ \
    +   @POSIX_LIBS@ $(top_builddir)/libxmms/libxmms.la @INTLLIBS@
     
      INCLUDES = @GTK_CFLAGS@ @XMMS_DEFINES@ @ARCH_DEFINES@
      -I$(top_builddir)/intl
    [skip]

    А вот что есть в реальном файле:

    [skip]
    xmms_LDADD = @GTK_LIBS@ @PTHREAD_LIBS@ @SM_LIBS@ @VM_LIBS@ \
    @POSIX_LIBS@ $(top_builddir)/libxmms/libxmms.la @LTLIBINTL@

    INCLUDES = @GTK_CFLAGS@ @XMMS_DEFINES@ @ARCH_DEFINES@ \
    -I$(top_builddir)/intl -I$(top_srcdir)
    [skip]

    Как видим, часть уже изменена до нашего пачта ( '=' вместо '+='), т.е.
    первую строчку нам менять не надо. Самое сложное: в патче присутствует
    @INTLLIBS@, а в файле из исходников используется уже @LTLIBINTL@. Я
    так понимаю, что следует изменить патч, чтобы он правил уже строчку из
    реальный исходников, добавляя tab? И там еще внизу разработчики
    разбили строку на две части. Тоже, как я понял нужно tab добавить.

    Вообщем, возможно, что все проблемы xmms'а в том, что часть патчей
    просто не наложена или наложена лишь часть :-(

    В связи с этим у меня есть предложение или даже FR на пакет rpmbuild:
    при вызове %patch должна вызываться программа patch без опции -s. Т.е.
    чтобы он не работал в тихом режиме. Тогда при сборке разработчик будет
    видеть и hunk'и и FAILED'ы. Я считаю, так будет намного правильнее.
    
    хм... у меня почти 12 часов ночи и я надеюсь, что более-менее ясно
    выразил свои мысли %) Интересно узнать мнение разработиков...

-- 
+ Временами и.о. cleanup_spec, php-coder ;-)