devel@ where you _can_ ask
 help / color / mirror / Atom feed
* [newbies] cbp2make
@ 2018-01-17 17:57 Vladimir D. Seleznev
  2018-01-17 18:15 ` Vladimir D. Seleznev
  0 siblings, 1 reply; 3+ messages in thread
From: Vladimir D. Seleznev @ 2018-01-17 17:57 UTC (permalink / raw)
  To: devel-newbies, alexey

Посмотрел на этот пакет [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

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


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-01-17 18:35 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-17 17:57 [newbies] cbp2make Vladimir D. Seleznev
2018-01-17 18:15 ` Vladimir D. Seleznev
2018-01-17 18:35   ` Vladimir D. Seleznev

devel@ where you _can_ ask

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-newbies/0 devel-newbies/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel-newbies devel-newbies/ http://lore.altlinux.org/devel-newbies \
		devel-newbies@lists.altlinux.org devel-newbies@lists.altlinux.ru devel-newbies@lists.altlinux.com
	public-inbox-index devel-newbies

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel-newbies


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git