Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 228935 - [QA] check for "command not found" (patch provided)
Summary: [QA] check for "command not found" (patch provided)
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Enhancement/Feature Requests (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Portage team
Keywords: InVCS
Depends on:
Blocks: 216231
  Show dependency tree
Reported: 2008-06-22 17:56 UTC by Olivier Huber
Modified: 2010-10-06 21:21 UTC (History)
0 users

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

small patch (portage-qa.patch,1.97 KB, patch)
2008-06-22 17:58 UTC, Olivier Huber
Details | Diff
right patch (portage-qa.patch,1.97 KB, patch)
2008-06-22 17:59 UTC, Olivier Huber
Details | Diff
dump ebuild (bidon-0.1.ebuild,425 bytes, text/plain)
2008-06-22 18:01 UTC, Olivier Huber

Note You need to log in before you can comment on or make changes to this bug.
Description Olivier Huber 2008-06-22 17:56:59 UTC
Sometimes, when there's a typo in the ebuild, or when an eclass is not inherit, we encounter this error :
/var/tmp/portage/xxx/yyy/temp/environment: line XXX: zzz command not found
/usr/portage/xxx/yyy/yyy.ebuild: line XXX: zzz command not found

I think it could be great to search such failure and display an QA message so people will report it.

I'll post a little patch based on what was written for "maintainer mode" and "Unrecognized options"

I know that there are not so much failures like that (I have found 4 by grepping logs), but I think it may be useful.
Comment 1 Olivier Huber 2008-06-22 17:58:00 UTC
Created attachment 158023 [details, diff]
small patch
Comment 2 Olivier Huber 2008-06-22 17:59:33 UTC
Created attachment 158025 [details, diff]
right patch
Comment 3 Olivier Huber 2008-06-22 18:01:27 UTC
Created attachment 158029 [details]
dump ebuild

Dumb ebuild to test features
Comment 4 Zac Medico gentoo-dev 2008-06-23 02:59:08 UTC
Thanks, I've committed a slightly condensed version of your patch:
Comment 5 Zac Medico gentoo-dev 2008-07-23 07:57:09 UTC
This is fixed in 2.2_rc2.