You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@river.apache.org by "Juan Ramirez (JIRA)" <ji...@apache.org> on 2007/08/10 20:33:43 UTC

[jira] Created: (RIVER-211) source files for ServiceItem, ServiceMatches, and ServiceTemplate do not match binary distribution

source files for ServiceItem, ServiceMatches, and ServiceTemplate do not match binary distribution
--------------------------------------------------------------------------------------------------

                 Key: RIVER-211
                 URL: https://issues.apache.org/jira/browse/RIVER-211
             Project: River
          Issue Type: Bug
          Components: build
    Affects Versions: jtsk_2.1
            Reporter: Juan Ramirez
            Priority: Minor


[6340076|http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6340076]

The source files for ServiceItem, ServiceMatches, and ServiceTemplate included with the Porter GA release of the starter kit are the Porter Beta 2 versions.  The binary distribution of these classes in jsk-platform.jar and jini-core.jar is the correct Porter GA version.  The source files are out of date relative to the binaries.  The Porter Beta 2 versions contain a bug in their toString() implementations (CRs 6326824, 6326851, 6326855)

Diffs between GA and Beta 2 vesions; GA version appears first:

ServiceItem.java:

81,87c81,83
<             sBuffer.append("[");
<             if (attributeSets.length > 0) {
<                 for (int i = 0; i < attributeSets.length - 1; i++)
<                     sBuffer.append(attributeSets[i]).append(" ");
<                 sBuffer.append(attributeSets[attributeSets.length - 1]);
<             }
<             sBuffer.append("]");
---
>           for (int i = 0; i < attributeSets.length - 1; i++)
>               sBuffer.append(attributeSets[i]).append(" ");
>           sBuffer.append(attributeSets[attributeSets.length - 1]);

ServiceMatches.java:

70,76c70,72
<             sBuffer.append("[");
<             if (items.length > 0) {
<                 for (int i = 0; items.length > 0 && i < items.length - 1; i++)
<                     sBuffer.append(items[i]).append(" ");
<                 sBuffer.append(items[items.length - 1]);
<             }
<             sBuffer.append("]");
---
>           for (int i = 0; i < items.length - 1; i++)
>               sBuffer.append(items[i]).append(" ");
>           sBuffer.append(items[items.length - 1]);


ServiceTemplate.java:

94,100c94,96
<             sBuffer.append("[");
<             if (serviceTypes.length > 0) {
<                 for (int i = 0; i < serviceTypes.length - 1; i++)
<                     sBuffer.append(serviceTypes[i]).append(" ");
<                 sBuffer.append(serviceTypes[serviceTypes.length - 1]);
<             }
<             sBuffer.append("]");
---
>           for (int i = 0; i < serviceTypes.length - 1; i++)
>               sBuffer.append(serviceTypes[i]).append(" ");
>           sBuffer.append(serviceTypes[serviceTypes.length - 1]);
106,113c102,105
<             sBuffer.append("[");
<             if (attributeSetTemplates.length > 0) {
<                 for (int i = 0; i < attributeSetTemplates.length - 1; i++)
<                     sBuffer.append(attributeSetTemplates[i]).append(" ");
<                 sBuffer.append(
<                     attributeSetTemplates[attributeSetTemplates.length - 1]);
<             }
<             sBuffer.append("]");
---
>           for (int i = 0; i < attributeSetTemplates.length - 1; i++)
>               sBuffer.append(attributeSetTemplates[i]).append(" ");
>           sBuffer.append(
>              attributeSetTemplates[attributeSetTemplates.length - 1]);

There is actually nothing to fix here.  The files were out of sync because
the manually modified final 2.1 build did not include the source files that
matched the binary changes.  I confirmed that the build does indeed include
the correct source files.

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


[jira] Closed: (RIVER-211) source files for ServiceItem, ServiceMatches, and ServiceTemplate do not match binary distribution

Posted by "Frank Barnaby (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/RIVER-211?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Frank Barnaby closed RIVER-211.
-------------------------------

       Resolution: Won't Fix
    Fix Version/s: AR1
         Assignee: Frank Barnaby

