From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <php-coder@ngs.ru>
Date: Sat, 26 Mar 2005 19:09:47 +0600
From: php-coder <php-coder@ngs.ru>
X-Mailer: The Bat! (v1.61)
X-Priority: 3 (Normal)
Message-ID: <146148960.20050326190947@ngs.ru>
To: ALT Linux Sisyphus discussion list <sisyphus@altlinux.ru>
Subject: =?koi8-r?B?UmU6IFtzaXN5cGh1c10gUmU6IOvByyDOxSDVxMHM0dTYINDBy8XUID8gKNPJ?=
	=?koi8-r?B?zsHQ1MnLLCAtLW5vZGVwcyk=?=
In-Reply-To: <1111828934.10489.7.camel@alpha.fastlink.ru>
References: <d22bdi$rpu$1@sea.gmane.org> <20050326082004.GL17512@osdn.org.ua>
	<1111828934.10489.7.camel@alpha.fastlink.ru>
MIME-Version: 1.0
Content-Type: text/plain; charset=koi8-r
Content-Transfer-Encoding: 8bit
X-BeenThere: sisyphus@altlinux.ru
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: ALT Linux Sisyphus discussion list <sisyphus@altlinux.ru>
List-Id: ALT Linux Sisyphus discussion list <sisyphus.altlinux.ru>
List-Unsubscribe: <https://lists.altlinux.ru/mailman/listinfo/sisyphus>,
	<mailto:sisyphus-request@altlinux.ru?subject=unsubscribe>
List-Archive: <http://lists.altlinux.ru/pipermail/sisyphus>
List-Post: <mailto:sisyphus@altlinux.ru>
List-Help: <mailto:sisyphus-request@altlinux.ru?subject=help>
List-Subscribe: <https://lists.altlinux.ru/mailman/listinfo/sisyphus>,
	<mailto:sisyphus-request@altlinux.ru?subject=subscribe>
X-List-Received-Date: Sat, 26 Mar 2005 13:04:59 -0000
Archived-At: <http://lore.altlinux.org/sisyphus/146148960.20050326190947@ngs.ru/>
List-Archive: <http://lore.altlinux.org/sisyphus/>


--- Вячеслав Диконов 2005-03-26 12:22:14 +0300
+++ php-coder 2005-03-26 16:55:32 +0600

    /me любит разглядывать чужие спеки :))

    Можно пару рекомендаций? Просто посоветую..

    + BuildRequires неплохо бы разбить на несколько строк, так
      читабельность повысится?

    + BuildRequires сгенерирован при помощи buildreq? Есть подозрение,
      что нет: во-первых, скрипт бы добавил свой "фирменный" комментарий,
      а, во-вторых, не хватает одной зависимости :)

      Там в спеке используется convert, которая находится в пакете
      ImageMagick. В BuildRequires я его что-то не увидел? Или, быть может,
      convert из Sisyphus'а находится в другом пакете?

    + s/rm -fr/%__rm -rf/g

    + s/ln -sf/%__ln_s -f/g

    + s/mkdir -p/%__mkdir_p/g
        
    + s/$RPM_BUILD_ROOT/%buildroot/g

> ?package(%name): command="%name" [skip]

    А тут IMHO лучше написать command="%_bindir/%name", т.к. может быть
    установлена эта же программа в /usr/local и в таком случае запустится
    именно она...

> %_bindir/%name

    %_bindir/*

> %_mandir/man1/*

    %_man1dir/*

> %_menudir/%name

    %_menudir/*

> %_liconsdir/%name.png
> %_iconsdir/%name.png
> %_miconsdir/%name.png

    %_miconsdir/*
    %_liconsdir/*
    %_iconsdir/%name.png

    + А где тег Packager?

    Эээ... надеюсь, вы не воспримите мои советы, как критику. Просто хотел
    помочь. Я всегда в своих спеках использую макросы где это только возможно.
    И как я понял использование $RPM_BUILD_ROOT тоже нежелательно, так как
    есть макрос %buildroot Я правильно понимаю?

    Удачи!

-- 
+ php-coder
+ php-coder[at]ngs[dot]ru
+ ALTLinux Master 2.4 (kernel 2.4.26-std-up-alt6)