Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 189939 - net-firewall/arptables-0.0.3-3 version bump
Summary: net-firewall/arptables-0.0.3-3 version bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Peter Volkov (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 162886
  Show dependency tree
 
Reported: 2007-08-23 13:46 UTC by Olivier Huber
Modified: 2007-08-30 20:59 UTC (History)
1 user (show)

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


Attachments
My ebuild proposal (arptables-0.0.3.3.ebuild,1.20 KB, text/plain)
2007-08-23 13:47 UTC, Olivier Huber
Details
patch needed to install the package (arptables-0.0.3.3-scripts-build.patch,655 bytes, patch)
2007-08-23 13:48 UTC, Olivier Huber
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Olivier Huber 2007-08-23 13:46:34 UTC
upstream has released  a new version. It builds fine with a 2.6.22 kernel. I try to update the ebuild a little bit. And by the way pva thanks for your advice with tc-getCC.

Reproducible: Always
Comment 1 Olivier Huber 2007-08-23 13:47:53 UTC
Created attachment 128955 [details]
My ebuild proposal
Comment 2 Olivier Huber 2007-08-23 13:48:56 UTC
Created attachment 128956 [details, diff]
patch needed to install the package
Comment 3 Peter Volkov (RETIRED) gentoo-dev 2007-08-29 11:36:59 UTC
arptables-0.0.3.3 is in portage.

Oliver, just few notes on your (and mine) ebuild:
* $WORKDIR should be quoted as it is user defined and may contain spaces there.
* I did not understood why do you need multilib eclass there
* To install package I just go with dobin and doman functions as there are only three files to be installed and no patches/sed required, but of course your solution works too.
* Next time attach into bugzilla diff of ebuild. This clearly tells what chaged and thus easier to review
Comment 4 Olivier Huber 2007-08-30 20:59:16 UTC
> Oliver, just few notes on your (and mine) ebuild:
> * $WORKDIR should be quoted as it is user defined and may contain spaces there.
> * I did not understood why do you need multilib eclass there

heu ... it was just a mistake : I copy/paste the inherit arguments too fast, without checking if I need all of them

> * To install package I just go with dobin and doman functions as there are only
> three files to be installed and no patches/sed required, but of course your
> solution works too.

sure, it was a little bit overkill here

> * Next time attach into bugzilla diff of ebuild. This clearly tells what chaged
> and thus easier to review
>
ok I'll do that next time

Thanks again