Yes, I saw it and fixed it. As I've stated all ITs fail on my Mac so
I'm blind and therefore use the CI to do all sanity checks. Not good,
but not much I can do about it.

/Anders

On Tue, Aug 21, 2012 at 10:40 PM, Graham Leggett <[email protected]> wrote:
> On 21 Aug 2012, at 9:59 PM, Anders Hammar wrote:
>
>> I've applied the patch. Wrt to rpm-3 I can't judge if your provided
>> patch is ok. But we should probably discuss this in JIRA.
>
> The patch in r17372 is now causing the rpm-2/pom.xml test to fail:
>
> ------------------------------------------------------------------------
> r17372 | afloom | 2012-08-21 15:08:39 -0500 (Tue, 21 Aug 2012) | 1 line
>
> Improved IT verification script.
>
> [INFO] --- maven-invoker-plugin:1.5:run (integration-test) @ rpm-maven-plugin 
> ---
> [INFO] Building: rpm-reactor/pom.xml
> [INFO] ..SUCCESS (14.4 s)
> [INFO] Building: rpm-artifact/pom.xml
> [INFO] ..SUCCESS (10.4 s)
> [INFO] Building: rpm-1/pom.xml
> [INFO] ..SUCCESS (7.5 s)
> [INFO] Building: rpm-2/pom.xml
> [INFO] ..FAILED (17.5 s)
> [INFO]   The post-build script did not succeed. No such property: success for 
> class: Script1
>
> After reverting the above commit, it succeeds:
>
> [INFO] --- maven-invoker-plugin:1.5:run (integration-test) @ rpm-maven-plugin 
> ---
> [INFO] Building: rpm-reactor/pom.xml
> [INFO] ..SUCCESS (14.6 s)
> [INFO] Building: rpm-artifact/pom.xml
> [INFO] ..SUCCESS (7.6 s)
> [INFO] Building: rpm-1/pom.xml
> [INFO] ..SUCCESS (6.0 s)
> [INFO] Building: rpm-2/pom.xml
> [INFO] ..SUCCESS (14.5 s)
> [INFO] Building: rpm-disable/pom.xml
> [INFO] ..SUCCESS (10.7 s)
> [INFO] Building: rpm-macros/pom.xml
> [INFO] ..SUCCESS (5.1 s)
> [INFO] Building: rpm-3/pom.xml
> [INFO] ..SUCCESS (5.8 s)
>
> Regards,
> Graham
> --
>

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to