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

Deprecate Saxon 9.7 or less #596

Closed
AirQuick opened this issue Aug 29, 2019 · 7 comments
Closed

Deprecate Saxon 9.7 or less #596

AirQuick opened this issue Aug 29, 2019 · 7 comments
Assignees
Milestone

Comments

@AirQuick
Copy link
Member

Saxon 9.7 or less should be deprecated explicitly:

  • In Release Note
  • In Wiki (probably Installation)
  • In run time message?

Rationale:

  • Burden on development and testing, both on XSLT and XQuery, primarily because of limited support of XPath/XSLT/XQuery 3.x, partially because of differences in command interface
  • No longer maintained by Saxonica
@AirQuick AirQuick added this to the v1.5.0 milestone Sep 8, 2019
@AirQuick
Copy link
Member Author

AirQuick commented Sep 9, 2019

@xspec/xspec
I'd like to propose

  • Create a Wiki page, titled Requirements, under Installation section.
  • Move Requirements blocks from Installation on Mac and Linux and Installation on Windows to the new Requirements page.
  • Add these warnings to the new Requirements page.
    • Saxon 9.7 (or less) is deprecated. XSpec is not tested regularly on 9.7.
    • For the moment, XSpec will be tested only occasionally on 9.7. (*1)
    • At some point in the near future, XSpec will stop being tested at all on 9.7 and probably will stop functioning on 9.7 HE (*2)

[1] I'm testing XSpec occasionally on 9.7 for some practical reasons. So I'll continue it for the moment.
[2] "All XSLT 3.0 features require at least Saxon-PE."

@cirulls
Copy link
Member

cirulls commented Sep 10, 2019

@AirQuick: this sounds good to me, the deprecation warning will be very useful for helping people migrating to the latest Saxon.

@AirQuick
Copy link
Member Author

Done.

@AirQuick AirQuick self-assigned this Sep 18, 2019
@AirQuick
Copy link
Member Author

  • Add these warnings to the new Requirements page.

Done.
Reflecting our CI configuration, I also added a statement, "XSpec is tested on Java 8."

@AirQuick
Copy link
Member Author

  • In run time message?

Done in #625. Thanks for the review, @cirulls and @galtm.

@cirulls
Copy link
Member

cirulls commented Sep 20, 2019

I think the move to Java 8 for testing XSpec and the deprecation message for Saxon 9.7 could be mentioned explicitly in the highlights for the next release notes so that users are aware of it.

@AirQuick
Copy link
Member Author

I think the move to Java 8 for testing XSpec and the deprecation message for Saxon 9.7 could be mentioned explicitly in the highlights for the next release notes so that users are aware of it.

How we communicate it, I'll leave it up to you.
Ironically, for the first time, we have continuous testing on Saxon 9.7 :)

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

No branches or pull requests

2 participants