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

Add script to call and process output of makeEnums.py in CODA-OSS #59

Merged
merged 2 commits into from
Apr 5, 2016

Conversation

JonathanMeans
Copy link
Contributor

No description provided.

os.remove(target)

with open(target, 'w') as f:
f.write("""/* =========================================================================
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't you just call the coda-oss makeEnums.py and point it to six/build/Enums.h.template with the -t flag?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

From: Adam Sylvester [mailto:[email protected]]
Sent: Tuesday, April 05, 2016 10:50 AM
To: ngageoint/six-library [email protected]
Cc: Jonathan Means [email protected]
Subject: Re: [ngageoint/six-library] Add script to call and process output of makeEnums.py in CODA-OSS (#59)

In six/modules/c++/six/build/makeEnums.pyhttp://cp.mcafee.com/d/5fHCN0pdEIfCzBVMsed7apKVJBxAsed7bXVJYsOMOyqejhOrhKZtVMQsTd7bNJ55B5dYQsKcfCN6IdCjS7a_Ia0Fl3mWsKrlgRKDbCTnHEcT7fZvCnNRQS1TnKnjpsd7feecfK8Ith5dqWqJTCel3PWApmU6CQjqpK_8K6zBV55BeXNKVIDeqR4IMzapdIxO-g8Wh-IEjOAwYKv8x7oBImrffNYQswoxaBCWkBm0H5ZVZZ5ZYsrxmfH7BwkhBcCSgVsS-MYUr1vF6y0QJM8Wh-IEjOAwYKv8x7oBziWq81110QgkJ6iQPh1eFEwJFVVEwS21EwmzmTQEq8aJPmrCq8aKCy1niLzk6RqsImd44E6y0i30s85MMc31Ewd506fCzAQkCPhOqejt0e2:

+script = os.path.join(os.getcwd(), 'externals', 'coda-oss',

  •                  'build', 'scripts', 'makeEnums.py')
    

+inputFile = os.path.join(os.path.split(file)[0], 'enums.txt')

+outDir = os.path.join(os.path.split(file)[0], os.pardir, 'include', 'six')

+assert os.path.exists(script)

+assert os.path.exists(inputFile)

+assert os.path.exists(outDir)

+target = os.path.join(outDir, 'Enums.h')

+if os.path.exists(target):

  • os.remove(target)

+with open(target, 'w') as f:

  • f.write("""/* =========================================================================

Can't you just call the coda-oss makeEnums.py and point it to six/build/Enums.h.template with the -t flag?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttp://cp.mcafee.com/d/1jWVIp40USyM-qenD1MUQsFCXCSm6hMUQsLLCTNPb3a9EVd79J6XRTD3hPsQsL6QkmkkTPhOUM-r4qMSpfosH-ME2BkdrFOVJl3mWsKrtuKwPss_R-pv7njo7tuVtdBMQsYUUM-UyNR4kRHFGTuoVkffGhBrwqrjdFCXYyUqenAkmkXL6XCOsVHkiP2cFASO7bV0zF7WOxfai3OVYy4tymNpIY_7PhPzIz_FDKfFSoohgZddVxZCNEVtd7bO1EVpogKghR5_tvgvjjQ38LLfLELLzzsaNZoYI2ycFASO7bCTS7D3obZ8Qg6BK17ifRB2ukA7BPV48X4Iqnjh08886y2BEOmCq89Rd45Jffd46Mgd42QqS-B3h1lKqPsPh1lQQgaWlYqwSHjByNEwB0Qg2go3x0K61wod41EE0NYQsCyASqejhOrz5oYaCixCnJ6

The information contained in this communication is confidential, is intended only for the use of the recipient(s) named above, and may be legally privileged. If the reader of this message is not the intended recipient, you are hereby notified that any dissemination, distribution, or copying of this communication is strictly prohibited. If you have received this communication in error, please re-send this communication to the sender and delete the original message or any copy of it from your computer system.

@asylvest asylvest merged commit 049a529 into master Apr 5, 2016
@asylvest asylvest deleted the makeEnums branch April 5, 2016 19:09
JonathanMeans pushed a commit that referenced this pull request Feb 24, 2020
47c416f Merge pull request #59 from mdaus/math-round
37526a4 Merge pull request #61 from mdaus/python_ld_update
5772485 Fixed OSX -bundle removal code
39820a3 Removed conflicting -bundle from osx linkflags
05df427 More changes for compiling on OSX
be7d392 Modified to hopefully support OSX's ld
eee65f8 Hopefully adds support for ld >2.17
b92bf61 Merge pull request #57 from mdaus/SysExecTerminate
4cd1cb4 adding static keyword
1d1d748 updating constants capitalization
3288100 Fixed banners and added missing std namespaces
f4f60fe Added math::round methods
c105967 removing the brackets and unnecessary break
2fc9ea7 Merge pull request #58 from mdaus/osx_config
0b70f4e Fixed typo in OSX code
327e8b4 Hopefully adding OSX support to sys-c++
9b60823 Updates to address all review updates except the curly braces in the case statement. Waiting for further clarification there
4a61eef updating the pipe naming
6d79b5e made the executive decision std::cin will be passed via the parent's std::cin. This is a simplification that should be useful in this context. This simplifies the class a bit, and provides all the functionality required on known applications.
00bb171 we can't close the pipes sent into the subproces or it will close early
f41b380 adding the ability to add arguments to the processing
a403721 updated the naming based on a #define from UNICODE
4786fde additional cleanup and closure of pipes
05e039c removing this debugging job
a4cd2da Updated to use the non-deprectated calls, and long buffering was the problem before. Setting the buffer size to 1 solves this issue.
a02211f preliminary check-in. This is working when I right to the parent's stdout pipe, but not in the FILE* context.
4332dc2 Updates to get the Linux side matching what I did for FSG. One character makes all the difference
16ce33d updating to work on Linux
a9da04a This is a branch for terminating the child process swiftly without discretion. For this we must create our own popen command, which does not internally perform a wait() command. This will allow the subprocess to schedule, but not require the completion of a job in order to terminate. This helps in the case when the parent process is terminated, and subprocess continues on in a zombie state until its underlying process legitimately completes. Now when the parent dies, so does the child.

git-subtree-dir: externals/coda-oss
git-subtree-split: 47c416f92b195938d2b089e77f6eaf74e0468ce2
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

Successfully merging this pull request may close these issues.

2 participants