[devel] I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[devel] I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Alexey M. Tourbin]

[-- Attachment #1: Type: text/plain, Size: 873 bytes --]

	6 NEW unexpanded_macros
etersoft-build-utils-0.9-alt0.2.noarch.rpm	DESCRIPTION	%URL
etersoft-build-utils-0.9-alt0.2.src.rpm	DESCRIPTION	%URL
libnatspec-0.2.2-alt2.i586.rpm	DESCRIPTION	%URL
libnatspec-0.2.2-alt2.src.rpm	DESCRIPTION	%URL
libnatspec-0.2.2-alt2.x86_64.rpm	DESCRIPTION	%URL
oracle-preinstall-0.3-alt2.noarch.rpm	PREIN	%HTDOCS

	9 OLD unexpanded_macros
etersoft-build-utils-0.9-alt0.1.noarch.rpm	DESCRIPTION	%Url
etersoft-build-utils-0.9-alt0.1.src.rpm	DESCRIPTION	%Url
libnatspec-0.2.1-alt1.i586.rpm	DESCRIPTION	%Url
libnatspec-0.2.1-alt1.src.rpm	DESCRIPTION	%Url
libnatspec-0.2.1-alt1.x86_64.rpm	DESCRIPTION	%Url
libqwt-4.2.0-alt3.src.rpm	SOURCE	%realname
oracle-preinstall-0.3-alt1.noarch.rpm	PREIN	%APACHE
perl-Module-CoreList-2.02-alt1.noarch.rpm	DESCRIPTION	%Module
perl-Module-CoreList-2.02-alt1.src.rpm	DESCRIPTION	%Module

Total 782 unexpanded_macros.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

[devel] Re: I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Alexey Tourbin]

[-- Attachment #1.1: Type: text/plain, Size: 510 bytes --]

On Wed, Aug 03, 2005 at 04:17:03AM +0400, Alexey M. Tourbin wrote:
> 	6 NEW unexpanded_macros
> oracle-preinstall-0.3-alt2.noarch.rpm	PREIN	%HTDOCS
> 	9 OLD unexpanded_macros
> oracle-preinstall-0.3-alt1.noarch.rpm	PREIN	%APACHE

В скрипте есть ошибка: для каждой секции (напр. PREIN) ищется
максимум один нераскрытый макрос.  То есть когда макрос %APACHE
исправили, оказалось, что про запас был/есть ещё один нераскрытый
макрос -- %HTDOCS.

Завтра я объясню, как этот скрипт "завернуть" в робота.

[-- Attachment #1.2: unexpanded_macros --]
[-- Type: text/plain, Size: 542 bytes --]

#!/bin/sh -ef

dump()
{
	find "$@" -type f -name '*.rpm' -print0 |perl -Mstrict -ln0 -e \
'	use RPM::Header; my $f = $_;
	my $rpm = RPM::Header->new($f) or die "$f: $RPM::err\n";
	while (my ($k, $v) = each %$rpm) {
		next if $k =~ /CHANGELOGTEXT|SIGGPG|SIGMD5|GIF/;
		my @v = ref($v) ? @$v : $v;
		my @lines = map { split /[\r\n]+/ } @v;
		foreach (@lines) {
			next if /^\s*#/;
			next unless /\%[a-zA-Z_][a-zA-Z0-9_]+/;
			use File::Basename;
			printf "%s\t%s\t%s\n", basename($f), $k, $&;
		}
	}
'
}

dump "${1-/raid/ALT/Sisyphus/files}"

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Denis Ovsienko]

Можно полный список, отсортированный по packager, положить куда-нибудь, чтобы
желающие могли себя поискать? И включить в sisyphus_check.

-- 
    DO4-UANIC

Re: [devel] I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Vitaly Lipatov]

On Wednesday 03 August 2005 04:17, Alexey M. Tourbin wrote:
> libnatspec-0.2.2-alt2.i586.rpm  DESCRIPTION     %URL
Подскажите мне глупому: так есть у нас макрос, раскрывающийся в 
Url?

-- 
Lav
Виталий Липатов
Санкт-Петербург
GNU! ALT Linux Team! WINE! LaTeX! LyX! http://freesource.info

Re: [devel] I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Alexey Rusakov]

Vitaly Lipatov wrote:

>On Wednesday 03 August 2005 04:17, Alexey M. Tourbin wrote:
>  
>
>>libnatspec-0.2.2-alt2.i586.rpm  DESCRIPTION     %URL
>>    
>>
>Подскажите мне глупому: так есть у нас макрос, раскрывающийся в 
>Url?
>  
>
Я пользуюсь %url - вроде нормально (если верить этой проверялке).

