Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change manpage command to man #6548

Merged
merged 2 commits into from
Feb 21, 2020
Merged

Change manpage command to man #6548

merged 2 commits into from
Feb 21, 2020

Conversation

phadej
Copy link
Collaborator

@phadej phadej commented Feb 19, 2020

And make it pipe output to man -l -.

@phadej phadej force-pushed the man-command branch 3 times, most recently from af20c03 to 2969b1d Compare February 19, 2020 21:00
And make it pipe output to `man -l -`.
@phadej phadej force-pushed the man-command branch 3 times, most recently from e8e6934 to bea78db Compare February 21, 2020 08:44
@phadej phadej merged commit e75c9f5 into haskell:master Feb 21, 2020
@phadej phadej deleted the man-command branch February 21, 2020 12:57
@phadej phadej mentioned this pull request Jul 10, 2020
@phadej phadej added this to the 3.4.0.0-rc1 milestone Jul 13, 2020
@andreasabel
Copy link
Member

Seems like renaming was not complete:

$  cabal man foo
Error: cabal: 'manpage' doesn't take any extra arguments: foo

@Mikolaj
Copy link
Member

Mikolaj commented Oct 7, 2021

Well spotted: git grep suggests this is the last vestige.

@andreasabel
Copy link
Member

andreasabel commented Oct 7, 2021

Also, on macOS it does not work:

$ cabal man 
man: invalid option -- l
man, version 1.6c

usage: man [-adfhktwW] [section] [-M path] [-P pager] [-S list]
	[-m system] [-p string] name ...
...
fd:13: commitBuffer: resource vanished (Broken pipe)

I'll open an issue.

@Mikolaj
Copy link
Member

Mikolaj commented Oct 7, 2021

Yes, please. On Ubuntu it works for me (with cabal 3.4).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants