Bug 209384 - security/clamav: Update to 0.99.2
Summary: security/clamav: Update to 0.99.2
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Xin LI
URL: http://blog.clamav.net/2016/05/clamav...
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-08 20:39 UTC by Leonid Nevecherya
Modified: 2016-05-31 05:07 UTC (History)
6 users (show)

See Also:
bugzilla: maintainer-feedback? (garga)


Attachments
Update_to_0.99.2 (851 bytes, patch)
2016-05-14 21:56 UTC, Leonid Nevecherya
no flags Details | Diff
The patch was mangled, regenerated and verified with GPG (797 bytes, patch)
2016-05-15 07:22 UTC, Xin LI
no flags Details | Diff
Fix patch (776 bytes, patch)
2016-05-28 20:22 UTC, Vladimir Omelchuk
no flags Details | Diff
Fix patch (752 bytes, patch)
2016-05-29 07:22 UTC, Vladimir Omelchuk
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Leonid Nevecherya 2016-05-08 20:39:23 UTC
ClamAV 0.99.2 has been released, and is available for download at ClamAV.net's download site.

Note:  As previously discussed for the last three releases, we are no longer uploading ClamAV to SourceForge for release.  0.99.2 is the first release that is ONLY released on ClamAV.net

http://blog.clamav.net/2016/05/clamav-0992-has-been-released.html

Please, update port.
Comment 1 Leonid Nevecherya 2016-05-14 21:56:40 UTC
Created attachment 170293 [details]
Update_to_0.99.2

Changes:
 http://blog.clamav.net/2016/05/clamav-0992-has-been-released.html
 * 7z: fix for FolderStartPackStreamIndex array index heck
 * try to minimize the err cleanup path
 * clamunrar: notice if unpacking comment failed
 * libclamav: print raw certificate metadata
 * adding libfreshclam
 * Add 'cdb' datafile to sigtools list of datafile types.
 * filetype consistency
 * add other...
Comment 2 Xin LI freebsd_committer freebsd_triage 2016-05-15 07:22:18 UTC
Created attachment 170310 [details]
The patch was mangled, regenerated and verified with GPG

Please also merge to quarterly branch.
Comment 3 Vladimir Omelchuk 2016-05-28 20:22:23 UTC
Created attachment 170764 [details]
Fix patch

Fix patch, because:

# make
===>  License GPLv2 accepted by the user
===>  Found saved configuration for clamav-0.99
===>   clamav-0.99.2 depends on file: /usr/local/sbin/pkg - found
===> Fetching all distfiles required by clamav-0.99.2 for building
===>  Extracting for clamav-0.99.2
=> SHA256 Checksum mismatch for clamav-0.99.2.tar.gz.
===>  Refetch for 1 more times files:  clamav-0.99.2.tar.gz 
===>  License GPLv2 accepted by the user
===>  Found saved configuration for clamav-0.99
===>   clamav-0.99.2 depends on file: /usr/local/sbin/pkg - found
=> clamav-0.99.2.tar.gz doesn't seem to exist in /usr/ports/distfiles/.
=> Attempting to fetch http://www.clamav.net/downloads/production/clamav-0.99.2.tar.gz
clamav-0.99.2.tar.gz                          100% of   15 MB  472 kBps 00m33s
===> Fetching all distfiles required by clamav-0.99.2 for building
===>  License GPLv2 accepted by the user
===>  Found saved configuration for clamav-0.99
===>   clamav-0.99.2 depends on file: /usr/local/sbin/pkg - found
===> Fetching all distfiles required by clamav-0.99.2 for building
=> SHA256 Checksum mismatch for clamav-0.99.2.tar.gz.
===>  Giving up on fetching files:  clamav-0.99.2.tar.gz 
Make sure the Makefile and distinfo file (/usr/ports/security/clamav/distinfo)
are up to date.  If you are absolutely sure you want to override this
check, type "make NO_CHECKSUM=yes [other args]".
*** Error code 1

Stop.
make[2]: stopped in /usr/ports/security/clamav
*** Error code 1

Stop.
make[1]: stopped in /usr/ports/security/clamav
*** Error code 1

Stop.
make: stopped in /usr/ports/security/clamav
Comment 4 Leonid Nevecherya 2016-05-29 06:48:50 UTC
To Xin LI & Vladimir Omelchuk
Remove 
+TIMESTAMP = 146....
from your patches. People don't need know your local timestamp of clamav-0.99.2.tar.gz
Comment 5 Vladimir Omelchuk 2016-05-29 07:22:29 UTC
Created attachment 170778 [details]
Fix patch

(In reply to Leonid Nevecherya from comment #4)

Done.
Comment 6 commit-hook freebsd_committer freebsd_triage 2016-05-29 22:08:44 UTC
A commit references this bug:

Author: delphij
Date: Sun May 29 22:08:05 UTC 2016
New revision: 416126
URL: https://svnweb.freebsd.org/changeset/ports/416126

Log:
  Update to 0.99.2.

  PR:		ports/209384
  Submitted by:	Leonid Nevecherya <nevecherya gmail com>
  Approved by:	maintainer timeout
  MFH:		2016Q2

Changes:
  head/security/clamav/Makefile
  head/security/clamav/distinfo
Comment 7 commit-hook freebsd_committer freebsd_triage 2016-05-29 22:09:46 UTC
A commit references this bug:

Author: delphij
Date: Sun May 29 22:08:47 UTC 2016
New revision: 416128
URL: https://svnweb.freebsd.org/changeset/ports/416128

Log:
  MFH: r416126

  Update to 0.99.2.

  PR:		ports/209384
  Submitted by:	Leonid Nevecherya <nevecherya gmail com>
  Approved by:	ports-secteam

Changes:
_U  branches/2016Q2/
  branches/2016Q2/security/clamav/Makefile
  branches/2016Q2/security/clamav/distinfo
Comment 8 Xin LI freebsd_committer freebsd_triage 2016-05-29 22:10:01 UTC
Committed, thanks!
Comment 9 Jason Unovitch freebsd_committer freebsd_triage 2016-05-30 01:02:29 UTC
(In reply to Leonid Nevecherya from comment #4)
This is incorrect. The TIMESTAMPS are there for the build system to set to enable reproducible builds. They should be left in place on future updates.
Comment 10 Xin LI freebsd_committer freebsd_triage 2016-05-30 05:55:10 UTC
(In reply to Jason Unovitch from comment #9)
Ah good to know.  Thanks for pointing it out.

I think we should do it the next time the port is updated then.  Is the design discussed somewhere? (can't find some in-depth discussion or design docs, and I'm inclined to agree with Bryan's comments in the review).
Comment 11 Jason Unovitch freebsd_committer freebsd_triage 2016-05-30 23:41:07 UTC
(In reply to Xin LI from comment #10)
It's relatively new.  There's more info in the commit message at https://svnweb.FreeBSD.org/changeset/ports/415078 and review at  https://reviews.FreeBSD.org/D6031.  I don't recall see any formal announcements of the efforts however.
Comment 12 Leonid Nevecherya 2016-05-31 05:07:18 UTC
Ok. I understood my mistake. :)