Skip to content

README.md: Added instructions for build-locally.sh#3699

Merged
olehermanse merged 1 commit into
cfengine:masterfrom
olehermanse:build-locally
Jun 23, 2026
Merged

README.md: Added instructions for build-locally.sh#3699
olehermanse merged 1 commit into
cfengine:masterfrom
olehermanse:build-locally

Conversation

@olehermanse

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse olehermanse marked this pull request as ready for review June 22, 2026 18:06
@olehermanse olehermanse requested a review from nickanderson June 22, 2026 18:06
@nickanderson

Copy link
Copy Markdown
Member

It generated as expected for me and I was able to click around various links.

@nickanderson nickanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

But, tmp was then populated with work/masterfiles/* owned by root.

ls -al ./work/masterfiles/templates/json_serial.mustache
-rw-r--r-- 1 root root 11 Jul 16  2011 ./work/masterfiles/templates/json_serial.mustache

@olehermanse

Copy link
Copy Markdown
Member Author

@nickanderson not for me.

$ ls -al ./tmp/work/masterfiles/templates/json_serial.mustache
-rw-r--r-- 1 olehermanse olehermanse 11 Jul 16  2011 ./tmp/work/masterfiles/templates/json_serial.mustache

@nickanderson

Copy link
Copy Markdown
Member

@nickanderson not for me.

$ ls -al ./tmp/work/masterfiles/templates/json_serial.mustache
-rw-r--r-- 1 olehermanse olehermanse 11 Jul 16  2011 ./tmp/work/masterfiles/templates/json_serial.mustache

For @larsewi it did.

$ ls -al ./tmp/work/masterfiles/templates/json_serial.mustache
-rw-r--r-- 1 root root 11 Jul 16  2011 ./tmp/work/masterfiles/templates/json_serial.mustache

@olehermanse olehermanse merged commit 0c3f35e into cfengine:master Jun 23, 2026
2 checks passed
@olehermanse olehermanse deleted the build-locally branch June 23, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants