Skip to content

Page modification request: figlet #16168

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

Closed
Anhsirkz opened this issue Apr 13, 2025 · 11 comments · Fixed by #16296
Closed

Page modification request: figlet #16168

Anhsirkz opened this issue Apr 13, 2025 · 11 comments · Fixed by #16296
Labels
help wanted You can help make tldr-pages better! page edit Changes to an existing page(s).

Comments

@Anhsirkz
Copy link

Command description

figlet input_text -f font_filename is wrong.
It should rather state:
figlet -f font_filename input_text

Command details

No response

Documentation

No response

Platform

Common

VCS repository link (e.g. GitHub, GitLab)

No response

Additional information

No response

@Anhsirkz Anhsirkz added help wanted You can help make tldr-pages better! page edit Changes to an existing page(s). labels Apr 13, 2025
@spageektti
Copy link
Member

Feel free to open a PR!

@Managor
Copy link
Collaborator

Managor commented Apr 14, 2025

Depending on the program, options can be in any order. Here for example both are valid commands.

@acuteenvy
Copy link
Member

@Anhsirkz how did you install figlet? If there's a different implementation we could try to make the examples work on both.

@Anhsirkz
Copy link
Author

Anhsirkz commented Apr 28, 2025

@acuteenvy
nix-shell -p figlet
Using the latest 24.11 channel.

@acuteenvy
Copy link
Member

Both commands work for me when using figlet from nixpkgs. What's the error you're getting?

@Managor
Copy link
Collaborator

Managor commented Apr 28, 2025

The argument I would make is consistency. While both are valid commands, we tend to put options before positional arguments in examples.

@Anhsirkz
Copy link
Author

@acuteenvy I don't know how this is possible:
figlet thin -f thin outputs thin -fthin stylized in the default font.
figlet -f thin thin outputs thin stylized with the thin font.

@Managor Wouldn't your statement be a contradiction to my findings?

Image

@Managor
Copy link
Collaborator

Managor commented Apr 28, 2025

I feel like you have a different implementation

Image

What does figlet -v return for you?

FIGlet Copyright (C) 1991-2012 Glenn Chappell, Ian Chai, John Cowan,
Christiaan Keet and Claudio Matsuoka
Internet: <info@figlet.org> Version: 2.2.5, date: 31 May 2012

FIGlet, along with the various FIGlet fonts and documentation, may be
freely copied and distributed.

If you use FIGlet, please send an e-mail message to <info@figlet.org>.

The latest version of FIGlet is available from the web site,
        http://www.figlet.org/

Usage: figlet [ -cklnoprstvxDELNRSWX ] [ -d fontdirectory ]
              [ -f fontfile ] [ -m smushmode ] [ -w outputwidth ]
              [ -C controlfile ] [ -I infocode ] [ message ]

@Anhsirkz
Copy link
Author

figlet -v returns the same output according to diff

@Managor
Copy link
Collaborator

Managor commented Apr 28, 2025

Very curious. Shame we were not able to find the root cause. Since the common command that works for both of us is figlet -f font text, I see no reason to not change the example. Would you be willing to fire up a pull request or shall I do it?

@Anhsirkz
Copy link
Author

Please go ahead and do the PR. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted You can help make tldr-pages better! page edit Changes to an existing page(s).
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants