Age | Commit message (Collapse) | Author |
|
|
|
|
|
breaks reflow.
|
|
explanation of modes.
|
|
argument, instead of a u_char*. Saves callers needing to cast.
|
|
Diff from Alexander Koeppe format_c -AT- online -DOT- de, thanks.
ok deraadt@
|
|
First step to adapt openssl(1) dgst command to new option handling.
There is no functional changes by this diff, and just moving variables
into dgst_config struct.
ok bcook@
|
|
Delete the "argc" argument from fs_search() which is now always 1,
and move error reporting to the main() program where it is more
logically placed and easier to see.
|
|
so delete the second copy. No functional change.
|
|
show the results in the order of the command line arguments.
Implemented by separating the code for man(1) and apropos(1)
in the main() program.
Surprisingly, the number of lines of code remains unchanged.
Issue reported by deraadt@, additional input from millert@.
|
|
Unify code to process one single input file
and move it into a dedicated new function.
|
|
crank the maximum channels number to 64.
|
|
Also, move setting of tag_files.tagname into tag_init().
No functional change.
|
|
For clarity, stop storing the same information (in this case, -O
settings) in two structs. Give the local struct in main.c a more
descriptive name (output state).
|
|
Mixing parser and formatter state in the same struct was a bad idea,
so pull the parser state and configuration out of it.
This makes sure output options are not passed into parser functions
and parser options are not passed into output functions.
While here, add comments to the important local variables in main().
|
|
McBurnett in GitHub issue 1857.
|
|
Move process group management out of main() into its own function
because it has its own, self-contained logic and its own local variables.
|
|
- Add a space before 'export_end:'
- Remove space after '*'
- Wrap lines by 80 columns
|
|
OK guenther@
|
|
ok bcook@ tb@
|
|
|
|
Bug reported by Marcel Logen <333200007110-0201 at ybtra dot de>.
While here, remove the misleading initialization of msgid
which is not a state variable.
OK millert@ deraadt@
|
|
input Janne Johansson, schwarze@; OK deraadt@ millert@
|
|
|
|
from Steven Roberts
|
|
they finish, GitHub issue 1852.
|
|
does in FreeBSD and in NetBSD, and fully document that behaviour.
Input, feedback, and OK jca@.
|
|
|
|
Adapt openssl(1) pkcs12 command to new option handling.
Added pkcs12_options struct, and replaced for-if-strcmp handling with
options_parse().
ok and comments jsing@
|
|
(and options ControlMaster+RemoteForward in ssh_config(5)) then the codepath
taken will call mux_client_request_session -> mm_send_fd -> sendmsg(2). Since
sendmsg(2) is not allowed in that codepath then pledge(2) kills the process.
The solution is to add "sendfd" to pledge(2), which is not too bad considering
a little bit later we reduce pledge(2) to only "stdio proc tty" in that
codepath.
Problem reported and diff provided by Timothy Brown <tbrown at freeshell.org>
OK deraadt@
|
|
Patch from hnj2 via github pr#141.
|
|
|
|
First step to adapt openssl(1) pkcs12 command to new option handling.
There is no functional changes by this diff, and just moving variables into
pkcs12_config struct.
I still keep long lines more than 80 for this review to minimize diffs.
ok jsing@ tb@
|
|
command or when dying from SIGTERM. This way, creating the recovery
file works again without re-adding "proc exec" to the pledge(2).
As reported by Jesper Wallin <jesper at ifconfig dot se>, this got
broken by common/main.c rev. 1.29 (Nov 19, 2015).
The general direction of the fix was suggested by brynet@.
OK brynet@ and no opposition when shown on tech@
|
|
though it is obvious that this can never become as good as for
mdoc(7) pages. As a first step, tag alphabetic arguments of .IP
macros, which are often used for lists of options and keywords.
Try "man -O tag=g as" to get the point.
Thanks to Leah Neukirchen for recently reminding me that exploring
how much can be done in this respect may be worthwhile: it is likely
to slightly improve usability while adding only small amounts of
relatively straightforward code.
|
|
Agreement from schwarze@ (who proposed a thorough but longer diff) and millert@
ok tb@ kn@ cheloha@
|
|
Mention -p (user password) while here.
|
|
problem reported by George Brown <321 dot george at gmail dot com> on tech@.
|
|
|
|
|
|
now, the diff is about -30 +150 lines (about 0.4%)
|
|
soon the condition can be detected and do not pass it to less(1).
This may happen for man(7) pages, for preformatted pages, and for
very simple pages like true(1). The main benefit is that :t inside
less(1) yields the clearer diagnostic message "No tags file" rather
than the mildly confusing "No such tag in tags file": the latter
might encourage further, futile attempts to jump to other tags.
Improvement suggested by Leah Neukirchen <leah at vuxu dot org>
from The Void.
|
|
It makes little sense from a security standpoint to unlock the terminal
and expose the user's session after fifteen minutes by default.
Default behavior is now to reserve the terminal forever. Add instructions
to the manpage to help the user employ the -t timeout option more safely.
Manpage greatly improved by jmc@; bug(s) caught by millert@; with input
from claudio@.
ok millert@
|
|
Just to get things working. Best get rid of that nonsense sooner
rather than later.
|
|
ok millert@ kurt@
code suggested by otto@
|
|
|
|
1805.
|
|
(i.e. "ssh-keygen -vF host") to print the matching host's random-
art signature too. bz#3003 "amusing, pretty" deraadt@
|
|
calls to a clean-up variable function.
|
|
the documented -m. As a rule, we only document long options when
users can't avoid them because they lack a short version.
As suggested by tedu@, as an exception, leave --context documented
because -C is awkward in so far as it takes an optional option
argument, which is fragile and error-prone and hence generally
discouraged, including by POSIX.
Two years ago, kettenis@, deraadt@, and tedu@ all agreed that this
is what should be done, and jmc@ was happy to accept the direction,
but somehow everybody forgot to commit.
|