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

Add support for UTF-8 in message #14

Closed
rbosse opened this issue Mar 21, 2019 · 6 comments · Fixed by #17
Closed

Add support for UTF-8 in message #14

rbosse opened this issue Mar 21, 2019 · 6 comments · Fixed by #17
Assignees

Comments

@rbosse
Copy link

rbosse commented Mar 21, 2019

Had this error:
Wide character in print at /usr/bin/msgconvert line 58.

@sigurdtheone
Copy link

This can be solved by adding "use open ":std", ":encoding(UTF-8)";" to the top of /home//perl5/bin/msgconvert

@mvz
Copy link
Owner

mvz commented May 16, 2020

If anyone seeing this error can provide the following, that would be great:

  • A msg file that triggers the problem
  • The command line you use to trigger the problem
  • The operating system you're using

@ojwb
Copy link
Contributor

ojwb commented Aug 26, 2020

There's actually a file which triggers this in the testsuite already, and no other command line options are required:

$ script/msgconvert t/files/plain_uc_wc_unsent.msg
Wide character in print at script/msgconvert line 58.

I'm using Debian unstable.

(I'm not yet sure if this is the same problem or not, but FWIW I'm getting a similar issue in my own code when I request body_str on an Email::MIME object I get from Email::Outlook::Message:

Wide character at /usr/share/perl5/Email/MIME.pm line 376.

I'm in the process of investigating, went to check if someone had reported it already, and found this ticket and that the file I was testing with triggers this issue too.)

@mvz mvz self-assigned this Aug 26, 2020
@mvz mvz added bug and removed bug labels Aug 26, 2020
@ojwb
Copy link
Contributor

ojwb commented Aug 26, 2020

@mvz I found that the output file in msgconvert used to be set to be :utf8 but that was removed by #7.

In #7 it is mentioned that things are still broken until rjbs/Email-MIME#36 is merged in Email::MIME, but that PR doesn't seem to have gone well and was closed without merging.

It's still unclear to me exactly what's broken and where, but I don't think it's msgconvert (or at least not entirely) as I'm seeing similar issues with my own code.

@ojwb
Copy link
Contributor

ojwb commented Aug 27, 2020

My proposed changes from #17 eliminate this error:

$ perl -Ilib script/msgconvert t/files/plain_uc_wc_unsent.msg
$

(-Ilib is just needed so that the in-tree module is used rather than the system-installed one.)

@mvz
Copy link
Owner

mvz commented Aug 27, 2020

Thanks for looking into this @ojwb

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

Successfully merging a pull request may close this issue.

4 participants