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

Build man page for linux using soelim. #67

Closed
wants to merge 2 commits into from

Conversation

greenius
Copy link

@greenius greenius commented Jul 8, 2019

Also renamed source man page to avoid potential problems building on case insensitive file systems which would treat Lsof.8 and lsof.8 as the same file (causing problem with adding lsof.8 to .gitignore)

Steven Green added 2 commits July 8, 2019 12:59
@masatake masatake self-assigned this Jul 8, 2019
@masatake
Copy link
Contributor

masatake commented Jul 8, 2019

Thank you. I wonder how large the impact of this change is on the other dialect like freebsd.

@masatake
Copy link
Contributor

masatake commented Jul 14, 2019

Ideally, converting Lsof.8 to lsof.8 should be done in "make dist". In that case soelim should be run in "make dist".
Linux allows users to run lsof without root-privilege. So the man page should be lsof.1.

I have to revise the process for releasing.

@lrosenman
Copy link
Contributor

lrosenman commented Jun 5, 2020

currently. the FreeBSD port makefile does:

(cd ${WRKSRC} && /usr/bin/soelim Lsof.8 > ${WRKSRC}/lsof.8)
${INSTALL_MAN} ${WRKSRC}/lsof.8 ${STAGEDIR}${MAN8PREFIX}/man/man8/lsof.8

so if the lsof makefile(s) do that, we can adapt to get the manpage in the right place.

@jiegec
Copy link
Contributor

jiegec commented Jan 14, 2023

Already done in current version.

@jiegec jiegec closed this Jan 14, 2023
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.

None yet

4 participants