-
Notifications
You must be signed in to change notification settings - Fork 31
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
Change operation for OpenEmbedded when --only is specified (#224) #233
Change operation for OpenEmbedded when --only is specified (#224) #233
Conversation
ca6e21a
to
982e11b
Compare
@@ -45,8 +45,11 @@ def commit_changes(self, distro, commit_msg): | |||
'files/{0}/newer-platform-components.list'.format(distro)) | |||
self.repo.git.add( | |||
'files/{0}/superflore-change-summary.txt'.format(distro)) | |||
info('Committing to branch {0}...'.format(self.branch_name)) | |||
self.repo.git.commit(m=commit_msg) | |||
if self.repo.git.status('--porcelain') == '': |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add this behavior to the Gentoo generator as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. I'll move this into a separate commit and add the change for the Gentoo generator (and create a new issue).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new issue is #245.
982e11b
to
a2ae3c8
Compare
Hi @allenh1 is there anything blocking this PR? We're almost ready to publish updated meta-ros with recipes generated with superflore modified with all these open PRs and it would be nice to have it in superflore/master before we do. |
de6b088
to
7b6d40d
Compare
7b6d40d
to
a795544
Compare
- The files under conf and files created by superflore-gen-oe-recipes are only useful when all of the recipes for a ROS distro are being generated => don't generate them when --only is specified. - Use a slighly different commit message when --only is specified. - This commit resolves issue ros-infrastructure#224.
a795544
to
26ed7fa
Compare
@allenh1 < Rebased to current HEAD of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
…structure#233) - The files under conf and files created by superflore-gen-oe-recipes are only useful when all of the recipes for a ROS distro are being generated => don't generate them when --only is specified. - Use a slighly different commit message when --only is specified. - This commit resolves issue ros-infrastructure#224.
The files under conf and files created by superflore-gen-oe-recipes
are only useful when all of the recipes for a ROS distro are being
generated => don't generate them when --only is specified.
Use a slightly different commit message when --only is specified.