-
Notifications
You must be signed in to change notification settings - Fork 93
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
recording.wsdl seems to be malformatted #214
Comments
If you copy the wsdl file to your PC you also have to copy the related schema files to the relative appropriate folder or change the import statements. Note that you can directly copy the URL of the wsdl file to the Microsoft tools. For recording.wsdl it is |
Command is run after cloning repo not on only file. And it is running properly on other wsdl files. |
Just verified that it works with following url |
it didnt work as well. Procedure which i apply is as follow; I expect that it shoukd work on pure repo |
The log above shows as URL: file:/D:/bvkrepo/devproj/specs/wsdl/ver10/recording.wsdl From my understanding file must be trailed with three slashes. |
I dont get your point. |
After having verified that both Oxygen XML verification passes and Visual Studio Service References imports the file online and from disk, ONVIF conclusion is that the file definition is correct. Nevertheless we are interested to know in case you should happen to find out the root cause so that this information can be shared in the community. |
Interesting because errors are so obvious. I shared again error lines which comes from svcutil. Btw, it is same error which is in my previous issue and it was fixed with latest commit. Take a look at #193 Error: Cannot import wsdl:portType Error: Cannot import wsdl:binding |
Events (and #193) are a special case, because ONVIF is relying on OASIS base notification schema. which have same name for port type and complex type. Unfortunately Microsoft tools cannot cope with that name collision while others work fine. For that reason ONVIF provides a special version including a modified OASIS component that compiles with Visual Studio. I've double checked the tt:StringList definition. It is defined in onvif.xsd and the Oxygen editor finds it as long as the import points to the correct file. |
You can safely ignore these validation errors. ONVIF is using XML schema 1.1 while the Microsoft parser sticks to 1.0 and the compiler complains about xs:any after optional elements. The generator still generates code. |
Verified with VS 2022 that also this version works as expected. Shows validation errors as in screenshots above due to XML 1.0 validation. However validation errors have no impact on project compilation and linking. |
During generating codes ( c# and java ) from recording.wsdl, i get errors. And both convertion tools ( svcutil and wsimport ) return with parsing erros. It seems wsdl file is not well formatted. Here are the outputs of both command line tools.
wsimport recording.wsdl parsing WSDL... [ERROR] Could not find wsdl:service in the provided WSDL(s): file:/D:/bvkrepo/devproj/specs/wsdl/ver10/recording.wsdl At least one WSDL with at least one service definition needs to be provided. Failed to parse the WSDL.
The text was updated successfully, but these errors were encountered: