Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 121317 - helpers should do error checking
Summary: helpers should do error checking
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Core (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-02-02 09:52 UTC by Simon Stelling (RETIRED)
Modified: 2006-11-17 00:53 UTC (History)
1 user (show)

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


Attachments
an attempt to fix the issue (helpers_error_checking.patch,6.70 KB, patch)
2006-02-02 09:52 UTC, Simon Stelling (RETIRED)
Details | Diff
install as much as possible, then exit appropriately (helpers_error_checking.patch,7.92 KB, patch)
2006-02-03 03:52 UTC, Simon Stelling (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Stelling (RETIRED) gentoo-dev 2006-02-02 09:52:06 UTC
currently, the do* scripts don't exit with a non-zero status when a critical command fails, except if it is the last one. However, they should rather abort than pretend everything works fine. At least dobin and dolib should do proper error checking so the ebuild can bail out if something breaks.
Comment 1 Simon Stelling (RETIRED) gentoo-dev 2006-02-02 09:52:35 UTC
Created attachment 78734 [details, diff]
an attempt to fix the issue
Comment 2 SpanKY gentoo-dev 2006-02-02 19:04:11 UTC
imo the scripts should exit non-zero, but install as many files/things as possible
Comment 3 Simon Stelling (RETIRED) gentoo-dev 2006-02-03 02:44:27 UTC
Comment on attachment 78734 [details, diff]
an attempt to fix the issue

that's true, didn't think of that possibility.. i'll provide an updated patch
Comment 4 Simon Stelling (RETIRED) gentoo-dev 2006-02-03 03:52:33 UTC
Created attachment 78796 [details, diff]
install as much as possible, then exit appropriately

here we go
Comment 5 SpanKY gentoo-dev 2006-02-03 21:33:26 UTC
cleaned up:
dodir

doconfd
doenvd
donitd

newconfd
newenvd
newinitd

newbin
newdoc
newexe
newins
newlib.a
newlib.so
newman
newsbin
Comment 6 SpanKY gentoo-dev 2006-02-03 21:56:12 UTC
cleaned up dobin / dosbin
Comment 7 SpanKY gentoo-dev 2006-02-13 16:25:38 UTC
dolib is fixed now
Comment 8 SpanKY gentoo-dev 2006-02-22 21:13:55 UTC
doman now updated
Comment 9 Simon Stelling (RETIRED) gentoo-dev 2006-11-17 00:53:38 UTC
i think this is done