Bug 1071552 - yast should stop relying on idnkit (idnconv)
yast should stop relying on idnkit (idnconv)
Status: RESOLVED FEATURE
Classification: openSUSE
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: YaST2
Current
Other Other
: P5 - None : Normal (vote)
: ---
Assigned To: E-mail List
Jiri Srain
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2017-12-06 15:07 UTC by Dominique Leuenberger
Modified: 2022-09-12 10:22 UTC (History)
2 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dominique Leuenberger 2017-12-06 15:07:52 UTC
./modules/Punycode.rb from yast2.rpm calls out to idnconv direclty and yast2 has Requires: idnkit.

idnkit, though, is scheduled for removal from the bind package (see sr https://build.opensuse.org/request/show/545570
The rationale givine in bind's changelog is:

+- Drop idnkit from the build, the bind uses libidn since 2007 to run
+  all the resolutions in dig/etc. bsc#1030306

There is a proposal on above mentioned submit request to change yast to use rubugem-simpleidn instead of calling out to external tools to achive the same functionality
Comment 1 Stefan Hundhammer 2017-12-06 15:13:15 UTC
The .spec file mentions bug #651893.

In the submit request, scarabeus_iv writes:

"yast2 uses it for punycode conversions seems:

It would be best to convert yats2 to use rubygem-simpleidn instead of invoking idnconv on commandline (probably prefered as that uses libidn properly).

OTOH it looks like it does not state anything on ruby 2.3+ https://github.com/mmriis/simpleidn @darix is it now integral part of ruby or something?"

Dimstar wrote in IRC:

[15:47:53] <DimStar> idnkit is needed by (installed) yast2-4.0.11-1.1.x86_64
[15:50:10] <DimStar> ./modules/Punycode.rb:  "/usr/bin/idnconv %1 %2 > %3",
Comment 2 Lukas Ocilka 2017-12-06 15:16:51 UTC
YaST team doesn't have any capacities left to maintain another package,
so either there is this idnkit (idnconv) in some package or someone
introduces simpleidn to SLE 15 and keeps maintaining it, please.

Then we can change the code to use simpleidn.
Comment 3 Imobach Gonzalez Sosa 2017-12-07 09:01:41 UTC
Please, could you open a FATE in order to get this change done?
Comment 4 Lukas Ocilka 2018-01-24 12:50:07 UTC
JFYI: This task has been planned for the current sprint...
Comment 5 Imobach Gonzalez Sosa 2018-02-01 17:15:30 UTC
We have replaced idnconv with the simpleidn library to handle punycode strings. So starting in yast2 4.0.48, idnkit or bind-utils are not needed anymore. You can find out the changes in the SR.

PR: https://github.com/yast/yast-yast2/pull/685
SR: https://build.suse.de/request/show/153668