This is no longer an issue.  The problem with the Porter (2.1) release was due to a manual oversight--the build process was and continues to be correct with regard to these source files.



> source files for ServiceItem, ServiceMatches, and ServiceTemplate do not match binary distribution
> --------------------------------------------------------------------------------------------------
>
>                 Key: RIVER-211
>                 URL: https://issues.apache.org/jira/browse/RIVER-211
>             Project: River
>          Issue Type: Bug
>          Components: build
>    Affects Versions: jtsk_2.1
>            Reporter: Juan Ramirez
>            Assignee: Frank Barnaby
>            Priority: Minor
>             Fix For: AR1
>
>
> [6340076|http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6340076]
> The source files for ServiceItem, ServiceMatches, and ServiceTemplate included with the Porter GA release of the starter kit are the Porter Beta 2 versions.  The binary distribution of these classes in jsk-platform.jar and jini-core.jar is the correct Porter GA version.  The source files are out of date relative to the binaries.  The Porter Beta 2 versions contain a bug in their toString() implementations (CRs 6326824, 6326851, 6326855)
> Diffs between GA and Beta 2 vesions; GA version appears first:
> ServiceItem.java:
> 81,87c81,83
> <             sBuffer.append("[");
> <             if (attributeSets.length > 0) {
> <                 for (int i = 0; i < attributeSets.length - 1; i++)
> <                     sBuffer.append(attributeSets[i]).append(" ");
> <                 sBuffer.append(attributeSets[attributeSets.length - 1]);
> <             }
> <             sBuffer.append("]");
> ---
> >           for (int i = 0; i < attributeSets.length - 1; i++)
> >               sBuffer.append(attributeSets[i]).append(" ");
> >           sBuffer.append(attributeSets[attributeSets.length - 1]);
> ServiceMatches.java:
> 70,76c70,72
> <             sBuffer.append("[");
> <             if (items.length > 0) {
> <                 for (int i = 0; items.length > 0 && i < items.length - 1; i++)
> <                     sBuffer.append(items[i]).append(" ");
> <                 sBuffer.append(items[items.length - 1]);
> <             }
> <             sBuffer.append("]");
> ---
> >           for (int i = 0; i < items.length - 1; i++)
> >               sBuffer.append(items[i]).append(" ");
> >           sBuffer.append(items[items.length - 1]);
> ServiceTemplate.java:
> 94,100c94,96
> <             sBuffer.append("[");
> <             if (serviceTypes.length > 0) {
> <                 for (int i = 0; i < serviceTypes.length - 1; i++)
> <                     sBuffer.append(serviceTypes[i]).append(" ");
> <                 sBuffer.append(serviceTypes[serviceTypes.length - 1]);
> <             }
> <             sBuffer.append("]");
> ---
> >           for (int i = 0; i < serviceTypes.length - 1; i++)
> >               sBuffer.append(serviceTypes[i]).append(" ");
> >           sBuffer.append(serviceTypes[serviceTypes.length - 1]);
> 106,113c102,105
> <             sBuffer.append("[");
> <             if (attributeSetTemplates.length > 0) {
> <                 for (int i = 0; i < attributeSetTemplates.length - 1; i++)
> <                     sBuffer.append(attributeSetTemplates[i]).append(" ");
> <                 sBuffer.append(
> <                     attributeSetTemplates[attributeSetTemplates.length - 1]);
> <             }
> <             sBuffer.append("]");
> ---
> >           for (int i = 0; i < attributeSetTemplates.length - 1; i++)
> >               sBuffer.append(attributeSetTemplates[i]).append(" ");
> >           sBuffer.append(
> >              attributeSetTemplates[attributeSetTemplates.length - 1]);
> There is actually nothing to fix here.  The files were out of sync because
> the manually modified final 2.1 build did not include the source files that
> matched the binary changes.  I confirmed that the build does indeed include
> the correct source files.

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