HomePhorge

Remove unnecessary fallbacks in change_{user,group}
ade5965e4e4dUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Remove unnecessary fallbacks in change_{user,group}

These methods were attempting to call change_privilege, and rescuing
NotImplementedError, to use our own implementation of similar behavior.
As it turns out, the only system on which change_privilege is not
implemented is Windows, which these methods already don't support. So
there's no reason to have a fallback that would only ever fail or not be
reached.

Details

Provenance
Nick Lewis <nick@puppetlabs.com>Authored on
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPU0a09a64457ef: Document uid/gid-related methods in Puppet::Util
Branches
Unknown
Tags
Unknown

Event Timeline

Nick Lewis <nick@puppetlabs.com> committed rPUade5965e4e4d: Remove unnecessary fallbacks in change_{user,group} (authored by Nick Lewis <nick@puppetlabs.com>).Feb 14 2012, 12:54 AM