472788 – app-antivirus/clamav-0.97.7: PATCHES not applied

Bug 472788 - app-antivirus/clamav-0.97.7: PATCHES not applied

Summary: app-antivirus/clamav-0.97.7: PATCHES not applied

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Net-Mail Packages

URL:
Whiteboard:
Keywords:	PATCH

Depends on:
Blocks:

Reported:	2013-06-09 20:38 UTC by Bertrand Jacquin
Modified:	2013-06-12 21:33 UTC (History)
CC List:	2 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
build.log (build.log,248.73 KB, text/x-log) 2013-06-09 20:40 UTC, Bertrand Jacquin	Details
clamav-0.97.7.ebuild-src_prepare.diff (clamav-0.97.7.ebuild-src_prepare.diff,502 bytes, patch) 2013-06-09 20:42 UTC, Bertrand Jacquin	Details \| Diff
emerge --info (info.log,12.97 KB, text/x-log) 2013-06-09 20:44 UTC, Bertrand Jacquin	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Bertrand Jacquin 2013-06-09 20:38:34 UTC

When trying to install app-antivirus/clamav-0.97.7, patches defined in PATCHES=() array are not applied, see build.log

Also, emerge --info is attached.

A patch proposal is attached also.

Reproducible: Always

Steps to Reproduce:
1. emerge -vat clamav::gentoo
2.
3.

Comment 1 Bertrand Jacquin 2013-06-09 20:40:11 UTC

Created attachment 350572 [details]
build.log

Comment 2 Bertrand Jacquin 2013-06-09 20:42:03 UTC

Created attachment 350574 [details, diff]
clamav-0.97.7.ebuild-src_prepare.diff

Patch proposal to inherit base eclass and so use base src_prepare func. Not sure this is the right way to do this.

Comment 3 Bertrand Jacquin 2013-06-09 20:44:28 UTC

Created attachment 350576 [details]
emerge --info

Comment 4 Tim Harder gentoo-dev

2013-06-12 21:15:52 UTC

Fixed in CVS. Generally it's best to just use epatch (until PATCHES functionality is implemented in a future EAPI) since we're trying to move away from the base eclass.

Comment 5 Bertrand Jacquin 2013-06-12 21:33:29 UTC

(In reply to Tim Harder from comment #4)
> Fixed in CVS. Generally it's best to just use epatch

And more readable also :) Thanks Tim