Overview

Request 554965 accepted

No description set
Loading...

Dominique Leuenberger's avatar

the removal of idnkit (causing changes in yast) needs to be handled via a 'feature removal requests' (fate)

Can you please file the appropriate ticket with justfication?


Dominique Leuenberger's avatar

@namtrac did anything happen here? Without YaST' team buy-in, we can't remove idnkit from bind


Tomáš Chvátal's avatar

Waiting for the rubygem-simpleidn integration to sle15 and yast to migrate, @kobliha is aware about it


Tomáš Chvátal's avatar

Or whatever nick is Lukas using on obs :D


Vítězslav Čížek's avatar

Just noting that we should ship bind 9.11 (or 9.12, which is in Beta) in SLE-15, because branches 9.9 and 9.10 are going out of maintenance in June 2018 already. https://lists.isc.org/pipermail/bind-announce/2018-January/001071.html


Dominique Leuenberger's avatar
+- Drop idnkit from the build, the bind uses libidn since 2007 to run
+  all the resolutions in dig/etc. [bsc#1030306](https://bugzilla.opensuse.org/show_bug.cgi?id=1030306)

Breaks YaST2 - nothing provides idnkit needed by yast2


Dominique Leuenberger's avatar

idnkit is a dependency to yast2 - needs coordination


Dominique Leuenberger's avatar

Unignored: returned to active backlog.


Tomáš Chvátal's avatar

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?


Dominique Leuenberger's avatar

filed https://bugzilla.opensuse.org/show_bug.cgi?id=1071552 for the yast issue so the YaST team can plan this into one of their upcoming sprints


Ismail Dönmez's avatar

@scarabeus_iv I see multiple problems:

user/group add macros ignore errors -> instead use getent to check if user/group exists and run the command if needed and don't ignore errors

macros in description, does that work?:

 %description chrootenv
 This package contains all directories and files which are common to the
 chroot environment of BIND named and lwresd.  Most is part of the
-structure below /var/lib/named.
+structure below %{_localstatedir}/lib/named.

--enable-rrl is getting removed without an explanation in the changes file.


Vítězslav Čížek's avatar

I'm fixing the user management in https://build.opensuse.org/request/show/554799

The macros in the description were added by spec-cleaner. And yes, it works.

--enable-rrl is no longer a recognized option. The changelog will be updated to state it explicitly.

Request History
Ismail Dönmez's avatar

namtrac created request


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto added repo-checker as a reviewer

Please review build success


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Staging Bot's avatar

staging-bot set openSUSE:Factory:Staging:L as a staging project

Being evaluated by staging project "openSUSE:Factory:Staging:L"


Staging Bot's avatar

staging-bot accepted review

Picked openSUSE:Factory:Staging:L


Saul Goodman's avatar

licensedigger accepted review

ok


Jan Engelhardt's avatar

jengelh accepted review


Dominique Leuenberger's avatar

dimstar_suse set openSUSE:Factory:Staging:I as a staging project

Being evaluated by staging project "openSUSE:Factory:Staging:I"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Moved to openSUSE:Factory:Staging:I


Repo Checker's avatar

repo-checker accepted review

cycle and install check passed


Dominique Leuenberger's avatar

dimstar_suse accepted review

ready to accept


Dominique Leuenberger's avatar

dimstar_suse approved review

ready to accept


Dominique Leuenberger's avatar

dimstar_suse accepted request

Accept to openSUSE:Factory

openSUSE Build Service is sponsored by