-
Notifications
You must be signed in to change notification settings - Fork 51
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
modify the ODD to allow for multiple refs #38
Comments
Could we add a message like "This file is autogenerated using the command ..." to the top? This might prevent more cases of manual editing. |
Can we add the new reference requirement to the ODD?
|
That's what the ticket is for :-) Posted it because I know I won't be able to act on this immediately. |
Oooooops. May I please see a snippet of XML that expresses the desired information and at the same time is not valid? It's struck that there's probably something wrong with that XML rather than the schema, in this case. Let's check that option first. TiA! |
Hi again. I've looked at the ODD (gosh, it does need a refresh), and there is no reference to a forced |
Ah, OK, Karl edited the schema directly. OK, so it's really good that this topic came up. |
@piotr I've updated the RNG again. All we want is a machine-readable licencing
statement. Karl has implemented ref with the clear text name as content and the
licence URL as target. I've added the possibility to have two ref's for
dual-licencing.
I do not care exactly how it is implemented, but we ought to find a way how to
encode the licence name and ideally the URL in a way that we can verify using a
schema of your choice.
|
I feel embarrassed just looking at our ODD, so I will change it asap. Will add some Schematron for the |
P5 (now?) has a licence element, which would fit better than |
Just a note for now: it's a rather bad idea to edit the .rng directly, because it's regenerated after each change of the ODD. So when our schema is tightened, there will be a new .rng.
So we simply need a new ODD, more relaxed in this respect.
On 29/11/17 22:16, Sebastian Humenda wrote:
The text was updated successfully, but these errors were encountered: