Skip to content

Add support, tests and documentation for Gentoo#335

Merged
ghoneycutt merged 1 commit into
voxpupuli:masterfrom
optiz0r:feature_gentoo
Oct 12, 2016
Merged

Add support, tests and documentation for Gentoo#335
ghoneycutt merged 1 commit into
voxpupuli:masterfrom
optiz0r:feature_gentoo

Conversation

@optiz0r
Copy link
Copy Markdown
Contributor

@optiz0r optiz0r commented Oct 8, 2016

No description provided.

:id => 'root',
:kernel => 'Linux',
:osfamily => 'Gentoo',
:operatingsystem => 'Gentoo',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

you must specify :lsbdistcodename here for TravisCI to be happy.

https://travis-ci.org/stankevich/puppet-python/jobs/166102244

@ghoneycutt
Copy link
Copy Markdown
Member

Thanks for adding Gentoo support. The code looks good and once the tests are passing, this should be able to be merged.

@optiz0r
Copy link
Copy Markdown
Contributor Author

optiz0r commented Oct 10, 2016

Thanks @ghoneycutt, I saw that it had failed but didn't yet find the time to investigate, thanks for pointing me directly at the fix. I've pushed this change (lsbdistcodename shows up as "n/a" for me on my system, so that's what I've used for the test). Looks like travis-ci timed out on all the ruby-2.3.1 builds, but working for all previous releases. I don't think that's a result of my code changes? The output seems to stop while running the Debian tests.

@ghoneycutt
Copy link
Copy Markdown
Member

Yeah, that's not related to this. Could you please squash your commits and I'll do a merge and release tomorrow.

@optiz0r
Copy link
Copy Markdown
Contributor Author

optiz0r commented Oct 11, 2016

@ghoneycutt squashed and hopefully ready to go!

@ghoneycutt ghoneycutt merged commit 31e68a9 into voxpupuli:master Oct 12, 2016
@ghoneycutt
Copy link
Copy Markdown
Member

Thank you!

Released in 1.18.0

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