search

ajwang / groovypptest

Automatically exported from code.google.com/p/groovypptest
0 stars 0 forks source link

G++ build fails due to Issue288Test#testCharStringGString() #306

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
I am on windows and Issue288Test#testCharStringGString() always fails for me.

Below are the details:
---------------------------------------------------------
assert "[$head]".replace('c', File.separatorChar) == '[ab/d]' | | | | abcd 
[ab\d] \ false

junit.framework.AssertionFailedError: assert "[$head]".replace('c', 
File.separatorChar) == '[ab/d]'
| | | |
abcd [ab\d] \ false
at 
org.mbte.groovypp.compiler.Issues.Issue288Test.testCharStringGString(Issue288Tes
t.groovy:39)
---------------------------------------------------------

Original issue reported on code.google.com by roshanda...@gmail.com on 14 Oct 2010 at 1:07

GoogleCodeExporter commented 8 years ago 
Should be easy to fix by replacing File.separatorChar with '/' I guess

Original comment by alex.tka...@gmail.com on 14 Oct 2010 at 1:14

GoogleCodeExporter commented 8 years ago 
Yes, should be.

Original comment by roshanda...@gmail.com on 14 Oct 2010 at 1:19

GoogleCodeExporter commented 8 years ago 
Roshan, you don't mind if I assign it to you?

Original comment by alex.tka...@gmail.com on 14 Oct 2010 at 1:27

GoogleCodeExporter commented 8 years ago 
I don't think I have the commit right on this new git setup. At least there 
hasn't been any communication regarding that.

Now who will go through all the git-patch-mail(-and-then-you-applying) pain for 
a 1-character fix!

Original comment by roshanda...@gmail.com on 14 Oct 2010 at 1:31

GoogleCodeExporter commented 8 years ago

Original comment by roshanda...@gmail.com on 14 Oct 2010 at 1:52

GoogleCodeExporter commented 8 years ago 
Done. Thanks for explaining the new git/github based contribution flow, Alex.

Original comment by roshanda...@gmail.com on 15 Oct 2010 at 5:26