Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CMake version variables and tag repository #21

Open
jcfr opened this issue Oct 21, 2013 · 4 comments
Open

Add CMake version variables and tag repository #21

jcfr opened this issue Oct 21, 2013 · 4 comments
Assignees

Comments

@jcfr
Copy link
Member

jcfr commented Oct 21, 2013

No description provided.

@ghost ghost assigned jcfr Oct 21, 2013
jcfr added a commit to jcfr/SlicerExecutionModel that referenced this issue Oct 21, 2013
@jcfr
Copy link
Member Author

jcfr commented Oct 21, 2013

@thewtex @pieper @hjmjohnson @millerjv @finetjul @naucoin

Hi Folks,

Following request to tag a version of the SlicerExecutionModel, I just introduced corresponding CMake variables and set the version to be 1.0.0. Let me know what you think.

Introducing a version would also help keeping track of the change related to the XML structure associated with CLI.

Currently (thanks to @finetjul) we have a XSD schema but there are no clear roadmap allowing our collaborators to know what the plan is ...

@thewtex
Copy link
Member

thewtex commented Oct 21, 2013

+1

What is the versioning plan with regards to the schema? Schema additions
in a minor release bump? Backwards incompatible schema changes in a major
release bump?

On Mon, Oct 21, 2013 at 5:39 PM, Jean-Christophe Fillion-Robin <
[email protected]> wrote:

@thewtex https://github.com/thewtex @pieper https://github.com/pieper
@hjmjohnson https://github.com/hjmjohnson @millerjvhttps://github.com/millerjv
@finetjul https://github.com/finetjul @naucoinhttps://github.com/naucoin

Hi Folks,

Following request to tag a version of the SlicerExecutionModel, I just
introduced corresponding CMake variables and set the version to be 1.0.0.
Let me know what you think.

Introducing a version would also help keeping track of the change related
to the XML structure associated with CLI.

Currently (thanks to @finetjul https://github.com/finetjul) we have a XSD
schemahttps://github.com/Slicer/SlicerExecutionModel/blob/master/ModuleDescriptionParser/ModuleDescription.xsdbut there are no clear roadmap allowing our collaborators to know what the
plan is ...


Reply to this email directly or view it on GitHubhttps://github.com//issues/21#issuecomment-26738786
.

@hjmjohnson
Copy link
Member

This looks like a good plan Matt.

From: Matt McCormick [email protected]
Reply-To: Slicer/SlicerExecutionModel
<reply+i-21330520-9d6bc595f3c2a956bb89b011c89286d8d0c01f3f-313970@reply.gith
ub.com>
Date: Monday, October 21, 2013 1:14 PM
To: Slicer/SlicerExecutionModel [email protected]
Cc: Hans Johnson [email protected]
Subject: Re: [SlicerExecutionModel] Add CMake version variables and tag
repository (#21)

+1

What is the versioning plan with regards to the schema? Schema additions
in a minor release bump? Backwards incompatible schema changes in a major
release bump?

On Mon, Oct 21, 2013 at 5:39 PM, Jean-Christophe Fillion-Robin <
[email protected]> wrote:

@thewtex https://github.com/thewtex @pieper https://github.com/pieper
@hjmjohnson https://github.com/hjmjohnson
@millerjvhttps://github.com/millerjv
@finetjul https://github.com/finetjul @naucoinhttps://github.com/naucoin

Hi Folks,

Following request to tag a version of the SlicerExecutionModel, I just
introduced corresponding CMake variables and set the version to be 1.0.0.
Let me know what you think.

Introducing a version would also help keeping track of the change related
to the XML structure associated with CLI.

Currently (thanks to @finetjul https://github.com/finetjul) we have a XSD

schema<https://github.com/Slicer/SlicerExecutionModel/blob/master/ModuleDescript
ionParser/ModuleDescription.xsd>but there are no clear roadmap allowing our
collaborators to know what the
plan is ...


Reply to this email directly or view it on
GitHub<#21 (comment)
38786>
.


Reply to this email directly or view it on GitHub
<#21 (comment)
759> .

@millerjv
Copy link
Member

My recollection is that the Schema does not match the XML parser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants