[newbies] cbp2make

[newbies] cbp2make

[Vladimir D. Seleznev]

Посмотрел на этот пакет [1].

Замечания.

  # https://sourceforge.net/projects/cbp2make/files/cbp2make-stl-rev147-all.tar.7z/download
  Source: %name-%version.tar

Здесь в пути, откуда был взял тарбол, необходимо заменить версию на
макрос %version. Другое дело, что 147 — это не версия, возможно, стоило
в качестве версии использовать rev147, или 0+rev147, но тогда следовало
использовать конструкцию вида

  %define rev rev147
  ...
  Version: 0+%rev
  ...
  # repacked ...cbp2make-stl-%rev-all.tar.7z/download
  Source: %name-%version.tar



  %build
  find bin/ -type f -delete
  find dox/ -type f -delete

не понятно, зачем использовать такую странную конструкцию. Лучше
использовать

  rm -rf bin/ dox/

и по смыслу это должно относиться к секции %prep


  %install
  mkdir -p %buildroot%_bindir
  mkdir -p %buildroot%_docdir/%name-%version/dox/
  cp bin/Release/%name %buildroot%_bindir
  cp COPYING %buildroot%_defaultdocdir/%name-%version/
  cp changelog.txt %buildroot%_defaultdocdir/%name-%version/
  cp usage.txt %buildroot%_defaultdocdir/%name-%version/
  cp -r dox/html/ %buildroot%_defaultdocdir/%name-%version/dox/html/

не используйте cp(1), используйте install(1) для установки файлов, с
нужными правами и пр.. Но в случае документации её не надо копировать в
%_defaultdocdir в секции %install, вместо этого нужно в секции %files
использовать директиву %doc с перечислением нужных файлов и каталогов.


Поле Packager не нужно.

Я предлагаю когда будет время подготовить следующий релиз с исправлением
замечаний в тестовом задании, и дать на ревью.

-- 

[1] http://git.altlinux.org/gears/c/cbp2make.git?p=cbp2make.git;a=blob;f=cbp2make.spec;h=e7d7bca0915bb340b0d2c4a30ae0b49d8d818c56;hb=cdc6173038834923f881ce48e73e3f83e6acc57d

-- 
   С уважением,
   Владимир Селезнев

Re: [newbies] cbp2make

[Vladimir D. Seleznev]

On Wed, Jan 17, 2018 at 08:57:34PM +0300, Vladimir D. Seleznev wrote:
> Но в случае документации её не надо копировать в
> %_defaultdocdir в секции %install, вместо этого нужно в секции %files
> использовать директиву %doc с перечислением нужных файлов и каталогов.

перечислением нужных файлов. /* fixed */

-- 
   С уважением,
   Владимир Селезнев

Re: [newbies] cbp2make

[Vladimir D. Seleznev]

On Wed, Jan 17, 2018 at 09:15:13PM +0300, Vladimir D. Seleznev wrote:
> On Wed, Jan 17, 2018 at 08:57:34PM +0300, Vladimir D. Seleznev wrote:
> > Но в случае документации её не надо копировать в
> > %_defaultdocdir в секции %install, вместо этого нужно в секции %files
> > использовать директиву %doc с перечислением нужных файлов и каталогов.
> 
> перечислением нужных файлов. /* fixed */

перечислением нужных файлов и каталогов. /* reverted */

-- 
   С уважением,
   Владимир Селезнев