Skip to content

Update Dockerfile#101

Merged
pablogs9 merged 2 commits intomicro-ROS:ironfrom
ncnynl:iron
Sep 14, 2023
Merged

Update Dockerfile#101
pablogs9 merged 2 commits intomicro-ROS:ironfrom
ncnynl:iron

Conversation

@ncnynl
Copy link
Copy Markdown
Contributor

@ncnynl ncnynl commented Sep 13, 2023

Change entry script call

Change entry script call
Remove redundant code
Copy link
Copy Markdown
Contributor Author

@ncnynl ncnynl left a comment

Choose a reason for hiding this comment

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

Remove redundant code

@pablogs9
Copy link
Copy Markdown
Member

Could you explain the rationale of this PR a bit?

Copy link
Copy Markdown
Contributor Author

@ncnynl ncnynl left a comment

Choose a reason for hiding this comment

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

ENTRYPOINT ["/entrypoint.sh"] it will have permission issues, causing it to not work properly when running docker container. so change into ENTRYPOINT ["/bin/sh", "/entrypoint.sh"] can fixed it

@pablogs9
Copy link
Copy Markdown
Member

@mergify backport humble rolling

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Sep 14, 2023

backport humble rolling

✅ Backports have been created

Details

@pablogs9 pablogs9 merged commit f8ebbaf into micro-ROS:iron Sep 14, 2023
mergify bot pushed a commit that referenced this pull request Sep 14, 2023
* Update Dockerfile

Change entry script call

* Update Dockerfile

Remove redundant code

(cherry picked from commit f8ebbaf)
mergify bot pushed a commit that referenced this pull request Sep 14, 2023
* Update Dockerfile

Change entry script call

* Update Dockerfile

Remove redundant code

(cherry picked from commit f8ebbaf)
pablogs9 pushed a commit that referenced this pull request Sep 14, 2023
* Update Dockerfile

Change entry script call

* Update Dockerfile

Remove redundant code

(cherry picked from commit f8ebbaf)

Co-authored-by: ncnynl <[email protected]>
pablogs9 pushed a commit that referenced this pull request Sep 14, 2023
* Update Dockerfile

Change entry script call

* Update Dockerfile

Remove redundant code

(cherry picked from commit f8ebbaf)

Co-authored-by: ncnynl <[email protected]>
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