[Neurodebian-devel] OpenSesame 0.27

[Yaroslav Halchenko]

On Tue, 11 Dec 2012, Sebastiaan Mathot wrote:
> Well, I guess that depends: How much nitpicking are we talking about here? ;-) 

I don't think that much since lintian is happy ;)


> I leave it up to you. If you think we can get OpenSesame into official debian 
> within a reasonable timeframe and without excessive effort than I'm all for 
> it. Otherwise NeuroDebian only is fine for now.

so let's start and then decide ;)

debian/changelog

*

  opensesame (0.27-nd1) unstable; urgency=low

  make version 0.27-1 (I am surprised lintian didn't complain)

  -1 signals debian revision, 0.27 is your upstream release, and those
  ~nd suffixes will be added automagically by our build script

*
  There is an open ITP #611640 for opensesame -- close it in this last
  changelog entry (i.e. add "(Closes: #611640)")

  aha -- I see it in the original Michael's entry, but since that one was never
uploaded, then still above advice holds -- just that Closes up to the last
one

* ideally all previous changelog entries should get 'UNRELEASED' instead
  of 'unstable' where no Debian package was provided and may be
  only 'neurodebian' for 0.25-1 which we still carry in neurodebian

  so later on looking at changelog becomes clear what version was
  uploaded where


debian/copyright

* s/Maintainer-Contact/Upstream-Contact/

  see http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
  you reference ;)


 there is a handy alias I usually use:

$> alias dep5_verify  
dep5_verify='cme check dpkg-copyright'

$> dlocate /usr/bin/cme
libconfig-model-perl: /usr/bin/cme

which gives quick check and helps to catch typos filtered out by my
vision: ;)


$> dep5_verify 
Warning in 'Format' value 'Format: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/': Format does not match the recommended URL for DEP-5
Configuration item 'Debian::Dpkg::Copyright Format' has a wrong value:
    value 'Format: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/' does not match regexp ^http://


debian/control

* 

Vcs-Git: git://github.com:smathot/OpenSesame.git


is incorrect, should have / instead of : in the middle

also provide Vcs-Git-Browser: field pointint to http page

* XB-Python-Version:  iirc not needed anylonger

*

  Recommends: python-serial (>= 2.3~), psychopy (>= 1.64.0),

  just so that you know (i.e. probably do not fix since I do consider them
useful) -- those versioned dependencies are not enforced anyhow in Recommends
AFAIK because those are optional dependencies.  if you wanted to make
sure that  there is no older than compatible version of psychopy, conflict with it, i.e.

Conflicts: psychopy (<< 1.64.0~)

BINARY package:

run lintian on resultant .changes as well

$ lintian opensesame_0.27-nd1_amd64.changes
W: opensesame: binary-without-manpage usr/bin/opensesame
W: opensesame: binary-without-manpage usr/bin/opensesamerun
W: opensesame: executable-not-elf-or-script usr/share/opensesame/resources/theme/gnome/os-custom-icons/index.theme
W: opensesame: executable-not-elf-or-script usr/share/opensesame/resources/theme/default/os-custom-icons/index.theme


so -- adjust permissions (probably just in debian/rules)

ideally there should be manpages... in your case their generation is very simple:

help2man opensesamerun 

;)  -- just give that call necessary tune up arguments... probably just
pre-generate since otherwise DISPLAY is needed (which could also be used by the
way via xvfb-run -- see e.g. ipython package which I believe uses it)


and then I guess it could also be uploaded to Debian proper ;)


> Cheers!
CHEERS!


-- 
Yaroslav O. Halchenko
Postdoctoral Fellow,   Department of Psychological and Brain Sciences
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834                       Fax: +1 (603) 646-1419
WWW:   http://www.linkedin.com/in/yarik