linuxboot / linuxboot/heads

Add info output function (for unified text prompts under Heads)

Open
#1,491 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Makefile
Stars
1.6k
Forks
211
Avg merge
4d 21h
Merged PRs (30d)
6

Description

We have die and warn which #1482 unifies screen output.

Everywhere in code, we seem to use echo "+++ Blah blah" as info. There is also +++++++" and variations on that. We could unify those and replicate warn and die, but without sleep calls

I suggest that we unify the codebase by adding non-sleeping `info` function which would prepend all text output with ` +++ blah` (with a space) to make it clear what output comes from Heads and what comes from tools Heads uses.

Example
![2023-09-01-173124](https://github.com/osresearch/heads/assets/827570/7b3a433e-4878-498a-83ae-e92110e1692a)

@JonathonHall-Purism

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the existing die and warn functions and the echo-based output calls described in the issue. Review how those functions format and pause output, then determine the complete set of text-output sites that should use the new non-sleeping info behavior; done means Heads output has one consistent info prefix.

Written by the indexing model from the issue text.

Assessment

Domain
cli
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.