This repository has been archived by the owner. It is now read-only.

Allow requirements to specify env options #16733

wants to merge 4 commits into
base: master


None yet
3 participants

jacknagel commented Dec 24, 2012

Requirements can now specify env :userpaths, env :std, or any future env options that we add, though right now only env :userpaths is respected.


This comment has been minimized.


adamv commented Dec 24, 2012

Add message to the DSL? Optional.


This comment has been minimized.


jacknagel commented Dec 25, 2012

Thought about it, but some messages interpolate instance variables so they would have to continue using the method form, so I didn't bother.

@@ -100,14 +100,16 @@ class MPIDependency < Requirement
attr_reader :lang_list
fatal true
env :userpaths

This comment has been minimized.


samueljohn Jan 9, 2013


... but when statisfied? is called, the build environment is not yet set up, so it is not satisfied :-(

@Homebrew Homebrew locked and limited conversation to collaborators Feb 16, 2016

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.