From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-3.2 required=5.0 tests=BAYES_00,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,RP_MATCHES_RCVD,T_DKIM_INVALID autolearn=unavailable autolearn_force=no version=3.4.1 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=altlinux.org; s=dkim; h=Subject:Content-Transfer-Encoding:Content-Type: MIME-Version:Message-ID:To:From:Date:Sender:Reply-To:Cc:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:In-Reply-To:References:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=USV02b35AbEdxG8+AXmeMIFiJgzyaxtUF2FM7h5MZE4=; b=xWXYxDm38pF4Z22sJHsEWUCSDb tGtw8BSWKI9gJvxvSsx5U1oYvooISIjGKkXcG9Sc8cQdpNb62ZoV4oZnwsBAjWSvE4Fakt4d/vP/4 bUZ7WnpZufdOSg0i8rJJn74wVSvV+ZdfOZXGxl74aPtr3ZtlpXmZ3lcjQW7k8fDqa6CD3S5gbFKXb K7TFnGWaV3Megxt/5FKDyCelLTUOLiRiARf7lA2KJqI6Y+qmIco7PJyv4p9IzuGzd50ZDgLnjByuz OoCb9P0l8X8IPlJ+dy6+BgM2bfYn37pQSFymwDrLQ8UUll82H1Mle7TFD3D/f/m7pYtRhLb3HS1uT qYWeBYOw==; Date: Mon, 22 Jan 2018 20:19:06 +0300 From: "Vladimir D. Seleznev" To: devel-newbies@lists.altlinux.org, mvoronov@altlinux.org, zerg@altlinux.org Message-ID: <20180122171906.GA7902@portlab> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit User-Agent: Mutt/1.8.3 (2017-05-23) X-SA-Exim-Connect-IP: 93.191.18.90 X-SA-Exim-Mail-From: vseleznv@cs.msu.ru X-SA-Exim-Version: 4.2 X-SA-Exim-Scanned: Yes (on mail.cs.msu.ru) Subject: [newbies] orion X-BeenThere: devel-newbies@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: "devel@ where you _can_ ask" List-Id: "devel@ where you _can_ ask" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 22 Jan 2018 17:19:09 -0000 Archived-At: List-Archive: Доброго времени суток! Поревьюил этот репозиторий [1]. Пара замечаний/вопросов. 1. Почему решили собирать версию 1.5.1? В репозитории есть 1.6.1, не лучше ли собрать её? 2. Шапка текущего спека намешана, лучше привести его к виду как здесь [2] (поле Packager добавлять не обязательно, и в общем случае не нужно). 3. В текущем спеке используются лишние конструкции, рекомендую прогнать его с помощью cleanup_spec из пакета rpm-utils. 4. Здесь в поле Source используется путь до тарбола, находящимся на внешнем ресурсе. Мы так не делаем, когда собираем пакеты из gear-репозитория: всё-равно мы пакуем свои тарболы, сгенерённые из тега. По этой же причине мы пакуем тарболы не в tar.gz, а в tar: чтобы избежать коллизий с апстримными тарболами, плюс смысла в этом большого нет, т.к. sourcerpm и так сжимается гораздо более сильным сжатием. Лучше переделать схему сжатия на tar, и убрать url из поля Source. -- [1] http://git.altlinux.org/people/mvoronov/packages/orion.git [2] https://www.altlinux.org/SampleSpecs/empty [3] http://git.altlinux.org/people/mvoronov/packages/?p=orion.git;a=blob;f=orion.spec;h=76ff892338659b42490c191be9e2a6430bd6168f;hb=27ecc271e8ee6ec28b39cbcb7b9e096e8eff25fd#l8 -- С уважением, Владимир Селезнев