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

TypeError: ord() expected a character, but string of length 9 found #73

Closed
SeanSobey opened this issue Oct 2, 2019 · 1 comment · Fixed by #81
Closed

TypeError: ord() expected a character, but string of length 9 found #73

SeanSobey opened this issue Oct 2, 2019 · 1 comment · Fixed by #81

Comments

@SeanSobey
Copy link

I am using the dockerized version to run the ligaturize the latest Hack font. I am getting this error:

Copyright (c) 2000-2019. See AUTHORS for Contributors.
 License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
 with many parts BSD <http://fontforge.org/license.html>. Please read LICENSE.
 Version: 20190801
 Based on sources from 10:15 UTC  2-Sep-2019-ML-D-GDK3.
Your version of iconv does not support the "Mac Roman" encoding.
If this causes problems, reconfigure --without-iconv.The following table(s) in the font have been ignored by FontForge
    Ignoring 'DSIG' digital signature table
Ligaturizing font input (Hack) as 'Hack Ligaturized'
     ...using ligatures from fonts/fira/distr/otf/FiraCode-Regular.otf
Traceback (most recent call last): 
   File "ligaturize.py", line 336, in <module> 
   File "ligaturize.py", line 333, in main 
        ligaturize_font(**vars(parse_args())) 
   File "ligaturize.py", line 273, in ligaturize_font 
        creator.add_ligature(lig_spec['chars'], lig_spec['firacode_ligature_name']) 
   File "ligaturize.py", line 164, in add_ligature 
        self.font[ord(char)].glyphname = char
TypeError: ord() expected a character, but string of length 9 found     

Here is the issue on the dockerized fork, and what seems a related issue.

@fsvehla
Copy link

fsvehla commented Nov 19, 2019

Can reproduce with Lekton

gaplo917 added a commit to gaplo917/Ligaturizer that referenced this issue Jan 26, 2020
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 a pull request may close this issue.

2 participants