Re: [Puppet-dev] [PATCH] (#4135) Update pluginsync to only load ruby files.

2011-09-15 Thread Nan Liu
On Thu, Sep 15, 2011 at 10:40 AM, Stefan Schulte wrote: > I'd like to review and commit your patch but it looks like > https://github.com/nanliu/puppet/tree/ticket/2.7.x/4135 is not based on > 2.7.x but on master. Can you please rebase? Rebase against 2.7.x now, please review. Thanks, Nan --

Re: [Puppet-dev] [PATCH] (#4135) Update pluginsync to only load ruby files.

2011-09-15 Thread Stefan Schulte
On Sat, Jul 16, 2011 at 09:36:46AM -0700, Nan Liu wrote: > Pluginsync allows replication of all files, but we should only load > ruby files available in the pluginsync directory. For example, > currently pluginsync will attempt and fail to load puppet-module > generated readme files. This change re

Re: [Puppet-dev] [PATCH] (#4135) Update pluginsync to only load ruby files.

2011-07-16 Thread Daniel Pittman
On Sat, Jul 16, 2011 at 09:36, Nan Liu wrote: > Pluginsync allows replication of all files, but we should only load > ruby files available in the pluginsync directory. For example, > currently pluginsync will attempt and fail to load puppet-module > generated readme files. This change restrict pup

Re: [Puppet-dev] [PATCH] (#4135) Update pluginsync to only load ruby files.

2011-07-16 Thread James Turnbull
Nan Liu wrote: Pluginsync allows replication of all files, but we should only load ruby files available in the pluginsync directory. For example, currently pluginsync will attempt and fail to load puppet-module generated readme files. This change restrict puppet agent to only load ruby files, but

[Puppet-dev] [PATCH] (#4135) Update pluginsync to only load ruby files.

2011-07-16 Thread Nan Liu
Pluginsync allows replication of all files, but we should only load ruby files available in the pluginsync directory. For example, currently pluginsync will attempt and fail to load puppet-module generated readme files. This change restrict puppet agent to only load ruby files, but preserve the abi