Bug 25998

Summary:	bugfix for setup -n "name with spaces"
Product:	Sisyphus	Reporter:	viy <viy>
Component:	rpm	Assignee:	placeholder <placeholder>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	at, glebfm, imz, ldv, placeholder, vt
Version:	unstable
Hardware:	all
OS:	Linux
URL:	http://git.altlinux.org/people/viy/packages/?p=rpm.git;a=commit;h=c9505d67b9449db8ea5beba3ec25451466d0e5b7

Description viy 2011-08-06 19:44:19 MSK

commit c9505d67b9449db8ea5beba3ec25451466d0e5b7
Authorcommit c9505d67b9449db8ea5beba3ec25451466d0e5b7
Author: Igor Vlasenko <viy@altlinux.org>
Date:   Sat Aug 6 18:38:30 2011 +0300

    build.c,parsePrep.c: quoted buildSubdir, to allow spaces.

можно протестировать на 
# 51641 sisyphus srpm=fonts-ttf-sil-gentium-basic-1.1-alt1_6.src.rpm
который собирается под патченным rpm

Comment 1 at@altlinux.org 2011-08-06 20:22:44 MSK

А
setup -q -n 'sngile-quoted'
разве не работает?

Дело в том, что single quote - это не совсем полноценный quote, и если появится name with single quote, то извернуться будет ещё сложнее.

Впрочем, это изменение соответствует rpm.org 412f9fcb17f7d808

Comment 2 viy 2011-08-06 21:42:17 MSK

sngile-quoted не пробовал, 
мне нужно это изменение для робота импорта из Федоры, чтобы быть совместимым с федорой.
# 51641 sisyphus srpm=fonts-ttf-sil-gentium-basic-1.1-alt1_6.src.rpm
--- это генерированный пакет.

Comment 3 at@altlinux.org 2011-08-06 22:26:24 MSK

Please share your task with the package in question.

Comment 4 viy 2011-08-06 22:37:17 MSK

(В ответ на комментарий №3)
> Please share your task with the package in question.

done
#51641 FAILED #1 [shared] sisyphus srpm=fonts-ttf-sil-gentium-basic-1.1-alt1_6.src.rpm

Comment 5 at@altlinux.org 2011-08-06 22:58:24 MSK

You also quote buildDir, but it's left unquoted in platform.in:
cd %{u2p:%_builddir}

Maybe the whole thing should be moved to %__build_pre:
cd %{u2p:%_builddir}%{?_buildsubdir}

This is not an immediate problem, though.

Comment 6 at@altlinux.org 2011-08-06 23:30:05 MSK

And now run the task.

Comment 7 Repository Robot 2011-08-06 23:58:41 MSK

rpm-4.0.4-alt100.30 -> sisyphus:

* Sat Aug 06 2011 Alexey Tourbin <at@altlinux> 4.0.4-alt100.30
- build.c, parsePrep.c: Quote buildSubidr in autogenerated shell
  scripts, to allow spaces (by Igor Vlasenko; closes: #25998).

Comment 8 viy 2011-08-07 00:00:09 MSK

thanks!

Comment 9 at@altlinux.org 2011-08-09 13:29:14 MSK

gtk2-themes-fishing-the-sky-0.3-alt1
        + cd 'fishing\ the\ sky'
        /usr/src/tmp/rpm-tmp.35796: line 97: cd: fishing\ the\ sky: No such file or directory

Comment 10 viy 2011-08-09 13:35:33 MSK

(В ответ на комментарий №9)
> gtk2-themes-fishing-the-sky-0.3-alt1
>         + cd 'fishing\ the\ sky'
>         /usr/src/tmp/rpm-tmp.35796: line 97: cd: fishing\ the\ sky: No such
> file or directory

fixed in gtk2-themes-fishing-the-sky-0.3-alt2.src.rpm

Comment 11 at@altlinux.org 2011-08-09 13:51:48 MSK

То есть не факт что стало лучше - кто-то полагался на старое поведение, для кторого был workaround. Но по крайней мере стало как в федоре.

Comment 12 viy 2011-08-09 13:56:55 MSK

(В ответ на комментарий №11)
> То есть не факт что стало лучше - кто-то полагался на старое поведение, для
> кторого был workaround. Но по крайней мере стало как в федоре.

у нас много устаревших отличий от современного rpm.
в новом rpm поддерживаются несколько флагов -f в %files,
секция %pre/%posttrans,
%patch -Fn option (be passed directly to patch)
unicode description and summary
и т.д.
руки не доходят.

Comment 13 Dmitry V. Levin 2011-08-09 14:03:05 MSK

(In reply to comment #11)
> То есть не факт что стало лучше - кто-то полагался на старое поведение, для
> кторого был workaround. Но по крайней мере стало как в федоре.

Ну да, с этим изменением совместимость по багам с федорой улучшилась на 1 :(