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

fix for kinetic #19

Merged
merged 1 commit into from
Jan 30, 2018
Merged

fix for kinetic #19

merged 1 commit into from
Jan 30, 2018

Conversation

furushchev
Copy link
Contributor

No description provided.

@dirk-thomas
Copy link
Contributor

@ablasdel @thedash Ping to the maintainers...

@UltronDestroyer
Copy link
Contributor

I haven't maintained this for a year or so. Not sure who is going to take it over.

@dirk-thomas
Copy link
Contributor

You are currently still listed as the maintainer of this package (

<maintainer email="[email protected]">Devon Ash</maintainer>
). So e.g. any notification email from the buildfarm is being sent to you.

@DLu
Copy link

DLu commented Jan 9, 2018

I have reviewed this PR and approve, but commenting is all I'm authorized for. It'd be best if it was tested in other distros besides kinetic, but I'm guessing it'd work.

Note: Releasing this package for kinetic would rely on PR2/pr2_power_drivers#67

@furushchev
Copy link
Contributor Author

@DLu Thank you for review! I also checked that this works on indigo.

@UltronDestroyer
Copy link
Contributor

@dirk-thomas Out of my hands now. AFAIK there is no official PR2 maintainership since Scott Hassan ceased the funding for the PR2 support. The community unfortunately will have to find someone else to take maintainership/release responsibility over them as I'm unable to at the moment.

@k-okada
Copy link
Contributor

k-okada commented Jan 16, 2018

@TheClearpathDash we have proposed the initiative to support this kind of packages at roscon 2017. See
video: https://vimeo.com/236129109#t=34m30s
slides: https://roscon.ros.org/2017/presentations/ROSCon%202017%20Lightning%20113.pdf

and I have start writing a draft of this procedure at
ros-infrastructure/rep#150 (https://github.com/k-okada/rep/blob/0d5b63cc04ba945fec73650f9453d576b939a67a/rep-0152.rst ).

If you're ok, please look at these ticket and add @k-okada to admin/write manber of these repositories.
PR2/pr2_power_drivers#68
#20

@k-okada
Copy link
Contributor

k-okada commented Jan 16, 2018

also please add me to pr2_power_drivers-release and rqt_pr2_dashboard-release repositories.

@v4hn
Copy link
Member

v4hn commented Jan 24, 2018

+1 for the pull-request here.

@awesomebytes
Copy link
Contributor

+1 for the PR, looks good to me.

@k-okada k-okada merged commit 2ad940e into PR2:hydro-devel Jan 30, 2018
@k-okada
Copy link
Contributor

k-okada commented Jan 30, 2018

Merged, It seems I have write permission, but I do not have admin permission...

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.

7 participants