Don't download the same files twice in one cycle.

debian/package.go

@@ -293,6 +293,20 @@ func (p *Package) DownloadList(packageRepo *Repository) (result [][]string, err
 	return result, nil
 }
 
+// VerifyFiles verifies that all package files have neen correctly downloaded
+func (p *Package) VerifyFiles(packageRepo *Repository) (result bool, err error) {
+	result = true
+
+	for _, f := range p.Files {
+		result, err = f.Verify(packageRepo)
+		if err != nil || !result {
+			return
+		}
+	}
+
+	return
+}
+
 // PackageCollection does management of packages in DB
 type PackageCollection struct {
 	db database.Storage

debian/package_test.go

@@ -40,10 +40,18 @@ func (s *PackageSuite) TestPackageFileVerify(c *C) {
 	c.Check(err, IsNil)
 	c.Check(result, Equals, false)
 
+	result, err = p.VerifyFiles(packageRepo)
+	c.Check(err, IsNil)
+	c.Check(result, Equals, false)
+
 	p.Files[0].Checksums.Size = 5
 	result, err = p.Files[0].Verify(packageRepo)
 	c.Check(err, IsNil)
 	c.Check(result, Equals, true)
+
+	result, err = p.VerifyFiles(packageRepo)
+	c.Check(err, IsNil)
+	c.Check(result, Equals, true)
 }
 
 func (s *PackageSuite) TestNewFromPara(c *C) {

debian/remote.go

@@ -186,14 +186,21 @@ func (repo *RemoteRepo) Download(d utils.Downloader, packageCollection *PackageC
 
 	// Download all package files
 	ch := make(chan error, list.Len())
+	queued := make(map[string]bool, 1024)
 	count := 0
 
 	err = list.ForEach(func(p *Package) error {
 		list, err := p.DownloadList(packageRepo)
 
 		for _, pair := range list {
-			d.Download(repo.PackageURL(pair[0]).String(), pair[1], ch)
-			count++
+			key := pair[0] + "-" + pair[1]
+			_, found := queued[key]
+			if !found {
+				d.Download(repo.PackageURL(pair[0]).String(), pair[1], ch)
+				count++
+			} else {
+				queued[key] = true
+			}
 		}
 
 		return err

debian/remote_test.go

@@ -144,8 +144,9 @@ func (s *RemoteRepoSuite) TestDownload(c *C) {
 	pkg, err := s.packageCollection.ByKey(s.repo.packageRefs.Refs[0])
 	c.Assert(err, IsNil)
 
-	// poolPath, _ := s.packageRepo.PoolPath(pkg.Filename, pkg.HashMD5)
-	// c.Check(pkg.VerifyFile(poolPath), Equals, true)
+	result, err := pkg.VerifyFiles(s.packageRepo)
+	c.Check(result, Equals, true)
+	c.Check(err, IsNil)
 
 	c.Check(pkg.Name, Equals, "amanda-client")
 }