Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: eclipse-emf-validation - Verify the integrity of EMF models
https://bugzilla.redhat.com/show_bug.cgi?id=537452
Summary: Review Request: eclipse-emf-validation - Verify the integrity of EMF models Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: fedora@matbooth.co.uk QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
This package depends eclipse-mdt-ocl, whose review request is bug #536684.
Spec URL: http://mbooth.fedorapeople.org/reviews/eclipse-emf-validation.spec SRPM URL: http://mbooth.fedorapeople.org/reviews/eclipse-emf-validation-1.3.1-1.fc12.s...
Description: The validation component provides the following capabilities.
* Constraint Definition - Provides API for defining constraints for any EMF meta-model (batch and live constraints). * Customizable model traversal algorithms - Extensibility API to support meta-models that require custom strategies for model traversal. * Constraint parsing for languages - Provides support for parsing the content of constraint elements defined in specific languages. The validation framework provides support for two languages: Java and OCL. * Configurable constraint bindings to application contexts - API support to define "client contexts" that describe the objects that need to be validated and to bind them to constraints that need to be enforced on these objects. * Validation listeners - Support for listening to validation events.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
Mat Booth fedora@matbooth.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |537454
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
Mat Booth fedora@matbooth.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |536684
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
Lubomir Rintel lkundrak@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |lkundrak@v3.sk AssignedTo|nobody@fedoraproject.org |lkundrak@v3.sk Flag| |fedora-review?
--- Comment #1 from Lubomir Rintel lkundrak@v3.sk 2009-11-21 14:28:02 EDT --- * Package name and version are in accordance with packaging guidelines * SPEC file clean, legible and well-commented * License tag correct, license packaged as %doc * RPMLint happy * a couple of warnings spelling errors in fc13, all of them false positives * Filelist okay, for directories it uses and does now it depends on packages that provide them * Provides/Require list ok (mock build did not finish yet)
Objections:
1.) Why do you check stuff out from CVS?
2.) Please don't pretty-format the Description, please use complete sentences. It does not always look well in package management front-ends.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
--- Comment #2 from Lubomir Rintel lkundrak@v3.sk 2009-11-21 18:51:08 EDT --- * mock build succeeded
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
Lubomir Rintel lkundrak@v3.sk changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #3 from Lubomir Rintel lkundrak@v3.sk 2009-11-22 08:35:24 EDT --- 1.) is answered in bug #536683. Please re-format the description before importing to CVS!
APPROVED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
--- Comment #4 from Mat Booth fedora@matbooth.co.uk 2009-11-24 05:34:48 EDT --- (In reply to comment #3)
1.) is answered in bug #536683. Please re-format the description before importing to CVS!
Done:
Spec URL: http://mbooth.fedorapeople.org/reviews/eclipse-emf-validation.spec SRPM URL: http://mbooth.fedorapeople.org/reviews/eclipse-emf-validation-1.3.1-2.fc12.s...
APPROVED
New Package CVS Request ======================= Package Name: eclipse-emf-validation Short Description: Verify the integrity of EMF models Owners: mbooth Branches: F-12
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
Mat Booth fedora@matbooth.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #5 from Mat Booth fedora@matbooth.co.uk 2009-11-24 05:40:15 EDT --- Apologies, I rebuilt the SRPM on a F-10 box, so please s/fc12/fc10/ in my last comment. Also forgot to set the cvs flag... d'oh.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #6 from Kevin Fenzi kevin@tummy.com 2009-11-27 00:30:27 EDT --- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
Bug 537452 depends on bug 536684, which changed state.
Bug 536684 Summary: Review Request: eclipse-mdt-ocl - Implementation of the OCL OMG metamodel for Eclipse https://bugzilla.redhat.com/show_bug.cgi?id=536684
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=537452
Mat Booth fedora@matbooth.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
--- Comment #7 from Mat Booth fedora@matbooth.co.uk 2009-12-01 16:23:20 EDT --- Built successfully in rawhide, closing.
package-review@lists.fedoraproject.org