Account Options>

  1. Sign in
Issues 2007-01-07
 Share
The version of the browser you are using is no longer supported. Please upgrade to a supported browser.Dismiss

 
$
%
123
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
|
 
Still loading...
ABCDEFG
1
IDVotesType/MemberMilestonePriOwnerSummary/Comments
2
3
22790PATCHtriageP4issues@sconsMake swig tool handle spaces in SWIGOUTDIR
4
consensus
5
6
Bill
7
Brandon
8
David
9
Garyanytimep3arveArve's last patch was excellent, this one seems good too. Except for the tests it's basically a one-liner. True, we want to do a better job handling quoting in general, but this one is easy and if it makes him happy, why not? I put Arve as owner since he's clearly doing a good job, but one of us ought to integrate it, preferably whoever has swig. (Hmm, I can probably install it on my ubuntu test VM.)
10
Gregsee comment +keywordWe're actively discussing revamping the substitution and quoting logic, but when are we going to field it? All of these issues should be accumulated under one keyword and a common milestone.
11
Jim
12
Ken
13
Sohail
14
Steven1.3p3arveGood patch, good tests, agree w/Gary re: not letting the lack of a better general solution get in the way of something that helps this particular case. I'd be okay with giving Arve commit rights, or else any of us could integrate it.
15
16
22810DEFECTtriageP4issues@sconsCan't attach an Action to an Alias
17
consensus
18
19
Bill
20
Brandon
21
David
22
Gary2.0p3I was going to say let's try to make it work, but since it already doesn't, erroring is really easy, and it leaves the door open to making it work someday.
23
Greg2.xp4?hmmm...Should be simple to detect if the object being extended is a File and generate an error message if not.
24
Jim
25
Ken
26
Sohail
27
Steven2.0p3An error is probably draconian, because sometimes an Alias *can* have an Action and get executed. So now that I think about it, the error might make sense, but not because it's an Alias, but because it's a Node that doesn't have an Action. I can go with that.
28
29
22820ENHANCEMENTtriageP4issues@sconsJAVAVERSION should be mentioned in Java descriptions
30
consensus
31
32
Bill
33
Brandon
34
David
35
GarySuggested wording is not too bad, with s/we/you/g. Schedule to be determined by author (also tbd)
36
GregDocumentation change with some suggested wording. Can we draft someone literate to do this?
37
Jim
38
Ken
39
Sohail
40
Steven1.3p4?Good doc change, might as well get it in sooner rather than later.
41
42
22830ENHANCEMENTtriageP4issues@sconsNSIS has tool configuration for mingw cross-compiler (and "mstoolkit")
43
consensus
44
45
Bill
46
Brandon
47
David
48
GaryCouldn't resolve paste.lisp.org to see the diffs, but the idea must be that Nullsoft is using SCons to build NSIS and has a patched version of mingw.py that can cross-compile, and the ticket would like us to accept that since we're the upstream provider. I basically agree w/ Greg, better to do this right than hack it in like this, but... given our progress rate and the value of existing, working code, I'm tempted to look into it.
49
Gregsee commentSpecial-case cross-compiling (boo, hiss), but has a patch and has been requested before. Maybe make it 2.x p4 and hope that the generalized cross-compilation gets there first?
50
Jim
51
Ken
52
Sohail
53
Steven2.xp3Seems pretty reasonable to me. Is there really a significant downside to letting it in so it helps people until a general cross-compilation solution becomes real? I can't think of one. If it doesn't hurt the long-term direction, I'd be inclined to pull it in even sooner, maybe 1.3.
54
55
22840FEATUREtriageP4issues@scons$TEMPFILEPREFIX does not allow whitespace
56
consensus
57
58
Bill
59
Brandon
60
David
61
GaryAgree w/ Greg. The workaround is not clear here; the OP needs a clear spec and guidance for how quoting is handled, and we don't have that yet.
62
Gregsee comment +keywordWe're actively discussing revamping the substitution and quoting logic, but when are we going to field it? All of these issues should be accumulated under one keyword and a common milestone.
63
JimSuggested workaround breaks @blah.lnk. If the function returns a string rather than a list, will that still be quoted? If not, it might allow the -raw solution to work.
64
Ken
65
Sohail
66
StevenDUPAlready submitted as #1914 (exact same use case, too).
67
68
Wow, how about that? I lied when I said there were a lot of new issues for this bug party. Now go over to the issues backlogged from 2005 and update those, and we'll see how many we can clear at the party.
Loading...
 
 
 
Sheet 1