Issues 2009-12-15
 Share
The version of the browser you are using is no longer supported. Please upgrade to a supported browser.Dismiss

 
View only
 
 
Still loading...
ABCDEFG
1
IDVotesType/MemberMilestonePriOwnerSummary/Comments
2
3
24700PATCHtriageP4issues@sconsDir.alter_target method failure with build_dir enabled
4
Gary to follow up with OP and report back
5
6
BillAgree with Greg. Is the user concerned with the functioning of non-public API and has submitted a patch to fix that?
7
Brandon
8
David
9
Garyresearchask OPWould like to see the user-level problem, like Bill & Greg, but I think his analysis is correct that there's a logic error in that code.
10
GregWhat user-level error does this cause?
11
Jim
12
Ken
13
Sohail
14
Stevenresearchask OPagree w/everyone else. this is likely a bug, but we need a user-level specification of the error
15
16
24740DEFECTtriageP4issues@sconsAlwaysBuild does not work on Install targets
17
Bypassed due to lack of consensus.
18
19
BillCould it be caused by the Command having no source?
20
Brandon
21
David
22
Garyresearchno ideas here
23
Gregdup of SEP0001He's running on a real operating system, so it's not a dup of 2124. My bet is that it's that the dependency is on a directory, which we already know about. I thought there was a bug for that, but I don't see it.
24
Jim
25
Ken
26
Sohail
27
Stevenresearchprobably comes down to the funky way in which directories kind of "depend" on their content targets. might be interesting to see if it reproduces with env.Command() instead of env.Install(). env.Install() specifies the target dir, but it's really building a target *file* within the dir (i.e., the file node is what's returned)
28
29
24820DEFECTtriageP4issues@sconssconf deep recursion
30
Greg to look at it and report back.
31
32
BillNo idea
33
Brandon
34
David
35
GaryNo idea here
36
Greg1.3p1me, but a backup should be chosen in case it's beyond meIt's a problem with the way the "temporary" Decider is created. The first one wraps the actual decider, but each subsequent test wraps the temporary decider, potentially creating a call chain dozens (or even hundreds) deep, causing performance to suck. It seems like there ought to be a simple cure (check if it's wrapping the temporary and not bother to do it again), but my Python-fu couldn't come up with a way to see if the functions were equal (it's got to be obvious, but I'm missing it!). I think it should be treated as a regression and fixed immediately. I'll take another stab at it if you want, but I may have to hand it off. Who would be the backup? LATE ADD: I think I figured it out. I've asked the OP to try it and I'm running regression tests now.
37
Jim
38
Ken
39
Sohail
40
Steven +sconf_revampseems like a deep issue that isn't worth fixing in the current version if the time can be spent giving us a better configure model
41
42
24900DEFECTtriageP4issues@sconsSCons 0.96.93 with C# support
43
Steven to look at it while awaiting test results and report back
44
45
Bill
46
Brandon
47
David
48
Garyattachment is entire SCons 0.96.93 src tree :-(. Asked for diffs.
49
GregI wrote the OP and he updated the issue with information about what was changed. If he can provide tests, I suggest 2.1 p3/4; if not, I suggest 2.x p3/4.
50
Jim
51
Ken
52
Sohail
53
Steven2.x?might be nice to add. haven't looked at the patch -- are there tests?
54
55
24970DEFECTtriageP4issues@sconsQt program without sources ignores LIBPATH
56
consensus
57
58
Bill
59
Brandon
60
David
61
Garyresearch
62
Greginvalid?The loader won't search libraries without names to bind, so I don't see how this could work at all. The Qt() tool does an AppendUnique(), but otherwise doesn't seem to manipulate LIBS, so I don't see why that should affect anything.
63
Jim
64
Ken
65
Sohail
66
Stevenresearchneeds some more characterization here, user-level behavior definitely seems like a bug
67
68
24980DEFECTtriageP4issues@sconsDelayed creation of default Environment overwrites scanner assignments
69
consensus
70
71
Bill
72
Brandon
73
David
74
Garythis looks like a Steven Thing to me...
75
Greg???(doc error at third ref to SourceFileScanner) The SCons.Tool module initializes SourceFileScanner and SCons.Script imports SCons.Tool. It's not recreated anywhere. This is weird; I don't know what to suggest.
76
Jim
77
Ken
78
Sohail
79
Stevenresearchoff the top of my head his first suggested solution sounds best, but needs some more thought
80
81
25000DEFECTtriageP4issues@sconsCCFLAGS shows up twice on command line for C++ compile with Intel C.
82
consensus
83
84
Bill
85
Brandon
86
David
87
GarystevenknightSteven wants it, fine
88
Greg??? +EasySounds pretty trivial.
89
Jim
90
Ken
91
Sohail
92
Stevenfixedstevenknightthought I fixed that; assign to me and I'll try to find the revision
93
94
25020DEFECTtriageP4issues@sconsExample for Ignore() function is wrong
95
consensus
96
97
Bill
98
Brandon
99
David
100
Gary1.3p1doc +Easyagree w/greg
Loading...
 
 
 
Sheet 1