Support REE 1.8.7-2010.02 #45

Merged
merged 2 commits into from Sep 22, 2011

Conversation

Projects
None yet
2 participants
@cgunther
Contributor

cgunther commented Sep 22, 2011

Using this on a production site and wanted to add support for it. Tested on OSX Lion.

Copied the missing folder fix from #39

share/ruby-build/ree-1.8.7-2010.02
+
+ { ./installer --auto "$PREFIX_PATH" --dont-install-useful-gems $options
+ } >&4 2>&1
+}

This comment has been minimized.

Show comment Hide comment
@josh

josh Sep 22, 2011

Contributor

Think we should move this function into the main script so its not duplicated.

@josh

josh Sep 22, 2011

Contributor

Think we should move this function into the main script so its not duplicated.

@cgunther

This comment has been minimized.

Show comment Hide comment
@cgunther

cgunther Sep 22, 2011

Contributor

I've moved the function, but the only part I'm unsure of how to tackle at this point is skipping the workaround for a missing folder that occurs in versions prior to REE-1.8.7-2011.01 (http://code.google.com/p/rubyenterpriseedition/issues/detail?id=42).

Contributor

cgunther commented Sep 22, 2011

I've moved the function, but the only part I'm unsure of how to tackle at this point is skipping the workaround for a missing folder that occurs in versions prior to REE-1.8.7-2011.01 (http://code.google.com/p/rubyenterpriseedition/issues/detail?id=42).

@josh

This comment has been minimized.

Show comment Hide comment
@josh

josh Sep 22, 2011

Contributor

mkdir -p is safe.

Contributor

josh commented Sep 22, 2011

mkdir -p is safe.

josh added a commit that referenced this pull request Sep 22, 2011

@josh josh merged commit 733926c into rbenv:master Sep 22, 2011

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