Hi Saurabh,

On 09/19/12 04:28 AM, Saurabh Vyas wrote:
Please review the changes for

7199160 Remove the exclude dependency on the exa* platforms within the
package/pkg manifests

webrev :
https://cr.opensolaris.org/action/browse/pkg/saurabhv/7199160-rev-0/webrev/

Technically, the change seems fine, but there's no information in the bug report explaining why this needs to happen, and I'm not familiar enough with the exa* package layout to say if the change is complete.

Is there followup work required in the repository that contains exa*, or is it simply a case of that product team needing to be able to update to the latest available pkg(5) bits as a result of new features?

If there was more detail in the CR here, or detail in the original defect.opensolaris.org bug where these dependencies were introduced (19023) then this changeset would be easier to review.

        cheers,
                        tim
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to