Project

General

Profile

Actions

Bug #14214

closed

ar usage message needs updating for -q

Added by Robert Mustacchi 7 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Category:
cmd - userland programs
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

ar's implementation actually doesn't generally support -a, -b, or -i when used with -q. The manual and source code make this clear; however, the -abi options still show up in the usage message. We should remove that.

Actions #1

Updated by Electric Monk 7 months ago

  • Gerrit CR set to 1723
Actions #2

Updated by Robert Mustacchi 6 months ago

This was tested through the additions to the ar test suite that were done in #14213. In particular tests were done that validate that -qa, -qb, and -qi do not work. As part of the broader testing around #14105 again, Dominik Hassler kindly built all of the OmniOS base and extra packages with the new ar (how we originally discovered issues with #14105) and at least here we didn't notice any additional issues.

Actions #3

Updated by Electric Monk 6 months ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

git commit fb25420ba8dbfa4c292d42c87555eee97a474854

commit  fb25420ba8dbfa4c292d42c87555eee97a474854
Author: Robert Mustacchi <rm@fingolfin.org>
Date:   2021-11-11T19:19:33.000Z

    14105 ar -s could work on its own
    14213 Want basic ar test suite
    14212 ar cra and crb don't work
    14214 ar usage message needs updating for -q
    Reviewed by: Rich Lowe <richlowe@richlowe.net>
    Reviewed by: Patrick Mooney <pmooney@pfmooney.com>
    Reviewed by: Andy Fiddaman <andy@omnios.org>
    Approved by: Dan McDonald <danmcd@joyent.com>

Actions

Also available in: Atom PDF