Skip to content

apply: forbid multiple -# argument - #1045

Merged
briandfoy merged 1 commit into
briandfoy:masterfrom
mknos:apply-many-num-args
Aug 29, 2026
Merged

apply: forbid multiple -# argument#1045
briandfoy merged 1 commit into
briandfoy:masterfrom
mknos:apply-many-num-args

Conversation

@mknos

@mknos mknos commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Following the OpenBSD version, raise an error for bad usage of conflicting (or multiple of the same) -# option. Tested on Linux and Windows.

  • Test case (invalid): perl apply -2 -3 echo 1 2 3 4 5 6
  • Previously the final -# option would win (-3); now print an error and exit

@mknos
apply: forbid multiple -# argument
26a9b07 
Following the OpenBSD version, raise an error for bad usage of conflicting (or multiple of the same)  -# option. Tested on Linux and Windows.
@mknos
mknos deployed to housekeeping August 20, 2026 02:26 — with GitHub Actions Active
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@mknos
mknos temporarily deployed to automated_testing August 20, 2026 02:26 — with GitHub Actions Inactive
@github-actions github-actions Bot added Type: enhancement improve a feature that already exists Priority: low get to this whenever Program: apply The apply program Status: needs verification issue needs to be verified Type: bug an existing feature does not work labels Aug 20, 2026
@briandfoy briandfoy removed the Type: enhancement improve a feature that already exists label Aug 29, 2026
@briandfoy briandfoy removed the Status: needs verification issue needs to be verified label Aug 29, 2026
briandfoy
briandfoy approved these changes Aug 29, 2026
View reviewed changes

@briandfoy briandfoy left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

A bit klunky but it gets the job done without rewriting everything

@briandfoy
briandfoy merged commit debc50b into briandfoy:master Aug 29, 2026
23 of 24 checks passed
@briandfoy
briandfoy deployed to housekeeping August 29, 2026 04:45 — with GitHub Actions Active
@briandfoy briandfoy added Status: accepted The fix is accepted and removed Priority: low get to this whenever labels Aug 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@briandfoy briandfoy briandfoy approved these changes

Assignees

@briandfoy briandfoy

Labels

Program: apply The apply program Status: accepted The fix is accepted Type: bug an existing feature does not work

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mknos @briandfoy