Bug 253629 – econf || die nolonger requires || die

• Status: RESOLVED FIXED
• Product: Documentation

[Description Daniel Black (RETIRED) 2009-01-03 23:16:25 UTC]

if this is the currently official guidance can we update the doco?

if so can we take out the '|| die' parts of:
[1] http://www.gentoo.org/proj/en/devrel/handbook/handbook.xml?part=2&chap=1 
[2] http://devmanual.gentoo.org/quickstart/index.html
[3] http://devmanual.gentoo.org/ebuild-writing/functions/src_compile/index.html

and where ever else it appears in official doco

does anything else need to be updated?

[Comment 1 Thilo Bangert (RETIRED) (RETIRED) 2009-02-25 18:36:50 UTC]

Created attachment 183171 [details, diff]
econf-needs-no-die.patch

[Comment 2 Thilo Bangert (RETIRED) (RETIRED) 2009-02-25 20:45:28 UTC]

Created attachment 183185 [details, diff]
econf-die-removal-try2.patch

this is an improvement to the earlier patch fixing all grepable instances of econf in the devmanual source

[Comment 3 Thilo Bangert (RETIRED) (RETIRED) 2009-02-25 20:46:48 UTC]

adding devrel for handbook.

[Comment 4 Peter Volkov (RETIRED) 2009-02-27 10:42:24 UTC]

Example (Code Listing 2.7: My first ebuild) in http://www.gentoo.org/doc/en/xml-guide.xml needs to be updated too. (And not to open separate bug: probably also it's a good idea to add || die after dodoc and dohtml and update Copyright date).

[Comment 5 nm (RETIRED) 2009-02-28 01:35:40 UTC]

(In reply to comment #4)

GDP done.

[Comment 6 Thomas Anderson (tanderson) (RETIRED) 2009-03-03 17:36:01 UTC]

Created attachment 183781 [details, diff]
Patch with a few more things(but lacking others)

[Comment 7 Thomas Anderson (tanderson) (RETIRED) 2009-03-03 19:32:21 UTC]

Created attachment 183794 [details, diff]
More Betterer patch

Combines all of the above patches into something a little better.

[Comment 8 Mark Loeser (RETIRED) 2009-03-09 19:50:23 UTC]

Added thanks