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.

Sign up for GitHub

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

Jump to bottom

Add ProcessRunner and PIDFile #4064

Closed
2 tasks done
aleks-f opened this issue Jun 28, 2023 · 0 comments · Fixed by #4225
Closed
2 tasks done

Add ProcessRunner and PIDFile #4064

aleks-f opened this issue Jun 28, 2023 · 0 comments · Fixed by #4225
Assignees
@aleks-f @pavledragisic
Labels
enhancement fixed
Milestone
Release 1.13.0

Comments

@aleks-f
Copy link
Member

aleks-f commented Jun 28, 2023 •
edited
Loading

  • Add ProcessRunner class, wrapping the existing Foundation::ProcessHandle functionality
  • Add PIDFile class, wrapping a PID file
@aleks-f aleks-f added this to the Release 1.13.0 milestone Jun 28, 2023
@aleks-f aleks-f changed the title Add ProcessRunner Add ProcessRunner and PIDFile Oct 26, 2023
pavledragisic pushed a commit that referenced this issue Oct 26, 2023
feat(Foundation): PIDFile and ProcessRunner #4064
c55e6e7
@pavledragisic pavledragisic linked a pull request Oct 26, 2023 that will close this issue
Add ProcessRunner and PIDFile #4225
Merged
pavledragisic pushed a commit that referenced this issue Oct 30, 2023
feat(Foundation): add PIDFile and ProcessRunner Tests #4064
590dc50
pavledragisic pushed a commit that referenced this issue Oct 31, 2023
fix(Foundation): failing ProcessRunner Test #4064
95a107b
pavledragisic pushed a commit that referenced this issue Nov 2, 2023
fix(PIDFile): remove append argument #4064
b02c2cc
pavledragisic pushed a commit that referenced this issue Nov 2, 2023
remove Windows TODO from ProcessRunner #4064
674104f
pavledragisic pushed a commit that referenced this issue Nov 2, 2023
feat(ProcessRunnerTest): add line to checkTimeout #4064
a08f5b5
pavledragisic pushed a commit that referenced this issue Nov 2, 2023
fix(ProcessRunner): add done flag to run() #4064
f417b4c
pavledragisic pushed a commit that referenced this issue Nov 3, 2023
fix(ProcessRunnerTest): add missing pidFile argument #4064
0ad17b7
pavledragisic pushed a commit that referenced this issue Nov 6, 2023
chore(ProcessRunner): remove comments #4064
025b1a0
pavledragisic pushed a commit that referenced this issue Nov 6, 2023
fix(ProcessRunner): add runCount flag #4064
ecb2d36
aleks-f added a commit that referenced this issue Nov 8, 2023
@aleks-f
fix(ProcessRunner): thread sanitizer reported data races #4064
98d98c8
pavledragisic pushed a commit that referenced this issue Nov 9, 2023
fix(build): pass env var to testrunner #4064
71d8f19
pavledragisic pushed a commit that referenced this issue Nov 9, 2023
chore(PIDFile): remove ; in comments #4064
1b1feaf
pavledragisic pushed a commit that referenced this issue Nov 20, 2023
feat(ProcessRunner): add Win argument format #4064
ce8ba27
pavledragisic pushed a commit that referenced this issue Nov 20, 2023
fix(Tests): add ProcessRunnerTest to vcxproj #4064
9de3ed2
pavledragisic pushed a commit that referenced this issue Nov 20, 2023
fix(Tests): change path to TestApp #4064
196a050
pavledragisic pushed a commit that referenced this issue Nov 20, 2023
feat(Tests): windows processrunner tests #4064
731d770
pavledragisic pushed a commit that referenced this issue Nov 20, 2023
fix(Tests): duplicate ProcessRunnerTest in TestSuite vcxproj #4064
85d9bf8
pavledragisic pushed a commit that referenced this issue Nov 20, 2023
fix(CodeQL): sw declaration hides variable #4064
def4dbe
pavledragisic pushed a commit that referenced this issue Nov 22, 2023
fix(Build): missing include/PIDFile.h buildWin #4064
611e811
pavledragisic pushed a commit that referenced this issue Nov 22, 2023
fix(Build): add PocoFoundation depend in buildWin #4064
7b69005
@aleks-f aleks-f added the fixed label Nov 23, 2023
aleks-f added a commit that referenced this issue Nov 24, 2023
@pavledragisic @aleks-f
Add ProcessRunner and PIDFile (#4225)
70bb3a4 
* feat(Foundation): PIDFile and ProcessRunner #4064
* feat(Thread): optional signal blocking on POSIX #2978
* fix(ProcessRunner):remove logger, code enhancement #4225
* feat(Foundation): add PIDFile and ProcessRunner Tests #4064
* fix(Foundation): failing ProcessRunner Test #4064
* fix(PIDFile): remove append argument #4064
* remove Windows TODO from ProcessRunner #4064
* feat(ProcessRunnerTest): add line to checkTimeout #4064
* fix(ProcessRunner): add done flag to run() #4064
* fix(ProcessRunnerTest): add missing pidFile argument #4064
* chore(ProcessRunner): remove comments #4064
* fix(ProcessRunner): add runCount flag #4064
* fix(test): SharedLibrary and Class tests paths
* fix(ProcessRunner): thread sanitizer reported data races #4064
* fix(build): pass env var to testrunner #4064
* chore(PIDFile): remove ; in comments #4064
* feat(ProcessRunner): add Win argument format #4064
* fix(Tests): add ProcessRunnerTest to vcxproj #4064
* fix(Tests): change path to TestApp #4064
* feat(Tests): windows processrunner tests #4064
* fix(Tests): duplicate  ProcessRunnerTest in TestSuite vcxproj  #4064
* fix(CodeQL): sw declaration hides variable  #4064
* fix test binaries path for cmake
* fix(Build): missing include/PIDFile.h buildWin #4064
* fix(Build): add PocoFoundation depend in buildWin #4064
* feat(ProcessRunner): test process launching multiple threads #2976

---------

Co-authored-by: Pavle <pavle@debian-gnu-linux-11.localdomain>
Co-authored-by: Alex Fabijanic <alex@pocoproject.org>
aleks-f added a commit that referenced this issue Nov 27, 2023
@pavledragisic @aleks-f
Add ProcessRunner and PIDFile (#4225)
9f4c068 
* feat(Foundation): PIDFile and ProcessRunner #4064
* feat(Thread): optional signal blocking on POSIX #2978
* fix(ProcessRunner):remove logger, code enhancement #4225
* feat(Foundation): add PIDFile and ProcessRunner Tests #4064
* fix(Foundation): failing ProcessRunner Test #4064
* fix(PIDFile): remove append argument #4064
* remove Windows TODO from ProcessRunner #4064
* feat(ProcessRunnerTest): add line to checkTimeout #4064
* fix(ProcessRunner): add done flag to run() #4064
* fix(ProcessRunnerTest): add missing pidFile argument #4064
* chore(ProcessRunner): remove comments #4064
* fix(ProcessRunner): add runCount flag #4064
* fix(test): SharedLibrary and Class tests paths
* fix(ProcessRunner): thread sanitizer reported data races #4064
* fix(build): pass env var to testrunner #4064
* chore(PIDFile): remove ; in comments #4064
* feat(ProcessRunner): add Win argument format #4064
* fix(Tests): add ProcessRunnerTest to vcxproj #4064
* fix(Tests): change path to TestApp #4064
* feat(Tests): windows processrunner tests #4064
* fix(Tests): duplicate  ProcessRunnerTest in TestSuite vcxproj  #4064
* fix(CodeQL): sw declaration hides variable  #4064
* fix test binaries path for cmake
* fix(Build): missing include/PIDFile.h buildWin #4064
* fix(Build): add PocoFoundation depend in buildWin #4064
* feat(ProcessRunner): test process launching multiple threads #2976

---------

Co-authored-by: Pavle <pavle@debian-gnu-linux-11.localdomain>
Co-authored-by: Alex Fabijanic <alex@pocoproject.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aleks-f aleks-f

@pavledragisic pavledragisic

Labels
enhancement fixed
Projects
None yet
Milestone
Release 1.13.0
Development

Successfully merging a pull request may close this issue.

Add ProcessRunner and PIDFile pocoproject/poco
2 participants
@aleks-f @pavledragisic