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-nfs-server -> sle12
## BNC# 878878 : "Build0404: yast2: nfs server start failure" (RESOLVED/FIXED)
Changes:
--------
--- /work/SRC/SUSE:SLE-12:GA/yast2-nfs-server/yast2-nfs-server.changes 2014-05-05 17:43:22.000000000 +0200
+++ /mounts/work_src_done/SLE12/yast2-nfs-server/yast2-nfs-server.changes 2014-05-22 10:57:49.000000000 +0200
@@ -1,0 +2,7 @@
+Wed May 21 14:21:22 UTC 2014 - mvidner@suse.com
+
+- Avoid "Unable to start idmapd" (bnc#878878).
+- Removed dead code dealing with the user space nfs-server.rpm.
+- 3.1.7
+
+-------------------------------------------------------------------
calling whatdependson for sle12-i586
Packages directly triggered for rebuild:
- yast2-nfs-server
- yast2-schema
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/SUSE:SLE-12:GA/yast2-nfs-server (Old)
and /mounts/work_src_done/SLE12/yast2-nfs-server (BS:build ID:38343 MAIL:yast-commit@opensuse.org) (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "yast2-nfs-server", Maintainer is "yast-commit@opensuse.org"
Old:
----
yast2-nfs-server-3.1.6.tar.bz2
New:
----
yast2-nfs-server-3.1.7.tar.bz2
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ yast2-nfs-server.spec ++++++
--- /var/tmp/diff_new_pack.WICGvz/_old 2014-05-23 16:14:01.000000000 +0200
+++ /var/tmp/diff_new_pack.WICGvz/_new 2014-05-23 16:14:01.000000000 +0200
@@ -17,7 +17,7 @@
Name: yast2-nfs-server
-Version: 3.1.6
+Version: 3.1.7
Release: 0
URL: https://github.com/yast/yast-nfs-server
++++++ yast2-nfs-server-3.1.6.tar.bz2 -> yast2-nfs-server-3.1.7.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-nfs-server-3.1.6/package/yast2-nfs-server.changes new/yast2-nfs-server-3.1.7/package/yast2-nfs-server.changes
--- old/yast2-nfs-server-3.1.6/package/yast2-nfs-server.changes 2014-05-05 16:35:34.000000000 +0200
+++ new/yast2-nfs-server-3.1.7/package/yast2-nfs-server.changes 2014-05-22 10:55:13.000000000 +0200
@@ -1,4 +1,11 @@
-------------------------------------------------------------------
+Wed May 21 14:21:22 UTC 2014 - mvidner@suse.com
+
+- Avoid "Unable to start idmapd" (bnc#878878).
+- Removed dead code dealing with the user space nfs-server.rpm.
+- 3.1.7
+
+-------------------------------------------------------------------
Mon May 5 15:54:01 CEST 2014 - locilka@suse.com
- Obsolete Service.Status calls replaced with Service.active?
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-nfs-server-3.1.6/package/yast2-nfs-server.spec new/yast2-nfs-server-3.1.7/package/yast2-nfs-server.spec
--- old/yast2-nfs-server-3.1.6/package/yast2-nfs-server.spec 2014-05-05 16:35:34.000000000 +0200
+++ new/yast2-nfs-server-3.1.7/package/yast2-nfs-server.spec 2014-05-22 10:55:13.000000000 +0200
@@ -17,7 +17,7 @@
Name: yast2-nfs-server
-Version: 3.1.6
+Version: 3.1.7
Release: 0
URL: https://github.com/yast/yast-nfs-server
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-nfs-server-3.1.6/src/clients/nfs_server.rb new/yast2-nfs-server-3.1.7/src/clients/nfs_server.rb
--- old/yast2-nfs-server-3.1.6/src/clients/nfs_server.rb 2014-05-05 16:35:34.000000000 +0200
+++ new/yast2-nfs-server-3.1.7/src/clients/nfs_server.rb 2014-05-22 10:55:13.000000000 +0200
@@ -165,13 +165,7 @@
"exports" => { :next => :ws_finish, :abort => :abort }
}
- # If nfs-kernel-server nor nfs-server is not installed, install nfs-kernel-server.
- # TODO: require installation of alternate packages
- if !Package.Installed("nfs-server")
- @use_star_for_anonymous = true
- @spaces_allowed = true
- return nil if !Package.InstallAll(NfsServer.required_packages)
- end
+ Package.InstallAll(NfsServer.required_packages) or return nil
if !NfsServer.Read
Builtins.y2error("read error, bye")
@@ -267,7 +261,7 @@
return false
end
host = Ops.get_string(options, "hosts", "")
- host = @use_star_for_anonymous ? "*" : "" if host == ""
+ host = "*" if host == ""
opts = Ops.get_string(options, "options", "")
opts = @default_options if opts == ""
default_allowed = [Builtins.sformat("%1(%2)", host, opts)]
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-nfs-server-3.1.6/src/include/nfs_server/ui.rb new/yast2-nfs-server-3.1.7/src/include/nfs_server/ui.rb
--- old/yast2-nfs-server-3.1.6/src/include/nfs_server/ui.rb 2014-05-05 16:35:34.000000000 +0200
+++ new/yast2-nfs-server-3.1.7/src/include/nfs_server/ui.rb 2014-05-22 10:55:13.000000000 +0200
@@ -33,16 +33,6 @@
Yast.import "Sequencer"
Yast.import "Wizard"
Yast.include include_target, "nfs_server/routines.rb"
-
- # The kernel-based server can handle spaces in paths,
- # the user-mode server cannot.
- @spaces_allowed = false
-
- # The kernel based server complains about an empty host specification
- # and wants us to use "*". But the user mode server would match it
- # only with an unqualified name. We must not use "*" there
- # to mean an anonymous client.
- @use_star_for_anonymous = false
end
# Ask user for a directory to export. Allow browsing.
@@ -105,21 +95,6 @@
_("The exports table already contains this directory.")
)
ret = nil
- elsif !@spaces_allowed &&
- Builtins.findfirstof(mountpoint, " \t") != nil
- Popup.Message(
- Builtins.sformat(
- # message popup; %1, %2 are package names
- _(
- "The user mode NFS server (%1) cannot export directories\n" +
- "with spaces in their names.\n" +
- "Use the kernel-based server (%2) to do that."
- ),
- "nfs-server",
- "nfs-kernel-server"
- )
- )
- ret = nil
elsif Ops.less_than(SCR.Read(path(".target.size"), mountpoint), 0) &&
!Mode.config
# the dir does not exist
@@ -219,12 +194,9 @@
end
hosts = Convert.to_string(UI.QueryWidget(Id(:hostsent), :Value))
- if @use_star_for_anonymous && hosts == ""
+ if hosts == ""
hosts = "*"
UI.ChangeWidget(Id(:hostsent), :Value, hosts)
- elsif !@use_star_for_anonymous && hosts == "*" # #91175
- hosts = ""
- UI.ChangeWidget(Id(:hostsent), :Value, hosts)
end
if ret == :optsent
@@ -519,37 +491,22 @@
)
# Help, part 2 of 4
- help_text = Ops.add(
- help_text,
+ help_text +=
_(
"<P><b>Host Wild Card</b> sets which hosts can access the selected directory.\n" +
"It can be a single host, groups, wild cards, or\n" +
"IP networks.</P>\n"
)
- )
- # #91175
- if @use_star_for_anonymous
- # Help, part 3 of 4, variant for kernel space server
- help_text = Ops.add(
- help_text,
+ # Help, part 3 of 4
+ help_text +=
_(
"<p>Enter an asterisk (<tt>*</tt>) instead of a name to specify all hosts.</p>"
)
- )
- else
- # Help, part 3 of 4, variant for user space server
- help_text = Ops.add(
- help_text,
- _("<p>Leave the field empty to specify all hosts.</p>")
- )
- end
# Help, part 4 of 4
- help_text = Ops.add(
- help_text,
+ help_text +=
_("<P>Refer to <tt>man exports</tt> for more information.</P>\n")
- )
exports = deep_copy(NfsServer.exports)
@@ -666,13 +623,7 @@
mountpoint2 = GetDirectory(nil, exports)
if mountpoint2 != nil
- default_allowed = [
- Builtins.sformat(
- "%1(%2)",
- @use_star_for_anonymous ? "*" : "",
- GetDefaultOpts(exports, "*")
- )
- ]
+ default_allowed = [ "*(%s)" % GetDefaultOpts(exports, "*") ]
exports = Builtins.add(
exports,
{ "mountpoint" => mountpoint2, "allowed" => default_allowed }
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-nfs-server-3.1.6/src/modules/NfsServer.rb new/yast2-nfs-server-3.1.7/src/modules/NfsServer.rb
--- old/yast2-nfs-server-3.1.6/src/modules/NfsServer.rb 2014-05-05 16:35:34.000000000 +0200
+++ new/yast2-nfs-server-3.1.7/src/modules/NfsServer.rb 2014-05-22 10:55:13.000000000 +0200
@@ -28,7 +28,6 @@
Yast.import "SuSEFirewall"
Yast.import "Wizard"
-
# default value of settings modified
@modified = false
@@ -36,27 +35,21 @@
#
@required_packages = ["nfs-kernel-server"]
-
# Write only, used during autoinstallation.
# Don't run services and SuSEconfig, it's all done at one place.
@write_only = false
-
# Enable nfsv4
@enable_nfsv4 = true
-
# GSS Security ?
@nfs_security = false
-
# Domain name to be used for nfsv4 (idmapd.conf)
@domain = ""
-
-
# Should the server be started?
- # New since 9.0: Exports are independent of this setting.
+ # Exports are independent of this setting.
@start = false
# @example
@@ -71,11 +64,6 @@
#
@exports = []
- # Do we have nfslock? (nfs-utils: yes, nfs-server: no)
- # FIXME: check nfs-kernel-server
- @have_nfslock = true
-
-
# Since SLE 11, there's no portmapper, but rpcbind
@portmapper = "rpcbind"
end
@@ -102,15 +90,6 @@
# @see #exports
def Import(settings)
settings = deep_copy(settings)
- # if (size (settings) == 0)
- # {
- # // Reset - just continue with Set (#24544).
- # }
-
- # To avoid enabling nfslock if it does not exist during autoinstall
- @have_nfslock = Convert.to_boolean(
- SCR.Read(path(".init.scripts.exists"), "nfslock")
- )
Set(settings)
true
end
@@ -148,9 +127,6 @@
:from => "any",
:to => "list