Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.2.3
    • Fix Version/s: 5.2.3
    • Component/s: None
    • Labels:
      None

      Description

      In the concurrent barrier test / concurrent target class, the return value of tryXXX methods is ignored. The tryXXX methods fail if the timeout expires. If the timeout expires during testing, the test will fail.

      1. 0001-TAP5-1326-Fixed-ConcurrentBarrier-test.patch
        4 kB
        Tom van Dijk
      2. 1326.diff
        4 kB
        Tom van Dijk

        Activity

        Hide
        Tom van Dijk added a comment -

        Pull from git://hetdiana.homeip.net/tapestry5.git branch fix1326 for the fix. Also attached.

        Show
        Tom van Dijk added a comment - Pull from git://hetdiana.homeip.net/tapestry5.git branch fix1326 for the fix. Also attached.
        Hide
        Howard M. Lewis Ship added a comment -

        ASF guidelines are that patches may only be accepted via JIRA, not via Git. git format-patch is helpful, though.

        Show
        Howard M. Lewis Ship added a comment - ASF guidelines are that patches may only be accepted via JIRA, not via Git. git format-patch is helpful, though.
        Hide
        Tom van Dijk added a comment -

        Uhm, I attached a diff, that wasn't enough?

        Here's a patch as well. Git patch, I dunno if it's compatible with svn...

        Show
        Tom van Dijk added a comment - Uhm, I attached a diff, that wasn't enough? Here's a patch as well. Git patch, I dunno if it's compatible with svn...
        Hide
        Hudson added a comment -

        Integrated in tapestry-5.2-freestyle #225 (See https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/225/)

        Show
        Hudson added a comment - Integrated in tapestry-5.2-freestyle #225 (See https://hudson.apache.org/hudson/job/tapestry-5.2-freestyle/225/ )

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Tom van Dijk
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development