-- 
  Alexey "Ktirf" Rusakov

[devel] Re: I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 4055 bytes --]

On Wed, Aug 03, 2005 at 10:38:12AM +0400, Denis Ovsienko wrote:
> Можно полный список, отсортированный по packager, положить куда-нибудь, чтобы
> желающие могли себя поискать? И включить в sisyphus_check.

Проверялка не точная, то есть она проверят хедеры и не может отличить
намерение maintainer'а (т.е. %%macro в спеке) от реально нераскрытых
макросов (т.е. %macro в спеке).

Тем не менее, большая часть обнаруженных "нераскрытых макросов"
действительно является нераскрытыми макросами.  Я недавно по этому
поводу развесил более 20 багов в багзилле.  Часть уже исправлено,
что и видно из письма.

На самом деле проверку лучше всего встроить в rpm, но это не по моей
части.  В sisyphus_check тоже можно встроить проверку, но только для
триггеров и скриптов, т.к. это особенно критично.

MPlayer-skins-1.1.0-alt1.noarch.rpm	DESCRIPTION	%setup
MPlayer-skins-1.1.0-alt1.src.rpm	DESCRIPTION	%setup
abook-0.5.3-alt1.i586.rpm	CHANGELOGNAME	%rlz1
abook-0.5.3-alt1.src.rpm	CHANGELOGNAME	%rlz1
alterator-backend-x11-0.3.7-alt1.i586.rpm	DESCRIPTION	%backend
alterator-backend-x11-0.3.7-alt1.i586.rpm	SUMMARY	%backend
apache2-suexec-2.0.54-alt1.i586.rpm	POSTUN	%postun_service
chkfontpath-2.0.1-alt1.i586.rpm	DESCRIPTION	%post
chkfontpath-2.0.1-alt1.src.rpm	DESCRIPTION	%post
chkfontpath-2.0.1-alt1.x86_64.rpm	DESCRIPTION	%post
etersoft-build-utils-0.9-alt0.2.noarch.rpm	DESCRIPTION	%URL
etersoft-build-utils-0.9-alt0.2.src.rpm	DESCRIPTION	%URL
glibc-core-debug-2.3.5-alt5.i586.rpm	DESCRIPTION	%_specdir
glibc-core-debug-2.3.5-alt5.x86_64.rpm	DESCRIPTION	%_specdir
glibc-devel-debug-2.3.5-alt5.i586.rpm	DESCRIPTION	%_specdir
glibc-devel-debug-2.3.5-alt5.x86_64.rpm	DESCRIPTION	%_specdir
glibc-nss-debug-2.3.5-alt5.i586.rpm	DESCRIPTION	%_specdir
glibc-nss-debug-2.3.5-alt5.x86_64.rpm	DESCRIPTION	%_specdir
gok-1.0.5-alt1.i586.rpm	PREUN	%_name
kpowersave-0.4.5-alt1.i586.rpm	DESCRIPTION	%debug_package
kpowersave-0.4.5-alt1.src.rpm	DESCRIPTION	%debug_package
libdvdplay-1.0.1-alt2.1.i586.rpm	DESCRIPTION	%libname
libdvdplay-1.0.1-alt2.1.src.rpm	DESCRIPTION	%libname
libnatspec-0.2.2-alt2.i586.rpm	DESCRIPTION	%URL
libnatspec-0.2.2-alt2.src.rpm	DESCRIPTION	%URL
libnatspec-0.2.2-alt2.x86_64.rpm	DESCRIPTION	%URL
mod_ssl-2.8.22-alt1.i586.rpm	PREIN	%postun
oracle-preinstall-0.3-alt2.noarch.rpm	PREIN	%HTDOCS
perl-Regexp-Common-2.120-alt1.noarch.rpm	DESCRIPTION	%RE
perl-Regexp-Common-2.120-alt1.src.rpm	DESCRIPTION	%RE
perl-Spreadsheet-ParseExcel-0.2602-alt1.noarch.rpm	URL	%module_dir
perl-Spreadsheet-ParseExcel-0.2602-alt1.src.rpm	URL	%module_dir
perl-Spreadsheet-ParseExcel-SaveParser-0.2602-alt1.noarch.rpm	URL	%module_dir
php-mcrypt-4.3.12-alt0.cvs20050530.i586.rpm	DESCRIPTION	%package_name
php-mcrypt-4.3.12-alt0.cvs20050530.src.rpm	DESCRIPTION	%package_name
powersave-0.9.25-alt2.i586.rpm	POSTUN	%postun_service
purge-0.1-alt1.i586.rpm	DESCRIPTION	%purge
purge-0.1-alt1.src.rpm	DESCRIPTION	%purge
qt3-settings-3.3-alt2.src.rpm	DESCRIPTION	%setup
scilab-3.0-alt1.i586.rpm	BASENAMES	%Block_e
scilab-3.0-alt1.i586.rpm	BASENAMES	%Block_p
scilab-3.0-alt1.i586.rpm	BASENAMES	%a2for
scilab-3.0-alt1.i586.rpm	BASENAMES	%a2sci
[ поскипано: в пакете scilab более 700 файлов с символом "%"; это ложное срабатывание ]
tcb-0.9.8.9-alt1.src.rpm	DESCRIPTION	%pam_name
tex2im-1.8-alt1.i586.rpm	URL	%real
tex2im-1.8-alt1.src.rpm	URL	%real
ups-monitor-0.6-alt1.i586.rpm	BASENAMES	%nam
urw-fonts-2.0-alt26.noarch.rpm	POSTIN	%ttfdir
webalizer-2.01.10-alt5.1.i586.rpm	TRIGGERSCRIPTS	%apache_webmaster
xemacs-lookup-1.14-alt2.noarch.rpm	POSTIN	%xemacs_mule_package
xemacs-lookup-1.14-alt2.noarch.rpm	PREUN	%xemacs_mule_package
xemacs-skk-1.23-alt4.noarch.rpm	POSTIN	%xemacs_mule_package
xemacs-skk-1.23-alt4.noarch.rpm	PREUN	%xemacs_mule_package
xmms-in-mac-0.2.1-alt2.1.i586.rpm	DESCRIPTION	%pack_ver
xmms-in-mac-0.2.1-alt2.1.src.rpm	DESCRIPTION	%pack_ver

