-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Should this replace ilianaw/crowbar-builder in crowbar's docs? #3
Comments
I would like to do so. Are you in favor of me creating a more generic Docker image based on this one? The current problem with this Docker image is that CircleCI can't use it as a build image for some reason that CircleCI gives an useless error and I cannot determine what changes need to be made to make it work. In my examples contributed to Crowbar, I simply use Would renaming this to |
A more generic image would be fine; it should have both the stable and nightly toolchains installed per iliana/rust-crowbar#36. I'm fine with you maintaining it, or merging your changes into the builder/ directory in ilianaw/rust-crowbar since that builder already doesn't work for most people. |
Great, I will create a "fork" which is extremely generic and incorporates both stable and nightly. The Docker image will be automated on changes to the base image and on the If a user would like to pin to a specific version, they can do so using |
Folks (including myself) have a lot of issues getting OpenSSL anything to build right for Lambda using the builder image linked to in crowbar's docs.
Ultimately the solution in my mind was to make this work with the lambci images, which you ultimately landed on.
Should we merge work and/or refer to your builder image in crowbar's docs?
(ref iliana/rust-crowbar#20 iliana/rust-crowbar#22)
The text was updated successfully, but these errors were encountered: