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

prompt ascii changed #1013

Closed
wants to merge 1 commit into from
Closed

prompt ascii changed #1013

wants to merge 1 commit into from

Conversation

PythonHacker24
Copy link

Description

This patch is the solution to a problem that I faced while running gef for gdb on cloud servers (I use them to use different CPU Architectures). For the last 6 months, I have been facing this problem and today was the day when I finally got the motivation to track down the problem. I am pretty sure that many gef users face the problem of getting a Unicode Encode Error and not using the goodness of gef tool. This was something that I solved in this patch.

The Unicode Encode Error occurs right from the start, the moment when gef is installed. Now this is not in every case, like gef was working fine with my local machine but fails in certain systems due to the error. This one-word change solved the problem of this error. A beginner who is trying to run gef first time on a machine that would throw this error might face an issue like this and end up not using the power of gef. The prompt arrow was causing the Unicode Encode Error. The Prompt had an error with the ASCII character "➤" and I replaced it with ">" which is common in the case of most of the terminals. This small change ran the gef, that I had abandoned for more than 6 months.

We all need to use the goodness of gef and beginners must start by using gef. But this small error of prompt pointer makes the tool unusable, and hence this will cause the user base to go down. By changing the prompt (I know that the earlier prompt was cool), the error can be fixed and the debugger is ready to be used and learned by a beginner. The practical usage is more useful. I don't want any other person to stop using this tool for more than 6 months due to a single character error.

  • My code follows the code style of this project.
  • My change includes a change to the documentation, if required.
  • If my change adds new code, adequate tests have been added.
  • I have read and agree to the CONTRIBUTING document.

@github-actions
Copy link

🤖 Coverage Update

  • Commit: 5927df4
  • Current Coverage: 71.6387%
  • New Coverage: 71.6387%
  • Diff: 0.0

To this point, this PR:

  • does not include changes to tests
  • does not include changes to documentation
  • does not include forbidden words

@Grazfather
Copy link
Collaborator

The prompt is non ascii. If it doesn't work, it's a problem with your terminal, not with GEF. You need to enable utf-8.

@hugsy
Copy link
Owner

hugsy commented Oct 24, 2023

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

Successfully merging this pull request may close these issues.

3 participants