-
-
Notifications
You must be signed in to change notification settings - Fork 925
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Truffle] RbConfig and mkmf updates to support making openssl
- Loading branch information
Brandon Fish
committed
Nov 8, 2016
1 parent
4217ec8
commit e814214
Showing
6 changed files
with
2,746 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
#include "../ruby.h" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
#include "../ruby.h" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
#include "../ruby.h" |
Oops, something went wrong.
e814214
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chrisseaton Could you give this a try? I've been running with:
Copy the
openssl.su
tojruby/lib/ruby/truffle/openssl
.Then give
require "openssl"
a try. I see an error upon requiring.I've added our own version of the
mkmf
file with# MODIFIED
tags where changes are made because the originalmkmf
appears to hard code assumptions about the ruby project directory structure.e814214
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bjfish When you edit an existing file like that, first commit the original, and then your modifications. That would allow to easily port the modifications to a newer version and see what got modified.
e814214
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bjfish with running
su-opt
, this now seems to do what we want.