hadoop-pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Daniel Dai (JIRA)" <j...@apache.org>
Subject [jira] Updated: (PIG-501) Make branches/types work under cygwin
Date Tue, 21 Oct 2008 00:50:44 GMT

     [ 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.


Mime
View raw message