Script 'mail_helper' called by ro Hello packager, This is just FYI. Your package was checked in in distribution "sle12" by autobuild-member: ro. Here comes the log... ---------------------------%<------------------------------ Hi, here is the log from ci_new_pac /mounts/work_src_done/SLE12/yast2-ldap -> sle12 Changes: -------- --- /work/SRC/SUSE:SLE-12:GA/yast2-ldap/yast2-ldap.changes 2014-02-14 14:00:54.000000000 +0100 +++ /mounts/work_src_done/SLE12/yast2-ldap/yast2-ldap.changes 2014-03-13 12:48:45.000000000 +0100 @@ -1,0 +2,13 @@ +Wed Mar 12 16:22:18 UTC 2014 - varkoly@suse.com + +- ldap.conf is not case sensitive +- @ldap_tls is a string not a boolean +- code clean up. Remove not used variables +- 3.1.7 + +------------------------------------------------------------------- +Thu Feb 13 13:59:06 UTC 2014 - varkoly@suse.com + +- Fix SCR.Reads and Writes first parameter. This must be path. + +------------------------------------------------------------------- calling whatdependson for sle12-i586 Packages directly triggered for rebuild: - yast2-ldap - at least 11 other packages ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/SUSE:SLE-12:GA/yast2-ldap (Old) and /mounts/work_src_done/SLE12/yast2-ldap (BS:build ID:34307 MAIL:yast-commit@opensuse.org) (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "yast2-ldap", Maintainer is "yast-commit@opensuse.org" Old: ---- yast2-ldap-3.1.6.tar.bz2 New: ---- yast2-ldap-3.1.7.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2-ldap.spec ++++++ --- /var/tmp/diff_new_pack.NPI8lR/_old 2014-03-14 11:51:22.000000000 +0100 +++ /var/tmp/diff_new_pack.NPI8lR/_new 2014-03-14 11:51:22.000000000 +0100 @@ -17,7 +17,7 @@ Name: yast2-ldap -Version: 3.1.6 +Version: 3.1.7 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build ++++++ yast2-ldap-3.1.6.tar.bz2 -> yast2-ldap-3.1.7.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-ldap-3.1.6/conf/ldap_conf.scr new/yast2-ldap-3.1.7/conf/ldap_conf.scr --- old/yast2-ldap-3.1.6/conf/ldap_conf.scr 2014-02-13 17:53:16.000000000 +0100 +++ new/yast2-ldap-3.1.7/conf/ldap_conf.scr 2014-03-13 12:44:26.000000000 +0100 @@ -28,7 +28,7 @@ `ag_ini( `IniAgent( "/etc/openldap/ldap.conf" , - $[ "options" : ["global_values", "repeat_names"], + $[ "options" : ["global_values", "repeat_names","ignore_case"], "comments" : [ "^#.*", "^[ \t]*$", ], "params" : [ $[ "match" : [ "^[ \t]*([a-zA-Z0-9_]+)[ \t]+(.*[^ \t]|)[ \t]*$", diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-ldap-3.1.6/package/yast2-ldap.changes new/yast2-ldap-3.1.7/package/yast2-ldap.changes --- old/yast2-ldap-3.1.6/package/yast2-ldap.changes 2014-02-13 17:53:16.000000000 +0100 +++ new/yast2-ldap-3.1.7/package/yast2-ldap.changes 2014-03-13 12:44:26.000000000 +0100 @@ -1,4 +1,17 @@ ------------------------------------------------------------------- +Wed Mar 12 16:22:18 UTC 2014 - varkoly@suse.com + +- ldap.conf is not case sensitive +- @ldap_tls is a string not a boolean +- code clean up. Remove not used variables +- 3.1.7 + +------------------------------------------------------------------- +Thu Feb 13 13:59:06 UTC 2014 - varkoly@suse.com + +- Fix SCR.Reads and Writes first parameter. This must be path. + +------------------------------------------------------------------- Thu Feb 13 11:12:38 UTC 2014 - varkoly@suse.com - Remove not used subroutines from the Ldap module diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-ldap-3.1.6/package/yast2-ldap.spec new/yast2-ldap-3.1.7/package/yast2-ldap.spec --- old/yast2-ldap-3.1.6/package/yast2-ldap.spec 2014-02-13 17:53:16.000000000 +0100 +++ new/yast2-ldap-3.1.7/package/yast2-ldap.spec 2014-03-13 12:44:26.000000000 +0100 @@ -17,7 +17,7 @@ Name: yast2-ldap -Version: 3.1.6 +Version: 3.1.7 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-ldap-3.1.6/src/Ldap.rb new/yast2-ldap-3.1.7/src/Ldap.rb --- old/yast2-ldap-3.1.6/src/Ldap.rb 2014-02-13 17:53:16.000000000 +0100 +++ new/yast2-ldap-3.1.7/src/Ldap.rb 2014-03-13 12:44:26.000000000 +0100 @@ -35,10 +35,7 @@ Yast.import "UI" textdomain "ldap" - Yast.import "Autologin" - Yast.import "Directory" Yast.import "FileUtils" - Yast.import "DNS" Yast.import "Hostname" Yast.import "Label" Yast.import "Message" @@ -47,7 +44,6 @@ Yast.import "Package" Yast.import "Pam" Yast.import "Popup" - Yast.import "ProductFeatures" Yast.import "Progress" Yast.import "Report" Yast.import "Service" @@ -73,7 +69,6 @@ # Are LDAP services available via nsswitch.conf? @start = false - @old_start = false # Is NIS service available? If yes, and LDAP client will be enabled, warn # user (see bug #36981) @@ -89,11 +84,9 @@ # which attribute have LDAP groups for list of members @member_attribute = "" - @old_member_attribute = "" # IP addresses of LDAP server. @server = "" - @old_server = "" # local settings modified? @modified = false @@ -103,10 +96,8 @@ # base DN @base_dn = "" - @old_base_dn = "" - @base_dn_changed = false - @ldap_tls = true + @ldap_tls = "no" # CA certificates for server certificate verification # At least one of these are required if tls_checkpeer is "yes" @@ -156,9 +147,6 @@ # DN of currently edited template @current_template_dn = "" - # if LDAP configuration objects should be created automaticaly - @create_ldap = false - # if eDirectory is used as server @nds = false @@ -170,9 +158,6 @@ # if OES is used as a client @oes = false - # expert ui means "server product" - @expert_ui = false - # defaults for adding new config objects and templates @new_objects = { "suseUserConfiguration" => { @@ -345,9 +330,8 @@ # Set new LDAP base DN # @param [String] new_base_dn a new base DN def SetBaseDN(new_base_dn) + @base_dn_changed = true if @base_dn != new_base_dn @base_dn = new_base_dn - @base_dn_changed = true if @base_dn != @old_base_dn && @old_base_dn != "" - nil end @@ -378,7 +362,7 @@ # @return entry value def ReadLdapConfEntry(entry, defvalue) value = defvalue - ret = SCR.Read(".ldap_conf.v."+ entry ) + ret = SCR.Read(path(".ldap_conf.v."+ entry )) if ret == nil value = defvalue elsif Ops.is_list?(ret) @@ -393,7 +377,7 @@ # @param [String] entry entry name # @return entry value def ReadLdapConfEntries(entry) - ret = SCR.Read( ".ldap_conf.v."+ entry ) + ret = SCR.Read(path( ".ldap_conf.v."+ entry )) if ret == nil return [] elsif Ops.is_list?(ret) @@ -407,7 +391,7 @@ # @param [String] entry name # @param [String] value; if value is nil, entry will be removed def WriteLdapConfEntry(entry, value) - SCR.Write( ".ldap_conf.v." + entry, value.nil ? nil : [value] ) + SCR.Write(path( ".ldap_conf.v." + entry ), value.nil ? nil : [value] ) nil end @@ -431,7 +415,7 @@ end end values = [Builtins.mergestring(value, " ")] if Builtins.size(current) == 0 - SCR.Write( ".ldap_conf.v." + entry, values ) + SCR.Write( path(".ldap_conf.v." + entry), values ) nil end @@ -444,7 +428,7 @@ current = Builtins.maplist(current) { |e| Builtins.tolower(e) } if !Builtins.contains(current, Builtins.tolower(value)) - SCR.Write( ".ldap_conf.v." + entry, current | [value] ) + SCR.Write( path(".ldap_conf.v." + entry), current | [value] ) end nil @@ -492,16 +476,13 @@ # Reads LDAP settings from the SCR # @return success def Read - @expert_ui = ProductFeatures.GetFeature("globals", "ui_mode") == "expert" - - CheckOES() ["passwd", "group", "passwd_compat", "group_compat"].each { |db| @nsswitch[db] = Nsswitch.ReadDb(db) } # 'start' means that LDAP is present in nsswitch somehow... either as 'compat'/'ldap'... @start = @nsswitch["passwd"].include?("ldap") || ( @nsswitch["passwd"].include?("compat") && @nsswitch["passwd_compat"].include?("ldap") ) || - ( @auth["oes"] && @nsswitch["passwd"].include?("nam") ) + ( CheckOES() && @nsswitch["passwd"].include?("nam") ) if @start # nss_ldap is used @@ -530,6 +511,8 @@ @tls_cacertdir = ReadLdapConfEntry("TLS_CACERTDIR", "") @bind_dn = ReadLdapConfEntry("BINDDN","cn=Administrator," + @base_dn ) @base_config_dn = "ou=ldapconfig," +@base_dn + + Builtins.y2milestone("Read LDAP Settings: server %1, base_dn %2, bind_dn %3, base_config_dn %4",@server, @base_dn, @bind_dn, @base_config_dn) true end @@ -796,7 +779,7 @@ args = { "hostname" => GetFirstServer(@server), "port" => GetFirstPort(@server), - "use_tls" => @ldap_tls ? "yes" : "no", + "use_tls" => @ldap_tls, "cacertdir" => @tls_cacertdir, "cacertfile" => @tls_cacertfile } @@ -806,7 +789,7 @@ ret = _("Unknown error. Perhaps 'yast2-ldap' is not available.") else @ldap_initialized = init - @tls_when_initialized = @ldap_tls + @tls_when_initialized = ( @ldap_tls == "yes" ) ret = LDAPError() if !init end ret @@ -986,7 +969,7 @@ args = { "hostname" => GetFirstServer(@server), "port" => GetFirstPort(@server), - "use_tls" => @ldap_tls ? "yes" : "no", + "use_tls" => @ldap_tls, "cacertdir" => @tls_cacertdir, "cacertfile" => @tls_cacertfile } @@ -1049,6 +1032,7 @@ # @param anonymous if anonymous access could be allowed # @return password def GetLDAPPassword(enable_anonymous) + Read() if @bind_dn.empty? UI.OpenDialog( Opt(:decorated), VBox( @@ -2247,19 +2231,15 @@ publish :variable => :required_packages, :type => "list <string>" publish :variable => :write_only, :type => "boolean" publish :variable => :start, :type => "boolean" - publish :variable => :old_start, :type => "boolean" publish :variable => :nis_available, :type => "boolean" publish :variable => :_autofs_allowed, :type => "boolean" publish :variable => :_start_autofs, :type => "boolean" publish :variable => :login_enabled, :type => "boolean" publish :variable => :member_attribute, :type => "string" - publish :variable => :old_member_attribute, :type => "string" publish :variable => :server, :type => "string" - publish :variable => :old_server, :type => "string" publish :variable => :modified, :type => "boolean" publish :variable => :openldap_modified, :type => "boolean" publish :variable => :base_dn, :type => "string", :private => true - publish :variable => :old_base_dn, :type => "string", :private => true publish :variable => :base_dn_changed, :type => "boolean", :private => true publish :variable => :ldap_tls, :type => "boolean" publish :variable => :tls_cacertdir, :type => "string" @@ -2280,12 +2260,10 @@ publish :variable => :bind_dn, :type => "string" publish :variable => :current_module_dn, :type => "string" publish :variable => :current_template_dn, :type => "string" - publish :variable => :create_ldap, :type => "boolean" publish :variable => :nds, :type => "boolean" publish :variable => :tls_switched_off, :type => "boolean" publish :variable => :nds_checked, :type => "boolean", :private => true publish :variable => :oes, :type => "boolean", :private => true - publish :variable => :expert_ui, :type => "boolean", :private => true publish :variable => :new_objects, :type => "map" publish :variable => :base_template_dn, :type => "string" publish :variable => :ldap_modified, :type => "boolean" continue with "q"... Checked in at Fri Mar 14 11:51:32 CET 2014 by ro Remember to have fun... -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org