Re: Please criticize the package I have been working on, https://github.com/pelliott80/simplescreenrecorder-dpm.git

2015-03-20 Thread Riley Baird
  d/changelog:
  +You should only list versions of the package that will actually be in
  Debian, so get rid of the vivid entry.
  +The changes you make before the initial release generally aren't
  important enough to go in the changelog.
  +Generally, new packages should have priority low.
  *Close the ITP bug.
 
 Yes, but remember I don't own this bug, I want to
 show these change to upstream and the bug owner.
 Perhaps they will clone, or pull these changes.
 
 If I get it working, let them have the fun of
 closing the bug.

Sorry, I wasn't clear enough. What I meant was that the bug should be
closed automatically by an entry in d/changelog. A lintian tag
new-package-should-close-itp-bug has been created for this purpose.

You can close the bug by including something like:

* Initial release (Closes: #)

In fact, this is really the only entry that you should have in your
changelog for a new package.

  
  d/control:
  *Where is simplescreenrecorder-lib?
 
 This is a very important point. Usually, libs
 are more difficult... But often initial version
 is not bad. it is a good idea to insure easy update.

I'm not sure if you can do this, but I'm not sure. Perhaps someone else
on the list can advise you whether or not this is allowed?


pgpT9REQq5xFk.pgp
Description: PGP signature


Re: Please criticize the package I have been working on, https://github.com/pelliott80/simplescreenrecorder-dpm.git

2015-03-14 Thread Paul Elliott
On Sat, Mar 14, 2015 at 07:15:17AM +1100, Riley Baird wrote:
 On Fri, 13 Mar 2015 08:27:23 -0500




 d/changelog:
 +You should only list versions of the package that will actually be in
 Debian, so get rid of the vivid entry.
 +The changes you make before the initial release generally aren't
 important enough to go in the changelog.
 +Generally, new packages should have priority low.
 *Close the ITP bug.

Yes, but remember I don't own this bug, I want to
show these change to upstream and the bug owner.
Perhaps they will clone, or pull these changes.

If I get it working, let them have the fun of
closing the bug.



 
 d/control:
 *Where is simplescreenrecorder-lib?

This is a very important point. Usually, libs
are more difficult... But often initial version
is not bad. it is a good idea to insure easy update.

 
 d/copyright:
 *DEP-5 convention is to use GPL-3+ instead of GPL-3.0+
 *Remove the  and  signs from the Source: field.
 *Include the licensing information for the build files like aclocal.m4,
 configure, m4/* and build-aux/*
 *There are some files in glinject/ that are MIT licensed.

OK


-- 
Paul Elliott   1(512)837-1096
pelli...@blackpatchpanel.com   PMB 181, 11900 Metric Blvd Suite J
http://www.free.blackpatchpanel.com/pme/   Austin TX 78758-3117


signature.asc
Description: Digital signature


Re: Please criticize the package I have been working on, https://github.com/pelliott80/simplescreenrecorder-dpm.git

2015-03-12 Thread Riley Baird
On Thu, 12 Mar 2015 16:42:36 -0500
Paul Elliott pelli...@blackpatchpanel.com wrote:
 Please criticize the package I have been working on
 https://github.com/pelliott80/simplescreenrecorder-dpm.git
 in a git repo in dpm format.

Okay, sure. I'm not a DD, so I can't sponsor your package, but I
thought that I'd take a look anyway.

d/changelog:
-Make sure that you use the Debian names, not the Ubuntu names (e.g.
experimental, not vivid)
-Your changelog needs to close the ITP bug. A good entry would be:
 * Initial release (Closes: #)

d/control:
-The latest Standards-Version is 3.9.6
-Wrap your lines if they go over 80 characters
-Vcs-Git and Vcs-Browser refer to the VCS of the Debian packaging, not
of upstream development
-Is there any reason that you are limited to i386 and amd64?
-Where is simplescreenrecorder-lib?

d/copyright:
-DEP-5 convention is to use GPL-3+ instead of GPL-3.0+
-Remove the  and  signs from the Source: field.
-Include the licensing information for the build files like aclocal.m4,
configure, m4/* and build-aux/*
-There are some files in glinject/ that are MIT licensed.

d/postinst:
-What is the point of this file? If you don't need it, remove it.

general:
-Did Maarten Baert (v2) make the Debian package? In any case, if it's
your package, you should be listed as the maintainer.

Good luck getting your package into Debian,

Riley Baird


pgpp4cfI0Nj4x.pgp
Description: PGP signature


Please criticize the package I have been working on, https://github.com/pelliott80/simplescreenrecorder-dpm.git

2015-03-12 Thread Paul Elliott
Please criticize the package I have been working on
https://github.com/pelliott80/simplescreenrecorder-dpm.git
in a git repo in dpm format.

I am not the owner of the ITP bug, but the owner knows
I am working on it. I plan to submit this work to
the ITP bug owner.

I want to get it in perfect shape for debian.
Package now has no lintian errors, except for lack of PGP
signatures, and that is an upstream problem.

Best wishes to all, and thank you.
-- 
Paul Elliott   1(512)837-1096
pelli...@blackpatchpanel.com   PMB 181, 11900 Metric Blvd Suite J
http://www.free.blackpatchpanel.com/pme/   Austin TX 78758-3117


signature.asc
Description: Digital signature