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

add python-defusedxml #10925

Merged
merged 1 commit into from
Mar 30, 2016
Merged

add python-defusedxml #10925

merged 1 commit into from
Mar 30, 2016

Conversation

codebot
Copy link
Contributor

@codebot codebot commented Mar 30, 2016

Tested on Ubuntu. I attempted to chase it down on other distros.

@tfoote
Copy link
Member

tfoote commented Mar 30, 2016

Note this is not available on Saucy so it cannot go into indigo until after the next sync when saucy is disabled.

@wjwwood
Copy link
Member

wjwwood commented Mar 30, 2016

You need OS X (pip).

@codebot
Copy link
Contributor Author

codebot commented Mar 30, 2016

Thanks for the feedback. I'll add a pip key and squash.

In terms of Saucy, I was thinking of only adding the actual rosmaster dependency and code in Kinetic. Is it OK to add the rosdep key still?

Tested on Ubuntu. I attempted to chase it down on other distros.
@tfoote
Copy link
Member

tfoote commented Mar 30, 2016

It's fine to add the rules at any time. We could do it for Kinetic only but this seems like an improvement that could be considered for backporting.

@tfoote tfoote merged commit 55fa05b into master Mar 30, 2016
@tfoote tfoote deleted the defusedxml branch March 30, 2016 21:32
@dirk-thomas
Copy link
Member

The package is available in Saucy.

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.

4 participants