[PR #1252] [MERGED] Fix docker build #4379

Closed
opened 2026-03-15 01:41:29 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ArchiveBox/ArchiveBox/pull/1252
Author: @c01o
Created: 10/24/2023
Status: Merged
Merged: 10/25/2023
Merged by: @pirate

Base: devHead: patch-1


📝 Commits (1)

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 Dockerfile (+1 -1)

📄 Description

Lack of fi resulted in a syntax error.
Also, change ; to || to ensure it builds successfully on architectures other than linux/arm/v7.

[2023-10-24T10:46:49.941Z] ------
 > [dev_container_auto_added_stage_label 11/18] RUN mkdir -p "/home/archivebox/.config/chromium/Crash Reports/pending/"     && chown -R archivebox "/home/archivebox/.config"     ; if [[ "linux/amd64" == "linux/arm/v7" ]]; then $exit 0; else exit 1:
0.401 /bin/bash: -c: line 2: syntax error: unexpected end of file
------
WARNING: buildx: git was not found in the system. Current commit information was not captured by the build
[2023-10-24T10:46:49.942Z] Dockerfile-with-features:135
--------------------
 134 |         && ln -s "$CHROME_BINARY" /usr/bin/chromium-browser
 135 | >>> RUN mkdir -p "/home/${ARCHIVEBOX_USER}/.config/chromium/Crash Reports/pending/" \
 136 | >>>     && chown -R $ARCHIVEBOX_USER "/home/${ARCHIVEBOX_USER}/.config" \
 137 | >>>     ; if [[ "$TARGETPLATFORM" == "linux/arm/v7" ]]; then $exit 0; else exit 1
 138 |         # ignore failure for architectures where no playwright release is available yet
--------------------
ERROR: failed to solve: process "/bin/bash -c mkdir -p \"/home/${ARCHIVEBOX_USER}/.config/chromium/Crash Reports/pending/\"     && chown -R $ARCHIVEBOX_USER \"/home/${ARCHIVEBOX_USER}/.config\"     ; if [[ \"$TARGETPLATFORM\" == \"linux/arm/v7\" ]]; then $exit 0; else exit 1" did not complete successfully: exit code: 2

Summary

Related issues

Changes these areas

  • Bugfixes
  • Feature behavior
  • Command line interface
  • Configuration options
  • Internal architecture
  • Snapshot data layout on disk

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ArchiveBox/ArchiveBox/pull/1252 **Author:** [@c01o](https://github.com/c01o) **Created:** 10/24/2023 **Status:** ✅ Merged **Merged:** 10/25/2023 **Merged by:** [@pirate](https://github.com/pirate) **Base:** `dev` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`08cffb8`](https://github.com/ArchiveBox/ArchiveBox/commit/08cffb8742a752f61e3442c71be9bb08d8ad3244) Fix docker build ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `Dockerfile` (+1 -1) </details> ### 📄 Description Lack of `fi` resulted in a syntax error. Also, change `;` to `||` to ensure it builds successfully on architectures other than `linux/arm/v7`. ``` [2023-10-24T10:46:49.941Z] ------ > [dev_container_auto_added_stage_label 11/18] RUN mkdir -p "/home/archivebox/.config/chromium/Crash Reports/pending/" && chown -R archivebox "/home/archivebox/.config" ; if [[ "linux/amd64" == "linux/arm/v7" ]]; then $exit 0; else exit 1: 0.401 /bin/bash: -c: line 2: syntax error: unexpected end of file ------ WARNING: buildx: git was not found in the system. Current commit information was not captured by the build [2023-10-24T10:46:49.942Z] Dockerfile-with-features:135 -------------------- 134 | && ln -s "$CHROME_BINARY" /usr/bin/chromium-browser 135 | >>> RUN mkdir -p "/home/${ARCHIVEBOX_USER}/.config/chromium/Crash Reports/pending/" \ 136 | >>> && chown -R $ARCHIVEBOX_USER "/home/${ARCHIVEBOX_USER}/.config" \ 137 | >>> ; if [[ "$TARGETPLATFORM" == "linux/arm/v7" ]]; then $exit 0; else exit 1 138 | # ignore failure for architectures where no playwright release is available yet -------------------- ERROR: failed to solve: process "/bin/bash -c mkdir -p \"/home/${ARCHIVEBOX_USER}/.config/chromium/Crash Reports/pending/\" && chown -R $ARCHIVEBOX_USER \"/home/${ARCHIVEBOX_USER}/.config\" ; if [[ \"$TARGETPLATFORM\" == \"linux/arm/v7\" ]]; then $exit 0; else exit 1" did not complete successfully: exit code: 2 ``` <!-- IMPORTANT: Do not submit PRs with only formatting / PEP8 / line length changes. --> # Summary <!--e.g. This PR fixes ABC or adds the ability to do XYZ...--> # Related issues <!-- e.g. #123 or Roadmap goal # https://github.com/pirate/ArchiveBox/wiki/Roadmap --> # Changes these areas - [x] Bugfixes - [ ] Feature behavior - [ ] Command line interface - [ ] Configuration options - [ ] Internal architecture - [ ] Snapshot data layout on disk --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-15 01:41:29 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ArchiveBox#4379
No description provided.