search

rimmartin / saxon-node

MIT License
64 stars 6 forks source link

Saxon/C 1.2.0 Major Release is out #24

Open ond1 opened 4 years ago

ond1 commented 4 years ago

We are pleased to announce the release of Saxon/C 1.2.0.

This is a major release of Saxon/C on Linux, Mac OS X and Windows. We have made a number of changes and fixed several bugs in the new release. With this release, support for the Python language is also now available, in addition to the C/C++ and PHP languages already supported. Saxon/C is available in the following products: Saxon-HE/C, Saxon-PE/C and Saxon-EE/C.

Saxon/C home page

Documented changes in the release:

We would think something similar would need to be done in saxon-node.

For a full list of bug fixes see: Release Notes

rimmartin commented 4 years ago

hi @ond1

I'll test it out and update if anything needs it. Thank you!

rimmartin commented 4 years ago

it is compiling and running the XsltProcessor with it's tests! Travis build on linux and mac passing.

I started an interface for the Xslt 3.0 processor and will make a new set of unit tests based on it

rimmartin commented 4 years ago

Hi @ond1,

any way to tell at compile time which edition? bool isSchemaAware(); is unresolved link error on windows HE/C. Would like to avoid it when the user is using the HE/C however when they have a license from you, they can run EE/C or PE/C through this nodejs interface. This nodejs interface is purely MIT license. Only if a user buys a license from you are the P & E features available to them. What do you think?

ond1 commented 4 years ago

Hi @rimmartin, The isSchemaAware() method can be avoided. Under HE that method just returns false. The HE/C code download comes without the PE & EE code base so it won't attempt to run commercial features. Also at compile time without a licensed version certain features will fail and they will see an error telling them that feature requires a license.

rimmartin commented 4 years ago

Under HE isSchemaAware doesn't have an implementation and the windows build crashes. I sed'd it out to make it get past it.

saxon-node isn't locked to HE/C. If one of your customers has a EE or PE license it used to work for them. I only build with HE/C because I'm not a company; just a hobbyist:-)

ond1 commented 4 years ago

Oh yes. That is an omission on our part. I will raise a bug issue against it. The bug fix will be in the next maintenance release, which is a few days away.

Thank and kind regards,

O'Neil

On 24 Oct 2019, at 22:05, rimmartin notifications@github.com wrote:

Under HE isSchemaAware doesn't have an implementation and the windows build crashes. I sed'd it out to make it get past it.

saxon-node isn't locked to HE/C. If one of your customers has a EE or PE license it used to work for them. I only build with HE/C because I'm not a company; just a hobbyist:-)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/rimmartin/saxon-node/issues/24?email_source=notifications&email_token=AA3XSMCDMXHU6PSFUZ4QSCLQQIE2PA5CNFSM4JB2JBPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECGN7KI#issuecomment-546103209, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA3XSMCLMCFNGEYB3MEEBTDQQIE2PANCNFSM4JB2JBPA.

ond1 commented 4 years ago

I have added the implementation for this method in the C++, PHP and Python APIs.

In the C++ API I had to rename the isSchemaAware method to isSchemaAwareProcessor due to some ambiguities with the method name when called in cython. Bug fixed and available in the next maintenance release. See bug issue: https://saxonica.plan.io/issues/4360

rimmartin commented 4 years ago

Thank you much, @ond1 :+1: