Skip to content
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

Minimal modifications to get bedrock and sage working #67144

Closed

Conversation

piotr-dajlido
Copy link

Description of the change

root.io/bedrock framework and roots.io/sage theme integration with bitnami/wordpress image

Benefits

Providing users with Bedrock framework for modern wordpress development using Bitnami goodies

Possible drawbacks

Depending on bedrock product instead of wordpress

Applicable issues

  • Wordpress instalation folder is still present ( need for cleanup but there is a strong coupling between bitnami scripts and old wordpress instalation )
  • Wordpress initialization script had to be disabled ( as they are not applicable any longer )
  • wp-cli is not configured to modify bedrock framework ( but i can be )

Additional information

!!! THIS IS NOT MENT TO BE USED IN PRODUCTION !!!
!!! THIS PR IS JUST TO DEMONSTRATE MODIFICATIONS REQUIRED TO RUN BEDROCK FRAMEWORK IN BITNAMI WORDPRESS CONTAINER !!!

@github-actions github-actions bot added wordpress triage Triage is needed labels May 20, 2024
@github-actions github-actions bot requested a review from carrodher May 20, 2024 12:54
@piotr-dajlido piotr-dajlido marked this pull request as draft May 20, 2024 12:54
Point wp-cli to bedrock wp-cli conf file

Signed-off-by: oaxfreeman <piotr-dajlido@users.noreply.github.com>
@piotr-dajlido
Copy link
Author

piotr-dajlido commented May 24, 2024

@carrodher JUST NOW - Pulling main from bitnami/containers and trying to build wordpress dockerfile without any modification results in:

Step 7/33 : RUN install_packages acl ca-certificates curl less libaudit1 libbrotli1 libbsd0 libbz2-1.0 libcap-ng0 libcom-err2 libcrypt1 libcurl4 libexpat1 libffi8 libfftw3-double3 libfontconfig1 libfreetype6 libgcc-s1 libgcrypt20 libglib2.0-0 libgmp10 libgnutls30 libgomp1 libgpg-error0 libgssapi-krb5-2 libhashkit2 libhogweed6 libicu72 libidn2-0 libjpeg62-turbo libk5crypto3 libkeyutils1 libkrb5-3 libkrb5support0 liblcms2-2 libldap-2.5-0 liblqr-1-0 libltdl7 liblzma5 libmagickcore-6.q16-6 libmagickwand-6.q16-6 libmd0 libmemcached11 libncurses6 libnettle8 libnghttp2-14 libonig5 libp11-kit0 libpam0g libpcre2-8-0 libpcre3 libpng16-16 libpq5 libpsl5 libreadline8 librtmp1 libsasl2-2 libsodium23 libsqlite3-0 libssh2-1 libssl3 libstdc++6 libsybdb5 libtasn1-6 libtidy5deb1 libtinfo6 libunistring2 libuuid1 libwebp7 libx11-6 libxau6 libxcb1 libxdmcp6 libxext6 libxml2 libxslt1.1 libzip4 libzstd1 openssl procps zlib1g
 ---> Running in 960f12c1cc9c
/bin/bash: line 1: /usr/sbin/install_packages: cannot execute: required file not found

Same happend on my friends PC who never pulled from this repo before :(

Is there an issue on bitnami/wordpress HEAD?

@carrodher
Copy link
Member

The install_packages script is placed at https://github.com/bitnami/containers/blob/main/bitnami/wordpress/6/debian-12/prebuildfs/usr/sbin/install_packages, you need to ensure that script is present to install packages

Copy link

This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution.

@github-actions github-actions bot added the stale 15 days without activity label Jun 12, 2024
Copy link

Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Pull Request. Do not hesitate to reopen it later if necessary.

@bitnami-bot bitnami-bot added stale 15 days without activity and removed stale 15 days without activity labels Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solved stale 15 days without activity triage Triage is needed wordpress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants