Skip to content

ENT-12600: produce-debug-symbols: documented & refactored script - #1961

Merged
larsewi merged 2 commits into
cfengine:masterfrom
larsewi:produce-debug-symbols
Oct 3, 2025
Merged

ENT-12600: produce-debug-symbols: documented & refactored script#1961
larsewi merged 2 commits into
cfengine:masterfrom
larsewi:produce-debug-symbols

Conversation

@larsewi

@larsewi larsewi commented Oct 1, 2025

Copy link
Copy Markdown
Contributor
  • produce-debug-symbols: document script
  • produce-debug-symbols: replace echo's with logging functions

larsewi added 2 commits October 1, 2025 21:22
@larsewi
produce-debug-symbols: document script
4d0ad05 
Ticket: ENT-12600
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@larsewi
produce-debug-symbols: replace echo's with logging functions
bbe8daf 
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@larsewi
larsewi force-pushed the produce-debug-symbols branch from 0b4c328 to bbe8daf Compare October 1, 2025 19:26
@larsewi

larsewi commented Oct 1, 2025

Copy link
Copy Markdown
Contributor Author

@cf-bottom Jenkins please :)

@cf-bottom

cf-bottom commented Oct 1, 2025

Copy link
Copy Markdown

Sure, I triggered a build:

Build Status

Jenkins: https://ci.cfengine.com/job/pr-pipeline/12783/

Packages: http://buildcache.cfengine.com/packages/testing-pr/jenkins-pr-pipeline-12783/

@larsewi
larsewi requested a review from craigcomstock October 1, 2025 19:36
@larsewi
larsewi marked this pull request as ready for review October 1, 2025 19:36
craigcomstock
craigcomstock approved these changes Oct 2, 2025
View reviewed changes

@craigcomstock craigcomstock left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah. Let's make sure this stuff actually works properly: https://northerntech.atlassian.net/browse/ENT-13250

@larsewi

larsewi commented Oct 2, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Yeah. Let's make sure this stuff actually works properly: https://northerntech.atlassian.net/browse/ENT-13250

@craigcomstock as a follow-up or now? I don't think this script has been run in many years. I think it's fairly safe to merge this.

@craigcomstock

craigcomstock commented Oct 2, 2025

Copy link
Copy Markdown
Contributor

Yeah. Let's make sure this stuff actually works properly: https://northerntech.atlassian.net/browse/ENT-13250

@craigcomstock as a follow-up or now? I don't think this script has been run in many years. I think it's fairly safe to merge this.

Definitely follow-up. That's why I made the ticket. 👍

@larsewi
larsewi merged commit b5d124e into cfengine:master Oct 3, 2025
29 of 32 checks passed
@larsewi
larsewi deleted the produce-debug-symbols branch April 27, 2026 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@craigcomstock craigcomstock craigcomstock approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@larsewi @cf-bottom @craigcomstock