From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 20 Apr 2005 23:53:49 +0700 From: php-coder To: ALT Linux Sisyphus discussion list Subject: Re: [sisyphus] Re: XMMS - =?cp1251?B?6u7j?= =?cp1251?B?5OA/IQ==?= Message-ID: <20050420165349.GA2714@mycomp.mydomain> Mail-Followup-To: ALT Linux Sisyphus discussion list References: <200504191010.09841.sisyphus@Murmansk.Net> <20050419134327.GF17768@osdn.org.ua> <42654B7C.1070207@inbox.ru> <20050420080728.GP15011@osdn.org.ua> Mime-Version: 1.0 Content-Type: text/plain; charset=cp1251 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20050420080728.GP15011@osdn.org.ua> X-Mailer: The Bat! (v1.61) X-BeenThere: sisyphus@altlinux.ru X-Mailman-Version: 2.1.5 Precedence: list Reply-To: ALT Linux Sisyphus discussion list List-Id: ALT Linux Sisyphus discussion list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 20 Apr 2005 16:46:33 -0000 Archived-At: List-Archive: --- 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 ;-)