Summary: | net-proxy/adzapper needs flush.pl (removed from dev-lang/perl-5.16.0) | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Alexandre <alexandre.guimaraes> |
Component: | [OLD] Development | Assignee: | Gentoo Network Proxy Developers (OBSOLETE) <net-proxy+disabled> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bpshacklett, perl, polidevk.polidevk |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 415485 | ||
Attachments: |
Convert flush.pl usage to IO::Handle flush
Update ebuild to include patch and use src_prepare Convert adzapper-20110915 to use IO::Handle adzapper-20110915-r1.ebuild.hacked adzapper-20110915-r1.ebuild.hacked.diff Update new ebuild to use patch The final edition of adzapper-20110915-r1.ebuild |
Description
Alexandre
2012-07-07 21:50:14 UTC
perldoc perl5120delta perldoc perl5160delta so not quite a bug - at least not in perl. Created attachment 317588 [details, diff]
Convert flush.pl usage to IO::Handle flush
This patch removes the use of flush.pl in adzapper and replaces it with IO::Handle's flush (and autoflush).
Created attachment 317590 [details, diff]
Update ebuild to include patch and use src_prepare
This patches the adzapper-20080508 ebuild to include the patch, it also changes it to use EAPI=4 so src_prepare can be used.
The patch for squid_redirect is perfect working here. ;-D Thanks Brennan Shacklett!!! Dear Brennan Shacklett, Can you make same thing with adzapper-20110915-r1, please? :) Created attachment 325354 [details, diff]
Convert adzapper-20110915 to use IO::Handle
The exact same patch essentially, they just changed the name of squid_redirect in the sources.
Created attachment 325366 [details]
adzapper-20110915-r1.ebuild.hacked
Dear Brennan Shacklett,
Can you review my hack of the official ebuild, please?
It seems to work, but my bash-scripting skills are non existent ;)
I'll upload "diff" also.
Created attachment 325368 [details]
adzapper-20110915-r1.ebuild.hacked.diff
diff adzapper-20110915-r1.ebuild.old adzapper-20110915-r1.ebuild.new >> adzapper-20110915-r1.ebuild.hacked.diff
Created attachment 325396 [details, diff]
Update new ebuild to use patch
All I had to do was add the inherit eutils line and the epatch line, I don't think you should need to mess around with $S or using cd to switch around in the sources. Here is the patch I used.
(In reply to comment #9) > Created attachment 325396 [details, diff] [details, diff] > Update new ebuild to use patch > > All I had to do was add the inherit eutils line and the epatch line, I don't > think you should need to mess around with $S or using cd to switch around in > the sources. Here is the patch I used. Yes! That was it! Thank you! I knew it was something simple ;) K.I.S.S. principle rules, as always! Created attachment 325408 [details]
The final edition of adzapper-20110915-r1.ebuild
Fixed in net-proxy/adzapper-20110915-r2 Thanks |