Mail archive
alpine-aports

Re: [alpine-aports] [PATCH] main/clamav: upgrade to 0.99. Minor fixes in APKBUILD, init and logrotate scripts

From: Natanael Copa <ncopa_at_alpinelinux.org>
Date: Mon, 7 Dec 2015 16:33:21 +0100

On Mon, 7 Dec 2015 14:36:46 +0200
Valery Kartel <valery.kartel_at_gmail.com> wrote:

> ---
> main/clamav/APKBUILD | 106 ++++++++++++++++++++--------------------
> main/clamav/clamd.initd | 3 +-
> main/clamav/clamd.logrotate | 11 ++---
> main/clamav/freshclam.initd | 13 ++++-
> main/clamav/freshclam.logrotate | 10 ++--
> 5 files changed, 75 insertions(+), 68 deletions(-)

...

> lib() {
> pkgdesc="ClamAV library"
> depends=""
> - replaces="clamav"
> + replaces="$pkgname"
> mkdir -p "$subpkgdir"/usr/lib
> mv "$pkgdir"/usr/lib/libclamav.so.* \
> "$subpkgdir"/usr/lib/
> _at_@ -98,7 +99,7 @@ lib() {
> libunrar() {
> pkgdesc="ClamAV unrar libraries"
> depends=""
> - replaces="clamav"
> + replaces="$pkgname"
> mkdir -p "$subpkgdir"/usr/lib
> mv "$pkgdir"/usr/lib/libclamunrar* \
> "$subpkgdir"/usr/lib/
> _at_@ -107,19 +108,18 @@ libunrar() {
> freshclam() {
> pkgdesc="Auto-updater for the Clam Antivirus scanner data-files"
> depends="logrotate"
> - replaces="clamav clamav-db"
> + replaces="$pkgname $pkgname-db"
> mkdir -p "$subpkgdir"/usr/bin \
> "$subpkgdir"/etc/init.d \
> "$subpkgdir"/etc/conf.d \


I don't like the replaces change to use $pkgname.

If the pkgname changes (to for example clamav1 or similar) then will we
automatically get a replaces="clamav1-db", which is completely useless
as there have never been any clamav1-db package.

The replaces="clamav1" in clamav-lib will also be meaningless.

We could probably remove the replaces since they were added to deal
with new organization of subpackages. Since then apk handles this kind
of things better. The replaces were there to solve issue when upgrade
for older version, but i dont think we support uppgrading from affected
versions at this point.

-nc


---
Unsubscribe:  alpine-aports+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-aports+help_at_lists.alpinelinux.org
---
Received on Mon Dec 07 2015 - 16:33:21 GMT