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

Question about testing on Windows #12

Open
smnrgrs opened this issue Jun 26, 2024 · 3 comments
Open

Question about testing on Windows #12

smnrgrs opened this issue Jun 26, 2024 · 3 comments

Comments

@smnrgrs
Copy link

smnrgrs commented Jun 26, 2024

This looks like a nice tool. How is it being run/tested currently - on Linux?

I ask because I've had trouble trying to get it to work on Windows. It goes well (discovers topics, readers, writers) until you try to load an IDL file, which fails as I think that idl processing on windows for python language is broken.

I'm trying to figure out where it goes wrong (it finds and dynamically loads the python lib ok) but haven't got far yet.

@trittsv
Copy link
Contributor

trittsv commented Jun 26, 2024

Hey, i am developing it mostly on windows and macos. Did you use the build.bat?
Which cyclondds and which cyclonedds-python and which cyclonedds-insight version are you using (commit hash)?

If you execute it via python you may you have to duplicate the python.exe and name it python3.exe.
There must be a python3.exe exe in your path because https://github.com/eclipse-cyclonedds/cyclonedds/blob/d7db65c7f9def668af029890c351de6186977d58/src/tools/idlc/src/idlc/generator.c#L167

Or use the compiled exe (build.bat) instead of execute it directly via python.

@trittsv
Copy link
Contributor

trittsv commented Jun 26, 2024

Here is the related issue in the c repo: eclipse-cyclonedds/cyclonedds#2050

@smnrgrs
Copy link
Author

smnrgrs commented Jun 26, 2024

I didn't use build.bat, I'm executing using python. I had already duplicated python->python3.
The app runs and discovers readers/writers/topics. I wanted to see the actual topic instance updates, and this is where I have trouble because IDL loading/processing doesn't work.

I'm running insight 1a3d732 from 19 Jun. I'll have to check the versions of cyclone but they are a week or two off the latest at most.

What I typically see is:
C:\bin\idlc -l py -x appendable -o C:\Users\SR\AppData\Roaming\cyclonedds\CycloneDDSInsight\datamodel\py C:\Users\SR\AppData\Roaming\cyclonedds\CycloneDDSInsight\datamodel\idl\MyTypes_cyclonedds.idl
Process failed with error code: -1073740940

If I run the command above manually I see no py files generated. Idlc seems to silently fail.
Do you import IDLs as part of your testing? Or if I use cyclone options -DENABLE_TOPIC_DISCOVERY=YES -DENABLE_TYPE_DISCOVERY=YES will I get the topic contents anyway?

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