Я зарядил робота, который время от времени будет о себе напоминать (мне).
Если будет что-то интересное, я буду переправлять в этот список рассылки.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Re: I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Sergey V Turchin]

[-- Attachment #1: Type: text/plain, Size: 301 bytes --]

On Wednesday 03 August 2005 13:09, Alexey Tourbin wrote:


> Проверялка не точная

[...]

> qt3-settings-3.3-alt2.src.rpm	DESCRIPTION	%setup
Да, ошибается

[...]

-- 
Regards, Sergey, ALT Linux Team, http://www.altlinux.ru
http://stinkfoot.org:11371/pks/lookup?op=get&search=0x1C2A3F08

[-- Attachment #2: Type: application/pgp-signature, Size: 190 bytes --]

[devel] Re: I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 317 bytes --]

On Thu, Aug 04, 2005 at 01:46:08PM +0400, Sergey V Turchin wrote:
> > Проверялка не точная
> > qt3-settings-3.3-alt2.src.rpm	DESCRIPTION	%setup
> Да, ошибается

$ rpm -qip qt3-settings-3.3-alt2.src.rpm
...
Description :
Qt3 settings files
%setup -q -T
$

То есть это не ошибка, а "артефакт", скажем так.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Re: I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 549 bytes --]

On Thu, Aug 04, 2005 at 03:01:47PM +0400, Alexey Tourbin wrote:
> On Thu, Aug 04, 2005 at 01:46:08PM +0400, Sergey V Turchin wrote:
> > > Проверялка не точная
> > > qt3-settings-3.3-alt2.src.rpm	DESCRIPTION	%setup
> > Да, ошибается
> 
> $ rpm -qip qt3-settings-3.3-alt2.src.rpm
> ...
> Description :
> Qt3 settings files
> %setup -q -T
> $
> 
> То есть это не ошибка, а "артефакт", скажем так.

Стало быть, в rpmbuild/sisyphus_check встроить затруднительно?
В rpmbuild >= 4.0.4-alt31 есть проверки тэгов зависимостей.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

[devel] Re: I: Sisyphus-20050803 unexpanded_macros: +6 -9 (782)

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 408 bytes --]

On Mon, Aug 08, 2005 at 06:12:14PM +0400, Dmitry V. Levin wrote:
> > $ rpm -qip qt3-settings-3.3-alt2.src.rpm
> > ...
> > Description :
> > Qt3 settings files
> > %setup -q -T
> > $
> > 
> > То есть это не ошибка, а "артефакт", скажем так.
> 
> Стало быть, в rpmbuild/sisyphus_check встроить затруднительно?

В rpmbuild, может быть, можно встроить проверку на ранней стадии
раскрытия макросов.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]