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

x:include nits #18

Closed
reschke opened this issue Mar 25, 2017 · 3 comments
Closed

x:include nits #18

reschke opened this issue Mar 25, 2017 · 3 comments

Comments

@reschke
Copy link
Contributor

reschke commented Mar 25, 2017

  1. Reference should be updated to point to https://www.w3.org/TR/2006/REC-xinclude-20061115/

  2. (It should be clarified how much of XInclude needs to be supported. I'm mentioning this because RFC7991 says that includes can not happen where no elements are allowed, but XInclude allows inclusion of plain text as well.)

  3. (Is support of the xpointer attribute required? If so, do we have any guidance how this will work when the document from which to include uses xml2rfc format, and the pointer relies on the id-ness of an attribute?)

  4. The final example uses the namespace prefix "x:", while the preceding text uses "xi:".

@hildjj
Copy link
Collaborator

hildjj commented Mar 28, 2017

  1. Agree

  2. There are uses for plain text, but I understand your point. Want other opinions.

  3. If the answer to 2 is "not everything", then yes, we should disallow xpointer.

  4. Bug.

@reschke
Copy link
Contributor Author

reschke commented Apr 19, 2020

Moved points 2. and 3. to #128

@reschke
Copy link
Contributor Author

reschke commented Apr 19, 2020

  1. add a note about the line breaks in the example

reschke added a commit to reschke/draft-iab-xml2rfc-v3-bis that referenced this issue Apr 19, 2020
interim-rfc-series-project-manager added a commit that referenced this issue Apr 20, 2020
Editorial fixes wrt x:include (#18)
@reschke reschke closed this as completed Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants