Skip to content

Fix: mk-job (linux) support versions without perl again - #775

Closed
Bastian-Kuhn wants to merge 1 commit into
Checkmk:masterfrom
Bastian-Kuhn:fix_mkjob
Closed

Fix: mk-job (linux) support versions without perl again#775
Bastian-Kuhn wants to merge 1 commit into
Checkmk:masterfrom
Bastian-Kuhn:fix_mkjob

Conversation

@Bastian-Kuhn

Copy link
Copy Markdown
Contributor

Current Linux Distributions like RHEL 8 + 9 do not install Perl as default.
So with the Update Checkmk 2.3, mk-job fails there since an earlier workaround for aix/ solaris to get a timestamp using perl, was made a default now for as well Linux.

This fix tries to restore the function on GNU Linux again if perl is missing, but without reverting the use of perl as default.
Since GNU date should always have the %s parameter, just BSD or BusyBox can lack the support, that's maybe a reasonable approach.

@racicLuka

Copy link
Copy Markdown
Contributor

Hi Bastian,

Thank you for your contribution. A change inspired by your solution has been merged into the master branch. This should resolve the issue.

Best regards,
Luka

@racicLuka racicLuka closed this Jul 6, 2025
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 6, 2025
CheckmkCI pushed a commit that referenced this pull request Jul 6, 2025
Current Linux Distributions like RHEL 8 + 9 do not install Perl as default.
So with the Update Checkmk 2.3, mk-job fails there since an earlier workaround for aix/ solaris to get a timestamp using perl, was made a default now for as well Linux.

This fix tries to restore the function on GNU Linux again if perl is missing, but without reverting the use of perl as default.
Since GNU date should always have the %s parameter, just BSD or BusyBox can lack the support, that's maybe a reasonable approach.

CMK-20768
closes: #775

Change-Id: Icf9e182511ef7fa0898883710189b2fcf0960e66
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants