search

rricharz / Tek4010

Free Tektronix 4010, 4013, 4014 and 4015 terminal emulator for Raspberry Pi, Linux, macOS (Macintosh) and Windows
GNU General Public License v3.0
244 stars 29 forks source link

Create manual page #42

Closed polluks closed 6 months ago

polluks commented 6 months ago

Requires help2man, you may adjust install.

polluks commented 6 months ago

I know what you mean, but a system dependent packaging tool, like apt install help2man, should not be part of a makefile. By the way it works pretty well for tek4010.

rricharz commented 6 months ago

help2man has more dependencies than tek4010. tek4010 was written to limit the number of dependencies.

In my opinion the proper solution would therefore be to

  1. Add something like „make man“ to make a manual page by the maintainer after the maintainer has installed for example help2man manually. Save the man page in the repo at a suitable place.

  2. Fix the „install“ script to install the provided man page in the man database without using commands which are not installed in the standard distribution.

If you agree, please provide and test this solution. I don‘t have much time to work on tek4010 myself at the moment.

polluks commented 6 months ago

done

rricharz commented 6 months ago

Unfortunately this does not work

make man fails with error in help2man

copy man page fails due to permission problem and location

polluks commented 6 months ago

Hmm, I suppose /usr/share/man/man1 is not better

Am Di., 28. Mai 2024 um 17:03 Uhr schrieb Rene Richarz < @.***>:

Unfortunately this does not work

make man fails with error in help2man

copy man page fails due to permission problem and location

— Reply to this email directly, view it on GitHub https://github.com/rricharz/Tek4010/pull/42#issuecomment-2135456977, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABD5J5KVHMC4PU2GZUEMKWTZESMD7AVCNFSM6AAAAABIHHLTPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZVGQ2TMOJXG4 . You are receiving this because you authored the thread.Message ID: @.***>

rricharz commented 6 months ago

This pull request has been closed. See issue #44 for a possible future enhancement.