Skip to content

What's up with Requirement 9? #309

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.

Sign up for GitHub

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

Jump to bottom
Closed
jyutzler opened this issue Mar 3, 2017 · 1 comment
Closed

What's up with Requirement 9? #309

jyutzler opened this issue Mar 3, 2017 · 1 comment
Milestone
1.2-comment period

Comments

@jyutzler
Copy link
Contributor

jyutzler commented Mar 3, 2017

If I'm reading this right, http://www.geopackage.org/spec/#_requirement-9 says:

SHALL NOT include any OMIT options from http://www.sqlite.org/compile.html#omitfeatures.

That would mean checking individually for all 30 or so of these things and failing if any of them are present. This seems like a bad idea. What is the point? Can we nuke this requirement?
@jyutzler jyutzler added this to the 1.2-comment period milestone Mar 3, 2017
@jyutzler jyutzler mentioned this issue Mar 7, 2017
Merged
@jyutzler
Copy link
Contributor Author

jyutzler commented Mar 7, 2017

In today's SWG we voted that this requirement is out of scope for the standard and that it should get a line-through. The ATS test will be removed.

@jyutzler jyutzler closed this as completed Mar 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.2-comment period
Development

No branches or pull requests
1 participant
@jyutzler