reverting commit

Jay Shaughnessy jshaughn at redhat.com
Wed Mar 27 15:04:57 UTC 2013


Ian,

It used to be OK :)  Now we've gotten more formal about public API 
changes and since we expose domain classes in our remotes we now 
consider the loss of a constructor as a public API change.  I agree that 
that change would likely be innocuous, but it got flagged nonetheless...

On 3/27/2013 10:28 AM, Ian wrote:
> Mazz, sorry about that. I thought a constructor of a history class 
> would be safe to change API-wise. In any case, after pulling the 
> latest from master (including your revert), I'm not seeing the GWT 
> compile error anymore, even after cleaning org/rhq/** from my local 
> repo then doing a mvn clean install. Very strange. If I do see it 
> again, I'll create a BZ.
>
> ---------
>
> [Ian - feel free to commit this back in a branch (not master) and 
> create a BZ for this. We'll cherry pick in master at the appropriate 
> time.]
>
> Its been requested that we revert commit df18b71df2d5114d9edc8a26ad42e0
> d7a9036eb2:
>
>     "do not pass not-yet-fully-initialized "this" reference to 
> GroupOperationHistory.addResourceOperationHistory() in 
> ResourceOperationHistory's constructor, as the GWT compiler fails when 
> it sees this (at least in my OS X 10.7.5, Oracle JDK 1.7.0_17 
> environment)"
>
> We are reverting for a couple reasons:
>
>     * This broke a jenkins build - an API change has been introduced 
> and it therefore fails the API check job. (I think its due to a change 
> in a public domain object's constructor). Keep in mind that for awhile 
> now we have a jenkins job that runs to make sure there are no backward 
> compatible changes to public APIs - if it detects one, the build will 
> fail. In this case, it appears a public domain object's constructor 
> was changed thus breaking that API check.
>
>     * Also, There is no BZ for this - we don't know what this 
> addressed. Please make sure BZs are written before committing bug 
> fixes so they can be tracked appropriately.
>
>
>
> _______________________________________________
> rhq-devel mailing list
> rhq-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/rhq-devel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/rhq-devel/attachments/20130327/927c9dd1/attachment.html>


More information about the rhq-devel mailing list