New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test fail on Windows due to hard coded directory separators #10
Comments
@nanis, I applied same technique as per pegex-pm PR/27. Please test and close if fixed. |
I am afraid that wasn't sufficient. I submitted test results for both modules using |
@nanis can you provide the code/script/actions/whatever that you are using to build this perl? Then I or someone else can reproduce and easily fix these bugs. |
@nanis, I want this to be sorted out. Please help me help you ;-) |
I do not have access to my Windows environment at the moment. However, you can see the sample test failures for Pegex. I explained how I built this I do appreciate your effort, and I am going to come back to this as soon as practicable. |
@mohawk2 You should have said "help me help you help me." |
OK, looking at the source code for TestML-0.51, I see that you are still using a regex match instead of With the following simple change, most of the test failures disappear:
The only test failure that remains after fixing that obvious problem is:
That one seems to be a genuine problem with the |
@nanis, can you please test TestML-0.52 and report back? |
C:\> cpanm TestML --> Working on TestML Fetching http://www.cpan.org/authors/id/I/IN/INGY/TestML-0.52.tar.gz ... OK Configuring TestML-0.52 ... OK Building and testing TestML-0.52 ... OK Successfully installed TestML-0.52 1 distribution installed |
Nice work people! |
Same issue as pegex-parser/pegex-pm#26
The text was updated successfully, but these errors were encountered: