Skip to content

Conversation

@ernix
Copy link
Contributor

@ernix ernix commented Jul 24, 2023

@ernix
Copy link
Contributor Author

ernix commented Jul 24, 2023

@pgnd
Copy link

pgnd commented Jan 12, 2024

@cfaerber

could you merge this and push update to CPAN ?

cpan build/install of Net::IDN::Encode on

fedora 39

logs a warning,

	lib/Net/IDN/Punycode.xs:256:19: warning: implicit declaration of function ‘uvuni_to_utf8_flags’; did you mean ‘uvchr_to_utf8_flags’? [-Wimplicit-function-declaration]
	  256 |                   uvuni_to_utf8_flags((U8*)skip_p, n, UNICODE_ALLOW_ANY);
	      |                   ^~~~~~~~~~~~~~~~~~~
	      |                   uvchr_to_utf8_flags

but on
fedora rawhide

fails at an error

	lib/Net/IDN/Punycode.xs:256:19: error: implicit declaration of function ‘uvuni_to_utf8_flags’; did you mean ‘uvchr_to_utf8_flags’? [-Werror=implicit-function-declaration]
	  256 |                   uvuni_to_utf8_flags((U8*)skip_p, n, UNICODE_ALLOW_ANY);
	      |                   ^~~~~~~~~~~~~~~~~~~
	      |                   uvchr_to_utf8_flags

this patch is currenty applied to the distro release,

https://src.fedoraproject.org/rpms/perl-Net-IDN-Encode/blob/rawhide/f/Net-IDN-Encode-2.500-use_uvchr_to_utf8_flags_instead_of_uvuni_to_utf8_flags.patch

karenetheridge added a commit to karenetheridge/Net-IDN-Encode that referenced this pull request Jan 12, 2024
@karenetheridge
Copy link

karenetheridge commented Jan 12, 2024

I've uploaded the fix in Net-IDN-Encode-2.501-TRIAL.tar.gz (https://github.com/karenetheridge/Net-IDN-Encode).

@pgnd
Copy link

pgnd commented Jan 31, 2024

fyi, unresponsive maintainer request @

https://www.nntp.perl.org/group/perl.module-authors/2024/01/msg11267.html

@karenetheridge
Copy link

@cfaerber Are you still interested in maintaining this module? There have been some inquiries about making this fix (which I uploaded as a TRIAL release in January 2024) into a stable release so it can be more easily used.

I see you haven't been seen on github since January 2022, so I hope you are well?

@mvsjes2
Copy link

mvsjes2 commented Oct 10, 2025

Well, I've spent a day and a half trying to get carmel to install Karen's work around (thanks Karen!) and have run into one roadblock after another. With perl5's slow sunsetting, we're going to run into this more and more, which will create a positive feedback loop. We need to make exceptions to fully adopting a module for small maintenance fixes when the author has long since left.

The author's last activity on github was January 2022.

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.

4 participants