Gentoo Archives: gentoo-dev

From: Doug Goldstein <cardoe@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in app-portage/genlop: ChangeLog genlop-0.30.8.ebuild
Date: Wed, 26 Sep 2007 22:05:10
Message-Id: 46FAD4C6.8020804@gentoo.org
In Reply to: Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in app-portage/genlop: ChangeLog genlop-0.30.8.ebuild by Mike Frysinger
1 Mike Frysinger wrote:
2 > On Wednesday 26 September 2007, Donnie Berkholz wrote:
3 >
4 >> On 16:11 Wed 26 Sep , Mike Frysinger wrote:
5 >>
6 >>> On Wednesday 26 September 2007, Christian Faulhammer wrote:
7 >>>
8 >>>> Joe Peterson <lavajoe@g.o>:
9 >>>>
10 >>>>> Thanks for the tip. I added "failed to install genlop (via dobin)" -
11 >>>>> not sure if there is a standard way to do this, as it seems many
12 >>>>> ebuilds just do "dobin failed", and some do "failed to install ...".
13 >>>>>
14 >>>> It is mainly to localise which die command caused the halt. So I know
15 >>>> of no standard.
16 >>>>
17 >>> if there is just one call to die in a function, then i usually dont
18 >>> bother ... but if there are multiple ones (possibly nested), then it can
19 >>> easily save time
20 >>>
21 >> Cardoe was just telling me that die messages are not that useful or
22 >> time-saving because portage posts the line number of the failure
23 >> already.
24 >>
25 >
26 > true, since portage has added this traceback feature (it hasnt always been
27 > there), the need for the message has decreased ... i want to say however that
28 > it still isnt 100% correct in some nested situations, but i may be
29 > remembering things wrong or outdated ...
30 >
31 I thought these issues were worked out already?
32
33 > also, ebuilds do change over time, so what line # may be correct one day may
34 > not be relevant the next ...
35 >
36 True. I will concede this point. I could attempt to argue this is why
37 it's important to know the version and revision of the package you are
38 emerging. But the counter point is evident, times when the ebuild is
39 changed without a bump pose a problem.
40
41 Which could bring up a point of would it be useful to see if we can
42 print out the actual line that caused the die. Now, I don't know if this
43 feasible or something the Portage devs want to do. But again, in the
44 effort to streamline this might be something to consider.
45
46 >
47 >> That prompts the question, should we get rid of die messages?
48 >>
49 >
50 > perhaps de-emphasize their general worth, but not get rid of them
51 > -mike
52 >
53 Which is what I'm after. Let's not force people to put a pointless
54 message in there if it's going to be pointless. Essentially, the
55 argument here is let's be consistent and put a message always. But a
56 better plan of action is let's use common sense and add it as needed.
57 --
58 gentoo-dev@g.o mailing list

Replies