HPR/hpr_generator
8
1
Fork 5
Code Issues 38 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

[i335] docker image for site generator #336

Open
rho_n wants to merge 6 commits from i335_docker-image-for-site-generator into main
pull from: i335_docker-image-for-site-generator
merge into: HPR:main
Conversation 2 Commits 6 Files Changed 4 +70 -4
rho_n commented 2026-05-17 15:46:50 +00:00
Owner
Copy Link

Add a Docker file and instructions for building and running a Docker site-generator app locally

Add a Docker file and instructions for building and running a Docker site-generator app locally
rho_n added 3 commits 2026-05-17 15:46:50 +00:00
Add a Dockerfile which builds a site-generator app 7cea36879c 
This builds the container with the needed Perl modules, templates,
and a default configuration file.
Set env to perl 5bdda3c1ef 
Instead of calling perl directly from /usr/bin, set environment so that the current
default perl program is called.
Add instructions for building and running Docker images 47b2c588b8
rho_n requested review from archer72 2026-05-17 15:46:50 +00:00
rho_n requested review from ken_fallon 2026-05-17 15:46:50 +00:00
rho_n requested review from sgoti 2026-05-17 15:46:50 +00:00
rho_n added 1 commit 2026-05-17 16:01:55 +00:00
Fix cpanm call in run layer 3cc4a6b5d3
rho_n commented 2026-05-17 18:35:50 +00:00
Author
Owner
Copy Link

When testing this PR, make sure to switch to the correct branch first

git fetch
git checkout origin i335_docker-image-for-site-generator

You will know you are in the correct branch when you see the Dockerfile

When testing this PR, make sure to switch to the correct branch first ``` git fetch git checkout origin i335_docker-image-for-site-generator ``` You will know you are in the correct branch when you see the Dockerfile
rho_n added 1 commit 2026-05-17 23:32:15 +00:00
Update Docker build instructions fd577942e6
norrist approved these changes 2026-05-18 21:18:37 +00:00
norrist left a comment
Member
Copy Link

Approved with Comments

Testing worked using the requested change.

Approved with Comments Testing worked using the requested change.
GETTING_STARTED.md Outdated
@@ -68,0 +80,4 @@
```-v <path to db directory>/hpr.db:/usr/src/app/hpr.db```
Build the image by running the following command from the hpr_generator directory:
```docker run -t hpr/site-generator .```
norrist commented 2026-05-18 18:30:01 +00:00
Member
Copy Link

I think this should be docker build instead of docker run

I think this should be `docker build` instead of `docker run`
rho_n commented 2026-05-19 18:01:45 +00:00
Author
Owner
Copy Link

Good catch norrist. Thanks!

Good catch norrist. Thanks!
rho_n added 1 commit 2026-05-19 18:00:54 +00:00
Fix Docker build instruction example e1be81a4ad
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin i335_docker-image-for-site-generator:i335_docker-image-for-site-generator
git checkout i335_docker-image-for-site-generator
Sign in to join this conversation.
Reviewers
No Reviewers
ken_fallon
sgoti
archer72
norrist
Labels
Clear labels
Bug Found
For issues related to the code or the content of the created HTML files
 Feature Request
General Discussion
For discussing HPR site generation in general
 Help Request
For issues running or installing the site-generator
Feature Request
Used when requesting a new Feature.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
archer72 davmo jknapp ken_fallon norrist paulj rho_n sgoti
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: HPR/hpr_generator#336
Delete Branch "i335_docker-image-for-site-generator"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?