Skip to content

chore(tvheadend): move from porting/ to apps/#2258

Draft
Crow-Control wants to merge 5 commits intomainfrom
chore/move-tvheadend-to-apps
Draft

chore(tvheadend): move from porting/ to apps/#2258
Crow-Control wants to merge 5 commits intomainfrom
chore/move-tvheadend-to-apps

Conversation

@Crow-Control
Copy link
Copy Markdown
Member

@Crow-Control Crow-Control commented May 6, 2026

Moves the tvheadend container from porting/ to apps/.

No file content changes — pure git mv. Validated with docker buildx bake --print from apps/tvheadend/.

Copilot AI review requested due to automatic review settings May 6, 2026 20:19
@Crow-Control Crow-Control requested a review from alfi0812 as a code owner May 6, 2026 20:19
@Crow-Control Crow-Control marked this pull request as draft May 6, 2026 20:19
Copilot AI reviewed May 6, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Crow-Control Crow-Control force-pushed the chore/move-tvheadend-to-apps branch from 5f46d3a to 5f6f210 Compare May 7, 2026 08:35
@Crow-Control
Copy link
Copy Markdown
Member Author

Crow-Control commented May 7, 2026

@claude[agent] Fix this

@Claude @Crow-Control
fix(ci): use app-specific docker-bake.hcl path in workflow
2c878ba 
The app-builder workflow was referencing ./docker-bake.hcl from the repository root,
but each app has its own docker-bake.hcl in apps/<app>/. This caused the build to fail
with "patches: not found" because the context wasn't set correctly.

Changed the workflow to reference ./apps/${{ inputs.app }}/docker-bake.hcl which correctly
resolves the docker-bake.hcl file for each app and sets the proper build context.

Agent-Logs-Url: https://github.com/trueforge-org/containerforge/sessions/7b3e701f-21aa-4e1a-bad2-1fbe3dc1ff40

Co-authored-by: Crow-Control <7613738+Crow-Control@users.noreply.github.com>
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 7, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Crow-Control
❌ Claude
You have signed the CLA already but the status is still pending? Let us recheck it.

@Claude @Crow-Control
trigger build
e568eb7 
Co-authored-by: Crow-Control <7613738+Crow-Control@users.noreply.github.com>
@Claude
Copy link
Copy Markdown
Contributor

Claude AI commented May 7, 2026

@claude[agent] Fix this

Fixed in 2c878ba. The CI was failing because the app-builder workflow was looking for ./docker-bake.hcl at the repository root instead of ./apps/tvheadend/docker-bake.hcl. This caused the build context to be incorrect and the patches/ directory couldn't be found.

The workflow now correctly references each app's docker-bake.hcl file. The workflow run needs manual approval since it modifies workflow files - once approved, the tvheadend build should succeed.

@Claude @Crow-Control
chore: remove .gitkeep
6267e92 
Co-authored-by: Crow-Control <7613738+Crow-Control@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@alfi0812 alfi0812 Awaiting requested review from alfi0812 alfi0812 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

area/apps

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Crow-Control @CLAassistant @Claude