Open
Conversation
added 30 commits
May 31, 2012 12:15
Remove several warnings printed when running swig by renaming the relevant functions to ones that are compatible with Java. Started working on porting the VTO interfaces to be Swig compatible. (Swig prefers "static const" over "const static.")
Linux and Windows handle case sensitivity differently. Namely, the former thinks that "dnp3test" and "DNP3Test" are different, while the latter believes that they are (mistakenly) the same. Since we generate binaries that match directories in a case insensitive manner (dnp3test->DNP3Test and testset->TestSet), simply adding 'dnp3test' and 'testset' to the .gitignore causes the source code directories with the "same" name to also be ignored. This isn't what we want, so the trick is to tell git to ignore the binaries but include the directories. We do this using the "negate" operator and the "directory-slash" suffix. These rules are at the top of the .gitignore so that all the other rules (like ignoring object files, swap files, etc.) behave like one would expect.
Thanks to William Sember for catching the non-thread-safe code generated by Swig. http://matt.eifelle.com/2007/11/23/enabling-thread-support-in-swig-and-python/
Conflicts: .gitignore
Support for 'make -j X' (where X is the number of parallel build threads to execute) now works properly for the Java Swig bindings.
Link local connections do not work. You need to bind to a global or loopback address.
Helps with backwards compatibility of existing testset XML files
Conflicts: DNP3/StartupTasks.cpp
Moved the ConvertMode() function to being owned by the XmlToConfig class
Moved the ConvertMode() function to being owned by the XmlToConfig class
For a master stack in testset, enabling time sync support was hardcoded to 'true'. This prevented testing with testset where time sync needs to be disabled on the master stack. Now we read the property and feed it into the master stack configuration struct properly.
Member
|
This apparently includes contributions from William Sember, do you know him? We don't have a CLA for him at the moment, which is problematic. |
Author
|
No, I don't know glaslos personally. He submitted these to my fork for inclusion. The patches looked good, just like I would have done, so I pulled it in rather than retyping the fix that he made. |
Member
|
I was thinking about the "Added Python Master demo from William Sember" commit. I just need to figure out legal issues with these contributions. |
Author
|
Oh, but do you really care, since the commit is coming through me, and I have a CLA signed? |
|
Guys has this been resolved? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Created release 1.1.0, which includes a variety of bug fixes since 1.0.0.