Re: [devel] [git update] packages/rpm: heads/mod

Re: [devel] [git update] packages/rpm: heads/mod

[Dmitry V. Levin]

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

On Fri, Sep 21, 2007 at 05:05:07PM +0400, Alexey M. Tourbin wrote:
[...]
>  	if [ -n "${RPM_BUILD_ROOT-}" ] && [ -z "${rep##$RPM_BUILD_ROOT*}" ]; then

Этот код не будет работать правильно при пустом значении RPM_BUILD_ROOT.
Лучше заменить ${RPM_BUILD_ROOT-} на ${RPM_BUILD_ROOT:-}.


-- 
ldv

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

Re: [devel] [JT] [git update] packages/rpm: heads/mod

[Damir Shayhutdinov]

> >       if [ -n "${RPM_BUILD_ROOT-}" ] && [ -z "${rep##$RPM_BUILD_ROOT*}" ]; then
>
> Этот код не будет работать правильно при пустом значении RPM_BUILD_ROOT.
> Лучше заменить ${RPM_BUILD_ROOT-} на ${RPM_BUILD_ROOT:-}.

Коллеги, если используете смайлики в шелл-скриптах, не забывайте
пририсовывать им глазки! :-}

Re: [devel] [JT] [git update] packages/rpm: heads/mod

[Led]

В сообщении от Friday 21 September 2007 17:12:12 Damir Shayhutdinov 
написал(а):
> > >       if [ -n "${RPM_BUILD_ROOT-}" ] && [ -z "${rep##$RPM_BUILD_ROOT*}"
> > > ]; then
> >
> > Этот код не будет работать правильно при пустом значении RPM_BUILD_ROOT.
> > Лучше заменить ${RPM_BUILD_ROOT-} на ${RPM_BUILD_ROOT:-}.
>
> Коллеги, если используете смайлики в шелл-скриптах, не забывайте
> пририсовывать им глазки! :-}

Однозначно - в fortunes!:)

-- 
Led

[devel] [JT] fortunes-ALT-devel! Re: [JT] [git update] packages/rpm: heads/mod [JT]

[Michael Shigorin]

On Fri, Sep 21, 2007 at 05:21:05PM +0300, Led wrote:
> > > >       if [ -n "${RPM_BUILD_ROOT-}" ] && [ -z "${rep##$RPM_BUILD_ROOT*}"
> > > > ]; then
> > > Этот код не будет работать правильно при пустом значении RPM_BUILD_ROOT.
> > > Лучше заменить ${RPM_BUILD_ROOT-} на ${RPM_BUILD_ROOT:-}.
> > Коллеги, если используете смайлики в шелл-скриптах, не
> > забывайте пририсовывать им глазки! :-}
> Однозначно - в fortunes!:)

А толку, fortunes-ALT никто не занимается :(

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] [git update] packages/rpm: heads/mod

[Alexey Tourbin]

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

On Fri, Sep 21, 2007 at 05:37:50PM +0400, Dmitry V. Levin wrote:
> On Fri, Sep 21, 2007 at 05:05:07PM +0400, Alexey M. Tourbin wrote:
> [...]
> >  	if [ -n "${RPM_BUILD_ROOT-}" ] && [ -z "${rep##$RPM_BUILD_ROOT*}" ]; then
> 
> Этот код не будет работать правильно при пустом значении RPM_BUILD_ROOT.

Обоснуй.

> Лучше заменить ${RPM_BUILD_ROOT-} на ${RPM_BUILD_ROOT:-}.

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

Re: [devel] [git update] packages/rpm: heads/mod

[Dmitry V. Levin]

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

On Fri, Sep 21, 2007 at 06:24:46PM +0400, Alexey Tourbin wrote:
> On Fri, Sep 21, 2007 at 05:37:50PM +0400, Dmitry V. Levin wrote:
> > On Fri, Sep 21, 2007 at 05:05:07PM +0400, Alexey M. Tourbin wrote:
> > [...]
> > >  	if [ -n "${RPM_BUILD_ROOT-}" ] && [ -z "${rep##$RPM_BUILD_ROOT*}" ]; then
> > 
> > Этот код не будет работать правильно при пустом значении RPM_BUILD_ROOT.
> 
> Обоснуй.

OK, будет и так работать.


-- 
ldv

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

Re: [devel] [git update] packages/rpm: heads/mod

[Alexey Tourbin]

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

On Fri, Sep 21, 2007 at 06:59:03PM +0400, Dmitry V. Levin wrote:
> On Fri, Sep 21, 2007 at 06:24:46PM +0400, Alexey Tourbin wrote:
> > On Fri, Sep 21, 2007 at 05:37:50PM +0400, Dmitry V. Levin wrote:
> > > On Fri, Sep 21, 2007 at 05:05:07PM +0400, Alexey M. Tourbin wrote:
> > > [...]
> > > >  	if [ -n "${RPM_BUILD_ROOT-}" ] && [ -z "${rep##$RPM_BUILD_ROOT*}" ]; then
> > > 
> > > Этот код не будет работать правильно при пустом значении RPM_BUILD_ROOT.
> > 
> > Обоснуй.
> 
> OK, будет и так работать.

Да это же по сути код
	if [ -n "$RPM_BUILD_ROOT" ] && [ -z "${rep##$RPM_BUILD_ROOT*}" ]; then
	...

в котором при первом использовании RPM_BUILD_ROOT добавлен минус,
потому что скрипты работают в режиме 'sh -u'.  Минус означает что
"я действительно знаю, что такой переменной может и не быть, это
не опечатка".

Кстати, код
	if [ -n "${RPM_BUILD_ROOT-}" -a -z "${rep##$RPM_BUILD_ROOT*}" ]; then
	...
в этом отношении уже не проканает, потому что с точки зрения шелла это
одна и та же команда, и если первый минус "гасит" -u, то во втором
RPM_BUILD_ROOT минуса уже нет, а подстановка происходит всех переменных
СРАЗУ (до интерпретации), и шелл по идее должен вывалиться.

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