Skip to content

Fix issues with the PCRE library and speedup build#54

Merged
sjinks merged 1 commit intomasterfrom
update/test-runner
Nov 22, 2021
Merged

Fix issues with the PCRE library and speedup build#54
sjinks merged 1 commit intomasterfrom
update/test-runner

Conversation

@sjinks
Copy link
Copy Markdown
Member

@sjinks sjinks commented Nov 19, 2021

This PR fixes two issues at once (I know it's not the best practice, but these changes are tied together and working on them in
multiple branches will lead to merge conflicts):

  1. The "grave" issue: "Compilation failed: unrecognised compile-time option bit(s) at offset 0". This one seems to be caused by an outdated PCRE library; the library has to be updated from the PPA the PHP is installed from.
  2. Build failures resulting from transient network conditions (like "Unable to connect to archive.ubuntu.com")
  3. Speed up the build process by using eatmydata

This PR fixes two issues at once (I know it's not the best practice,
but these changes are tied together, and working on them in
multiple branches will lead to merge conflicts):

1. The "grave" issue: "Compilation failed: unrecognised compile-time
option bit(s) at offset 0".  This one seems to be caused by an
[outdated PCRE library](https://www.mail-archive.com/debian-bugs-dist@lists.debian.org/msg1830385.html);
the library has to be updated from the PPA the PHP is installed from.
2. Build failures resulting from transient network consitions (like
"Unable to connect to archive.ubuntu.com")
3. Speed up the build process by using [eatmydata](https://manpages.debian.org/testing/eatmydata/eatmydata.1.en.html)
@sjinks sjinks marked this pull request as ready for review November 19, 2021 22:45
@sjinks sjinks added bug Something isn't working docker Pull requests that update Docker code [Status] Needs Review labels Nov 19, 2021
Copy link
Copy Markdown
Contributor

@pschoffer pschoffer left a comment

Choose a reason for hiding this comment

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

LGTM

@sjinks sjinks merged commit c9eb673 into master Nov 22, 2021
@sjinks sjinks deleted the update/test-runner branch November 22, 2021 11:03
jesse-greathouse pushed a commit that referenced this pull request Jan 17, 2022
Fix issues with the PCRE library and speedup build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working docker Pull requests that update Docker code [Status] Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants