[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] ant pull request #68: bz-62655 throw a BuildException from augment task

GitHub user jaikiran opened a pull request:

    bz-62655 throw a BuildException from augment task 

    The manual of the augment task[1] states that it's supposed to throw a `BuildException` if the referenced id value isn't known. I admit that the bugzilla is more about the id attribute not being specified whereas the manual seems to talk about the value of id being unknown reference, but I think the issue itself can be considered valid.
    The referenced bugzilla issue shows that it throws an `IllegalStateException`. That exception then does indeed get thrown as a BuildException[2] but the `reason` that gets reported to the build listeners[3] is the original cause (in this case the `IllegalStateException`).
    The commit here is trivial and it throws the `BuildException` from the `augment` task when either `id` isn't specified or it points to an unknown reference. However, given that it appears that this task has always been in this manner, I wanted to make sure there isn't any specific reason of its current implementation.
    There's already tests for this task which pass both with and without this change[4]

You can merge this pull request into a Git repository by running:

    $ git pull bz-62655-19x

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #68
commit 21de7add9a935c4ae61716cce269f58db26e949a
Author: Jaikiran Pai <jaikiran@...>
Date:   2018-08-25T11:43:31Z

    bz-62655 throw a BuildException from augment task if the id attribute isn't specified or if the value points to an unknown reference



To unsubscribe, e-mail: dev-unsubscribe@xxxxxxxxxxxxxx
For additional commands, e-mail: dev-help@xxxxxxxxxxxxxx