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:In-Reply-To:Content-Transfer-Encoding: Content-Type:MIME-Version:References:Message-ID:Cc:To:From:Date:Sender: Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender :Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=KlaYALNQPYUmk7iCh6Ypi49QB+Rs07XixPnLki8zkMM=; b=sPPXIMYejM81utHRnj2HZGpPyl IER2H/o0ROU11u3CappkOUnXSl0lTCUReQGOOOqrypxOysdd0TthiFU00bStl7/ceuv9nK1fzhsg9 uJKG4hW1AqoNsy0D456ZpWqIjci5i0MI54StCV+idkltOhSQR+Jo1UU/b2HkPyHyCbpJNhJGpRsMz N7/s44vRABx2ojT1J9X82mK73ScqyN+ORhzKkGb9v9ulTi9foTZuTBENN2/MobsmxWI5e9SstQGwf 6B0AFgwAEETImwEoZQf+AD4MAiWRCZJLUodYQHTWSJS4/Pa3bW34Hjcv5TbYj0rOxNgLN+pyg07Xe /zE8Pa7g==; Date: Sat, 27 Jan 2018 11:47:19 +0300 From: "Vladimir D. Seleznev" To: Maxim Voronov Message-ID: <20180127084719.GB13704@portlab> References: <20180122171906.GA7902@portlab> <2749878.XKQiG4duR1@zerg.malta.altlinux.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.8.3 (2017-05-23) X-SA-Exim-Connect-IP: 95.129.140.49 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) Cc: darktemplar@basealt.ru, mvoronov@altlinux.org, devel-newbies@lists.altlinux.org, Sergey V Turchin Subject: Re: [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: Sat, 27 Jan 2018 08:47:22 -0000 Archived-At: List-Archive: On Tue, Jan 23, 2018 at 11:38:13AM +0300, Maxim Voronov wrote: > Обновил репозиторий согласно рекомендациям. > > On 23 Jan 2018 9:20 am, "Sergey V Turchin" wrote: > > > On Monday, 22 January 2018 20:19:06 MSK Vladimir D wrote: > > > Доброго времени суток! > > > > > > Поревьюил этот репозиторий [1]. Пара замечаний/вопросов. > > > > > > 1. Почему решили собирать версию 1.5.1? В репозитории есть 1.6.1, не > > > лучше ли собрать её? > > Это моя идея. После отправки в p8 в Сизиф соберем новую, т.к. зависит на > > версию Qt. > > > > > 2. Шапка текущего спека намешана, лучше привести его к виду как здесь [2] > > > (поле Packager добавлять не обязательно, и в общем случае не нужно). > > Или можно на мои спеки смотреть. Там похоже. > > > > > 3. В текущем спеке используются лишние конструкции, рекомендую прогнать > > > его с помощью cleanup_spec из пакета rpm-utils. > > Так же cleanup_spec полезно делать сразу, когда берете spec-рыбу из другого > > дистрибутива. > > > > > 4. Здесь в поле Source используется путь до тарбола, находящимся на > > > внешнем ресурсе. Мы так не делаем, когда собираем пакеты из > > > gear-репозитория: всё-равно мы пакуем свои тарболы, сгенерённые из тега. > > Да. Интернет-путь -- анахронизм. > > > > > По этой же причине мы пакуем тарболы не в tar.gz, а в tar: чтобы > > > избежать коллизий с апстримными тарболами, плюс смысла в этом большого > > > нет, т.к. sourcerpm и так сжимается гораздо более сильным сжатием. > > Скорее, второе. Еще и жать сжатое всё равно будет. > > > > > Лучше переделать схему сжатия на tar, и убрать url из поля Source. > > Да. Если спек был взят явно из другого проекта, то надо бы оставить как минимум упоминание об этом с указанием откуда. Но это на усмотрение ментора. -- С уважением, Владимир Селезнев