Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

Get rake compile working #62

Closed
wants to merge 1 commit into from

Conversation

jtgeibel
Copy link

@jtgeibel jtgeibel commented Jul 26, 2020

This should be run on Ubuntu 16.04 (Xenial), otherwise a too new libc is required during a deploy to older stacks.

This should get things building again so that #61 can be addressed.

This should be run on Ubuntu 18.04 (Xenial), otherwise a too new libc is
required during deploy.
@@ -1,5 +1,6 @@
#!/bin/bash

BP_DIR=$(cd $(dirname ${0:-}); cd ..; pwd)
export BUILDPACK_LOG_FILE=/dev/null
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was necessary because I was seeing errors like /tmp/buildpack20200726-1270-8zse1ys/lib/vendor/stdlib_v7.sh: line 138: /dev/fd/3: No such file or directory on deploy. It appears that BUILDPACK_LOG_FILE is being set to /dev/fd/3 somewhere, but that the nodejs buildpack is not being run with that additional fd. There is probably a better way to address this so that the fd is passed along to the nodejs buildpack.

conf.gem mgem: 'mruby-simple-random'
conf.gem github: 'hone/mruby-io', branch: 'popen_status'
conf.gem github: 'hone/mruby-process', branch: 'header'
conf.gem github: 'mattn/mruby-onig-regexp', checksum_hash: '4873e87fc1842e8e2a417f6071b63a7d91523ef4'
conf.enable_cxx_abi
#conf.enable_cxx_abi
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With this enabled, it looks like the build was passing C++ options to the C compiler, causing lots of C code to compile in C++ mode, resulting in lots of errors. mruby-docopt is the only dependency that appears to require C++, so I dropped that requirement and manually handle option parsing.

@edmorley
Copy link
Member

edmorley commented Aug 2, 2022

Thank you for the PR. However this buildpack is deprecated and it's also no longer possible to build new versions of the buildpack due to the issues mentioned in #61.

@edmorley edmorley closed this Aug 2, 2022
@jtgeibel jtgeibel deleted the get-things-building branch August 2, 2022 16:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants