-
Notifications
You must be signed in to change notification settings - Fork 19
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
Can't compile sims after Trick update with "write template directive before template instance (#1679)" pull request #1697
Comments
Please try PR#1696 to see if your problem is fixed |
I ended up having to clone the repo pointed to in PR#1696 and I was able to compile my simulations using it. However, I don't think this is a valid test unless PR#1696 is based off the latest Trick master branch and included the code from PR#1679 that is causing the issue. What is the git command to merge the pull request against my clone of the Trick master branch? |
PR#1696 does include the code from PR#1679. PR#1696 only does't have is a minor documentation update comparing with the master. Any PR#1696 was really for fixing a typo in PR#1679 |
That's good news then and it appears PR#1696 fixed the issue I was seeing. |
Thanks for confirming. |
These changes have been merged into master, so please confirm that the issue is resolved in the latest master commit. |
That fixed it. Thanks. |
The latest Trick with the pull request "write template directive before template instance (#1679)" results in swig file errors and now I can't compile any of my simulations.
For example, when I compile the TrickHLA/sims/SpaceFOM/SIM_sine simulation I get this error:
The Object_py.i file generated with Trick with the pull request is missing the following few lines for the Attribute class (diff Object_py.i. PR1679_Object_py.i):
Object_py.i code segment from Trick that does compile the simulation.
Object_py.i code segment from Trick with Pull Request 1679 that does Not compile the simulation. It is missing the "class Attribute" section.
I am using this swig version:
The text was updated successfully, but these errors were encountered: