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

Use librarian-puppet for module dependencies. #11

Closed

Conversation

@christianchristensen
Copy link

christianchristensen commented Sep 3, 2012

No description provided.

@christianchristensen
Copy link
Author

christianchristensen commented Sep 3, 2012

Addresses issue #6

Note: I grepped and "guessed" what I needed to change in debian package to depend on puppet (and librarian-puppet) from rubygems instead of apt-get.

For testing I edited the post commit hook, the sudoers, and used this cloudinit

#cloud-config
apt_sources:
 - source: "ppa:brightbox/puppet"
packages:
 - puppet-git-receiver
 - rubygems
runcmd:
- cp -ar /home/ubuntu/.ssh /var/lib/puppet-git-receiver/
- chown -R puppet-git.puppet-git /var/lib/puppet-git-receiver/.ssh
- gem install librarian-puppet --no-ri --no-rdoc
@christianchristensen
Copy link
Author

christianchristensen commented Sep 3, 2012

This greatly simplifies the git repo representing your server:

tree
.
├── manifests
│   └── site.pp
├── Puppetfile
└── README.md

1 directory, 4 files
@johnl
Copy link
Contributor

johnl commented Mar 24, 2013

erk, sorry for the delay responding to this.

I'm not happy changing the debian package to use rubygems for puppet though (or even use rubygems at all during installation). Instead, could you change it to just check if the librarian-puppet executable is available in the path?

That way, the user can just get it installed however they wish, but otherwise it's unobtrusive.

@johnl
Copy link
Contributor

johnl commented Aug 24, 2013

@DexterTheDragon implemented this without the dependency on rubygems - merged that pull request (#13)

@johnl johnl closed this Aug 24, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.