Chef does not honor custom gem options set in system-wide gemrc

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 0.10.8
  • Fix Version/s: None
  • Component/s: Chef Client
  • Environment:

    Ubuntu 10.10, Redhat 5.6, Redhat 5.7 verified

Description

If Gem.configuration["gem"] is set to something, that indicates that either the users .gemrc or /etc/gemrc has been configured to pass custom parameters to gem on every invocation.

Crowbar uses this functionality to ensure that gem-provided binaries are installed to a location that is always in $PATH without having to patch every gem install in the recipes we use.

I have a fix for this issue at https://github.com/VictorLowther/chef/tree/vlowther-really-honor-gemrc, and there is a pull request open.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Tommy Bishop added a comment -

Hi Victor,

I noticed that you linked to a fix, but the ticket is in an 'open' state. The folks over at Opscode typically only review tickets for merging which are in a 'resolved' state. If you resolve this ticket, then it should be reviewed at the next Opscode triage meeting (typically weekly).

You can find details about the the ticket/triage/merge process at http://wiki.opscode.com/display/chef/Release+Process

Show
Tommy Bishop added a comment - Hi Victor, I noticed that you linked to a fix, but the ticket is in an 'open' state. The folks over at Opscode typically only review tickets for merging which are in a 'resolved' state. If you resolve this ticket, then it should be reviewed at the next Opscode triage meeting (typically weekly). You can find details about the the ticket/triage/merge process at http://wiki.opscode.com/display/chef/Release+Process
Hide
Permalink
Victor Lowther added a comment -

Pull request to resolve this issue open.

Show
Victor Lowther added a comment - Pull request to resolve this issue open.
Hide
Permalink
Daniel DeLeo added a comment -

Can you add some tests for this new behavior? We need them to make sure that future improvements won't break anything.

Show
Daniel DeLeo added a comment - Can you add some tests for this new behavior? We need them to make sure that future improvements won't break anything.
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Removing fix version

Show
Bryan McLellan [Opscode] added a comment - Removing fix version

People

  • Assignee:
    Victor Lowther
    Reporter:
    Victor Lowther
Vote (0)
Watch (1)

Dates

  • Created:
    Updated: