Issues 2009-03-18
 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
11230DEFECTtriageP3issues@sconsVariable expansion removes trailing space
4
consensus2.xp3 +subst
5
6
Bill
7
Brandonagree
8
David
9
Gary2.xp3 +substagree w/ greg
10
Greg2.xp3 +substSee comments in issue
11
Jim
12
Ken
13
Sohail
14
Steven2.xp3 +substagree w/greg
15
16
14490DEFECTtriageP3mightyllamasTEMPFILE doesn't work on commands with curly brackets
17
consensus2.xp3 +subst
18
19
Bill
20
Brandonagree
21
David
22
Garyregex is fundamentally unable to handle this case (nested braces). Need a more serious parser, so I'm not sure subst revamp will fix it...
23
Greg2.xp3 +substConcur with Jim. I don't completely understand the issue, but it's clearly quoting gone wild.
24
Jim
25
Ken
26
Sohail
27
Steven2.xp3 +substagree w/greg
28
29
14500DEFECTtriageP3mightyllamasTempFileMunge should separate command line arguments with newline
30
consensus2.1p3DOS person
31
32
Bill
33
Brandonwhat is DOS? :)
34
David
35
Gary2.1p3DOS personagree w/ all. Should not be a big deal.
36
Greg2.1p3DOS personGo with Jim's recommendation. Potentially destablizing for 1.3, so do it in 2.1.
37
Jim
38
Ken
39
Sohail
40
Steven2.1p3DOS personagree w/Jim (and Greg)
41
42
16320DEFECTtriageP3mightyllamasPrecompiled headers fail with whitespace in the directory path
43
consensus
44
45
Bill
46
BrandonMake a parent bug for all subst issues?
47
David
48
Gary2.1p3Is this even really a subst issue? I think the fix is right as it stands (though maybe the preceding /Yu%s also needs quoting). At least this is a pretty unusual case to expect automatic quoting to work.
49
GregHmmmOn the one hand, I don't like all these piecemeal one-off partial fixes, but on the other hand, we need to talk about scheduling SubstQuoteEscapeCache (or some other permanent fix). If the latter is going to be "soon" (for some reasonable definition of "soon"), then +subst and fix it then. Otherwise, I agree with Jim and 2.1 p3 DOS person is reasonable. See also 2363 below.
50
Jim
51
Ken
52
Sohail
53
Steven2.1p3Agree that piecemeal fixes aren't great, but the alternative is waiting for the rewrite. Until then, I think piecemeal is okay *provided* that the fix is, in fact, accompanied by test changes that can be used to verify this particular behavior that we want to preserve when it finally gets rewritten.
54
55
19080PATCHtriageP3mightyllamasImplemented --cache-verify and --cache-force now overwrites
56
consensus2.x?? +easy
57
58
Bill
59
BrandonIt sounds like this might be Windows file share errors, in which case the problem might be that the bug is with the Windows share failing rather than SCons? In any case the cache could probably have a million command line parameters. It might make sense to break the command line options into a "typical" and "advanced" set.
60
David
61
Gary2.x?? +easySounds at least like there's a use case for this where user error can pollute the cache; I don't pretend to understand fully but if he thinks he needs it, I'm OK with it. But if it's a debug-only option, I'd prefer something with the name "debug" in it.
62
Greg2.x??draft choiceI'm still leery of new command-line options that we may need to deprecate in the future, but it looks like there's enough evidence that it does happen, so put it in the queue to get done. Maybe +Easy to get a self-selecting draft choice?
63
Jim
64
Ken
65
Sohail
66
Steven2.x?? +easyI like the +easy suggestion. Otherwise, I don't feel strongly and will go with the flow.
67
68
19140DEFECTtriageP3mightyllamasTEMPFILEPREFIX doesn't work for a prefix that requires a trailing space
69
consensus2.xp3 +subst
70
71
Bill
72
Brandonagree
73
David
74
Gary2.xp3 +substok, but not sure why it doesn't work as is -- string + string shouldn't eliminate trailing spaces.
75
Greg2.xp3 +substConcur with Jim.
76
Jim
77
Ken
78
Sohail
79
Steven2.xp3 +substagree w/Jim
80
81
20180DEFECTtriageP3issues@sconsINSTALLSTR is always stripped ?
82
consensus2.xp3 +subst
83
84
Bill
85
Brandonagree
86
David
87
Gary2.xp3 +substagree
88
Greg2.xp3 +substSee comments in issue.
89
Jim
90
Ken
91
Sohail
92
Steven2.xp3 +substagree w/greg
93
94
22880ENHANCEMENTtriageP4issues@sconsRPM package should not copy every source files
95
consensus
96
97
Bill
98
Brandon
99
David
100
Garyphilipp
Loading...
 
 
 
Sheet 1