[ 
https://issues.apache.org/jira/browse/PIG-501?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Daniel Dai updated PIG-501:
---------------------------

    Attachment: PIG_cygwin_original_javacc.Patch

It is because the javacc.jar. My patch assume the new version of javacc.jar.

Originally the assert statement is:
assertTrue(e.getMessage().startsWith("Encountered \"is\" at line 2, column 
6."));

Since new version of javacc change the error message, I change the assert 
statement to 
assertTrue(e.getMessage().startsWith("Encountered \" <IDENTIFIER> \"is \"\" at 
line 2, column 6."));

Now we still use the original javacc.jar, so we should not change this line. I 
attached the patch without this change.

> Make branches/types work under cygwin
> -------------------------------------
>
>                 Key: PIG-501
>                 URL: https://issues.apache.org/jira/browse/PIG-501
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>    Affects Versions: types_branch
>         Environment: cygwin
>            Reporter: Daniel Dai
>            Assignee: Daniel Dai
>             Fix For: types_branch
>
>         Attachments: javacc.jar, PIG_cygwin.Patch, PIG_cygwin2.Patch, 
> PIG_cygwin_original_javacc.Patch
>
>
> We've already make all unit tests pass under cygwin for trunk (See 
> [PIG-243|https://issues.apache.org/jira/browse/PIG-243]). We need to do the 
> same for branches/types. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to