comments from NEW review

[Sebastien Bacher]

I'm reviewing that package in the saucy NEW queue, some comments:

* the debian directory contains a "unity-china-video-scope.pkg" subdirectory that seems to be a copy of the debian directory, that seems wrong

* compat is "8" but control has a build-depends on debhelper (>= 9), you can bump the compat to 9

* you use valac-0.18, would be nicer to build-depends on "valac" so it follows the default version (current 0.20)

* debian/control has:

"# If you aren't a member of ~unity-team but need to upload packaging changes,
# just go ahead. ~unity-team will notice and sync up the code again."

* debian/copyright:
Copyright: <2012> <Pawel Stolowski <email address hidden>>

that should be "Copyright (C) 2012 Canonical Ltd"

None of those are blockers to get in saucy, I'm accepting the source, but it would be nice to fix those issues in the next upload

[Jack Yu (jackyu)]

Thanks. These comments will be included in the next release.

[shijing (shijing)]

All the bugs have fixed!

[Launchpad Janitor (janitor)]

This bug was fixed in the package unity-china-video-scope - 1.1-0ubuntu2

---------------
unity-china-video-scope (1.1-0ubuntu2) saucy; urgency=low

  * Fix several minor issues from new review (LP: #1206613)
 -- Jeremy Bicha <email address hidden> Thu, 29 Aug 2013 00:48:51 -0400