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

Revert "Fix setup" #110

Merged
merged 1 commit into from
Apr 14, 2023
Merged

Revert "Fix setup" #110

merged 1 commit into from
Apr 14, 2023

Conversation

pwaller
Copy link
Owner

@pwaller pwaller commented Apr 14, 2023

Reverts #108.

Apologies to @mkinney for jumping the gun. (Re: #108 (comment))

This reverts commit d9c1ba0.
@pwaller pwaller merged commit 000f410 into master Apr 14, 2023
@pwaller pwaller deleted the revert-108-fix_setup branch April 14, 2023 12:36
@mkinney
Copy link
Contributor

mkinney commented Apr 14, 2023

Thanks for reverting. This was not as simple as I had hoped.

I've been working on a better solution. Will share when I have something.

I've tried merging the fonts with the base package, but the code needs to change for it to work. This line is problematic:

https://github.com/pwaller/pyfiglet/blob/master/pyfiglet/__init__.py#L61

It should be relative to the package installation, not a hard coded path.

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.

2 participants