Bug 59902 – [Patch] Add epatch check to repoman

• Status: RESOLVED WONTFIX
• Product: Portage Development

[Description Danny van Dyk (RETIRED) 2004-08-09 14:56:53 UTC]

[Comment 1 Brian Harring (RETIRED) 2004-08-09 15:02:28 UTC]

err... kugelfang?

[Comment 2 Danny van Dyk (RETIRED) 2004-08-09 15:05:05 UTC]

Created attachment 37123 [details, diff]
Patch for repoman to check epatch lines

[Sorry, the keyboard has fallen down :-/, here goes the real "bug"]

Anyway: I already talked w/ jstubbs about this patch, it allows repoman to
check
all occurences of epatch in an ebuild. If it detects env. variables in the
argument to epatch,  it resolves them. (not all, up to now only FILESDIR, P,
PV,
PN) If the patchfile is not found in at its place, repoman fails with
"epatch.missing".  If there are variables in epatch which are unknown to
repoman, it still raises a warning (epatch.unknown).

I'd really like to have this function in portage, as ppl who do version bumps
sometimes just overlook arch dependent epatch lines :-/ This shouldn't happen
anymore with this patch in repoman.

[Comment 3 Danny van Dyk (RETIRED) 2004-08-09 15:06:19 UTC]

Brian: was an accident, i just had opened the form, and the keyboard fell down. Shit happens ;-)

[Comment 4 Danny van Dyk (RETIRED) 2004-08-14 15:36:34 UTC]

Seems this patch needs some loving. (BUG #60345)

For example the line in comment #11 would be nice to be recognized.

Brian: do you have any idea how to accomplish that ?

[Comment 5 Brian Harring (RETIRED) 2005-02-28 00:52:26 UTC]

Whatever happened with this check... and anyone up for trying to get it going again?

[Comment 6 Alec Warner (RETIRED) 2006-01-25 22:17:41 UTC]

Personally I think this is kinda messy, although most of epatch's are standard vars (FILESDIR, P,PV), I would hope that the arch teams/AT's will find crap like this before it hits the stable tree.

[Comment 7 Marius Mauch (RETIRED) 2007-01-10 06:42:03 UTC]

Too many false positives to be usable. Reopen in case you have a better version.