commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Simone Tripodi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CHAIN-53) Global Update of Chain - Generics, JDK 1.5, Update Dependency Versions
Date Tue, 23 Aug 2011 12:43:29 GMT

    [ https://issues.apache.org/jira/browse/CHAIN-53?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13089427#comment-13089427
] 

Simone Tripodi commented on CHAIN-53:
-------------------------------------

Elijah,
as discussed in the [ML|http://markmail.org/message/lnusnwl6gd6eb3p2], I created a [branch|https://svn.apache.org/repos/asf/commons/proper/chain/branches/version-2.0-work]
where I'm trying to apply the patch, unfortunately I didn't have success; what I executed
is

{code}
patch -p0 < v2_patch_2011-08-18.diff
{code}

and got the following error list

{code}
patching file src/test/java/org/apache/commons/chain/impl/NonDelegatingCommand.java
Hunk #1 succeeded at 29 with fuzz 1.
patching file src/test/java/org/apache/commons/chain/generic/DispatchLookupCommandTestCase.java
patching file src/test/java/org/apache/commons/chain/generic/LookupCommandTestCase.java
patching file src/test/java/org/apache/commons/chain/generic/DispatchCommandTestCase.java
patching file src/test/java/org/apache/commons/chain/web/servlet/MockServletContext.java
patching file src/test/java/org/apache/commons/chain/web/servlet/ServletWebContextTestCase.java
patching file src/test/java/org/apache/commons/chain/web/portlet/MockPortletSession.java
patching file src/test/java/org/apache/commons/chain/web/portlet/MockPortletContext.java
patching file src/test/java/org/apache/commons/chain/web/portlet/MockPortletRequest.java
patching file src/main/java/org/apache/commons/chain/impl/ContextBase.java
Hunk #2 succeeded at 52 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/impl/CatalogBase.java
patching file src/main/java/org/apache/commons/chain/impl/CatalogFactoryBase.java
patching file src/main/java/org/apache/commons/chain/impl/ChainBase.java
Hunk #1 FAILED at 28.
1 out of 8 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/impl/ChainBase.java.rej
patching file src/main/java/org/apache/commons/chain/CatalogFactory.java
patching file src/main/java/org/apache/commons/chain/Chain.java
Hunk #1 FAILED at 47.
1 out of 3 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/Chain.java.rej
patching file src/main/java/org/apache/commons/chain/Context.java
Hunk #1 succeeded at 60 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/config/ConfigRegisterRule.java
patching file src/main/java/org/apache/commons/chain/config/ConfigCatalogRule.java
patching file src/main/java/org/apache/commons/chain/config/ConfigDefineRule.java
patching file src/main/java/org/apache/commons/chain/Filter.java
Hunk #1 FAILED at 37.
1 out of 2 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/Filter.java.rej
patching file src/main/java/org/apache/commons/chain/Catalog.java
patching file src/main/java/org/apache/commons/chain/generic/DispatchLookupCommand.java
Hunk #1 FAILED at 47.
1 out of 7 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/generic/DispatchLookupCommand.java.rej
patching file src/main/java/org/apache/commons/chain/generic/LookupCommand.java
Hunk #1 FAILED at 40.
1 out of 6 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/generic/LookupCommand.java.rej
patching file src/main/java/org/apache/commons/chain/generic/CopyCommand.java
Hunk #1 FAILED at 25.
1 out of 2 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/generic/CopyCommand.java.rej
patching file src/main/java/org/apache/commons/chain/generic/RemoveCommand.java
Hunk #1 FAILED at 24.
1 out of 2 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/generic/RemoveCommand.java.rej
patching file src/main/java/org/apache/commons/chain/generic/DispatchCommand.java
patching file src/main/java/org/apache/commons/chain/web/MapEntry.java
Hunk #1 FAILED at 24.
1 out of 7 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/web/MapEntry.java.rej
patching file src/main/java/org/apache/commons/chain/web/faces/FacesGetLocaleCommand.java
patching file src/main/java/org/apache/commons/chain/web/faces/FacesWebContext.java
patching file src/main/java/org/apache/commons/chain/web/faces/FacesSetLocaleCommand.java
patching file src/main/java/org/apache/commons/chain/web/AbstractGetLocaleCommand.java
Hunk #1 FAILED at 27.
1 out of 3 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/web/AbstractGetLocaleCommand.java.rej
patching file src/main/java/org/apache/commons/chain/web/WebContext.java
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletApplicationScopeMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletGetLocaleCommand.java
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletParamMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletPathMapper.java
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletHeaderValuesMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletCookieMap.java
patching file src/main/java/org/apache/commons/chain/web/servlet/PathInfoMapper.java
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletSetLocaleCommand.java
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletSessionScopeMap.java
Hunk #2 succeeded at 37 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/servlet/ChainProcessor.java
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletParamValuesMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletRequestScopeMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletInitParamMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletWebContext.java
patching file src/main/java/org/apache/commons/chain/web/servlet/ServletHeaderMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/servlet/RequestParameterMapper.java
patching file src/main/java/org/apache/commons/chain/web/ChainServlet.java
patching file src/main/java/org/apache/commons/chain/web/ChainListener.java
patching file src/main/java/org/apache/commons/chain/web/portlet/PortletSetLocaleCommand.java
patching file src/main/java/org/apache/commons/chain/web/portlet/PortletApplicationScopeMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/portlet/PortletSessionScopeMap.java
Hunk #2 succeeded at 37 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/portlet/PortletGetLocaleCommand.java
patching file src/main/java/org/apache/commons/chain/web/portlet/PortletParamMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/portlet/PortletParamValuesMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/portlet/PortletRequestScopeMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/portlet/PortletInitParamMap.java
Hunk #2 succeeded at 36 with fuzz 1.
patching file src/main/java/org/apache/commons/chain/web/portlet/PortletWebContext.java
patching file src/main/java/org/apache/commons/chain/web/ChainResources.java
patching file src/main/java/org/apache/commons/chain/web/AbstractSetLocaleCommand.java
Hunk #1 FAILED at 28.
1 out of 3 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/web/AbstractSetLocaleCommand.java.rej
patching file src/main/java/org/apache/commons/chain/Command.java
Hunk #1 FAILED at 73.
1 out of 2 hunks FAILED -- saving rejects to file src/main/java/org/apache/commons/chain/Command.java.rej
patching file pom.xml
patching file checkstyle.xml
{code}

Did I make something wrong or missed something?
Many thanks in advance!!!

> Global Update of Chain - Generics, JDK 1.5, Update Dependency Versions
> ----------------------------------------------------------------------
>
>                 Key: CHAIN-53
>                 URL: https://issues.apache.org/jira/browse/CHAIN-53
>             Project: Commons Chain
>          Issue Type: Improvement
>            Reporter: Elijah Zupancic
>              Labels: newbie, patch
>         Attachments: v2_patch_2011-08-18.diff
>
>
> As posted in the mailing list, I've done this work outside of an offical branch.
> In this patch:
> * Global upgrade to the JDK 1.5
> * Added @Override annotations
> * Upgraded to the Servlet 2.5 API
> * Upgraded to the Faces 2.1 API
> * Upgraded to the Portlet 2.0 API
> * Upgraded the Maven Parent POM version
> * Added generics support to Command so that Command's API looks like:
> public interface Command<T extends Context> {
> ...
>    boolean execute(T context) throws Exception;
> }
> The only incompatibility with the existing API is in: org.apache.commons.chain.web.servlet.ServletHeaderValuesMap
on line 97. Previously the API was returning Set<Entry<String, Enumeration<String>>
when by all indications it actually should have been returning Set<Entry<String, String[]>>.
I believe that I fixed a previously undiscovered bug here.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message