Skip to content

Conversation

jimklimov
Copy link
Member

Addresses part of issue #3099

@jimklimov jimklimov added this to the 2.8.5 milestone Oct 4, 2025
@jimklimov jimklimov added packaging Solaris/illumos Solaris and illumos systems (OpenIndiana, OmniOS, SmartOS, TribbliX...) portability We want NUT to build and run everywhere possible labels Oct 4, 2025
@jimklimov jimklimov changed the title scripts: Avoid tail -n +/-NUM syntax as less portable than tail +/- NUM scripts: Avoid tail -n +/-NUM syntax as less portable than tail +/-NUM Oct 4, 2025
…is.sh.sample, scripts/upsdrvsvcctl/nut-driver-enumerator.sh.in: avoid "tail -n +/-NUM" syntax as unportable [networkupstools#3099]

Signed-off-by: Jim Klimov <[email protected]>
… upsdrvctl activity; add tweak-ability for debugging NUT programs in the install script context (possibly with custom package builds) [networkupstools#3099]

Signed-off-by: Jim Klimov <[email protected]>
…nit script usage on Solaris older than 10 [networkupstools#3099]

Signed-off-by: Jim Klimov <[email protected]>
@jimklimov
Copy link
Member Author

Odd, tests did fail (shellcheck mismatch for NDE) on the older Linux distros. So something did behave differently :\

GH status says

  • slowbuild-run/MATRIX_TAG="linux-ubuntu-trusty-make-shellcheck"
  • slowbuild-run/MATRIX_TAG="linux-centos-7-make-shellcheck"

But I only see one in the build log though (linux-centos-7-make-shellcheck):

will investigate sometime...

@jimklimov
Copy link
Member Author

jimklimov commented Oct 6, 2025

Indeed, at least CentOS 7's tail can do tail -n +3 but cannot tail +3 because no such file "+3" :
And classic Solaris tail has no -n :\

@jimklimov jimklimov marked this pull request as draft October 6, 2025 08:23
@jimklimov jimklimov marked this pull request as ready for review October 6, 2025 08:31
@jimklimov
Copy link
Member Author

jimklimov commented Oct 6, 2025

CI in run 5: errors on openbsd worker not related to scripting changes (got some FS troubles).
Re-running as build 6 (same commit so old notifications that are not getting overwritten - remain).

@jimklimov jimklimov added the ready / gonna merge The PR is in final cycles leading to merge unless someone logs an objection before we hit the button label Oct 8, 2025
@jimklimov jimklimov merged commit 239aa86 into networkupstools:master Oct 8, 2025
30 of 34 checks passed
@jimklimov jimklimov deleted the issue-3099 branch October 8, 2025 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packaging portability We want NUT to build and run everywhere possible ready / gonna merge The PR is in final cycles leading to merge unless someone logs an objection before we hit the button Solaris/illumos Solaris and illumos systems (OpenIndiana, OmniOS, SmartOS, TribbliX...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant