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

adding posix-ipc python package for semaphore support #21

Merged
merged 1 commit into from
Mar 2, 2023
Merged

adding posix-ipc python package for semaphore support #21

merged 1 commit into from
Mar 2, 2023

Conversation

dskard
Copy link
Collaborator

@dskard dskard commented Mar 2, 2023

Description

Adding the Python package posix-ipc so we can use semaphores in test cases.

Connected to #19

Testing Notes

  1. check out this branch 
    git checkout dsk-19-selene1x-posix-ipc
  2. build the docker container 
    make build
  3. check that python in the docker container now has the package posix-ipc installed 
    make run
pip freeze | grep posix-ipc
    should return results like: posix-ipc==1.1.1

rich-rstudio
rich-rstudio approved these changes Mar 2, 2023
View reviewed changes
Copy link

@rich-rstudio rich-rstudio left a comment

Choose a reason for hiding this comment

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

LGTM

@dskard dskard merged commit fb7bbb6 into selene-1.x Mar 2, 2023
@dskard dskard deleted the dsk-19-selene1x-posix-ipc branch March 2, 2023 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rich-rstudio rich-rstudio rich-rstudio approved these changes

@lah95x lah95x Awaiting requested review from lah95x

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dskard @rich-rstudio