[yast-commit] [ci_new_pac] JFYI yast2 -> sle12
Script 'mail_helper' called by bg Hello packager, This is just FYI. Your package was checked in in distribution "sle12" by autobuild-member: bg. Here comes the log... ---------------------------%<------------------------------ Hi, here is the log from ci_new_pac /mounts/work_src_done/SLE12/yast2 -> sle12 ## BNC# 872960 : "Yast Inst Server: Uknown service 'service:vsftpd', Uknown service 'service:apache2'" (RESOLVED/FIXED) ## BNC# 893583 : "Yast Firewall / Services show: "template service" / "opens ports for foo in order to allow bar"" (RESOLVED/FIXED) Changes: -------- --- /work/SRC/SUSE:SLE-12:GA/yast2/yast2.changes 2014-08-21 15:03:51.000000000 +0200 +++ /mounts/work_src_done/SLE12/yast2/yast2.changes 2014-08-27 14:38:58.000000000 +0200 @@ -1,0 +2,12 @@ +Wed Aug 27 13:24:34 CEST 2014 - locilka@suse.com + +- Lazy-loading SuSEfirewall2 services - some new services can be + added when Yast is already running, and the config has been read + already (bnc#872960) +- Preventing from showing incorrect service name or description if + they are coming from TEMPLATE file (bnc#893583) +- Fixed service file parsing - comments starting with more than one + '#' characters were reported as unexpected input (just warning) +- 3.1.102 + +------------------------------------------------------------------- calling whatdependson for sle12-i586 Packages directly triggered for rebuild: - yast2 - at least 73 other packages ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/SUSE:SLE-12:GA/yast2 (Old) and /mounts/work_src_done/SLE12/yast2 (BS:build ID:43181 MAIL:yast-commit@opensuse.org) (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "yast2", Maintainer is "yast-commit@opensuse.org" Old: ---- yast2-3.1.101.tar.bz2 New: ---- yast2-3.1.102.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2.spec ++++++ --- /var/tmp/diff_new_pack.Y7udeU/_old 2014-08-27 16:11:31.000000000 +0200 +++ /var/tmp/diff_new_pack.Y7udeU/_new 2014-08-27 16:11:31.000000000 +0200 @@ -17,7 +17,7 @@ Name: yast2 -Version: 3.1.101 +Version: 3.1.102 Release: 0 URL: https://github.com/yast/yast-yast2 ++++++ yast2-3.1.101.tar.bz2 -> yast2-3.1.102.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-3.1.101/library/network/src/modules/SuSEFirewall.rb new/yast2-3.1.102/library/network/src/modules/SuSEFirewall.rb --- old/yast2-3.1.101/library/network/src/modules/SuSEFirewall.rb 2014-08-21 13:39:49.000000000 +0200 +++ new/yast2-3.1.102/library/network/src/modules/SuSEFirewall.rb 2014-08-27 14:29:38.000000000 +0200 @@ -2480,7 +2480,7 @@ Progress.New( read_caption, " ", - 4, + 3, [ # TRANSLATORS: Progress step _("Check for network devices"), @@ -2488,8 +2488,6 @@ _("Read current configuration"), # TRANSLATORS: Progress step _("Check possibly conflicting services"), - # TRANSLATORS: Progress step - _("Read dynamic definitions of installed services") ], [ # TRANSLATORS: Progress step @@ -2498,8 +2496,6 @@ _("Reading current configuration..."), # TRANSLATORS: Progress step _("Checking possibly conflicting services..."), - # TRANSLATORS: Progress step - _("Reading dynamic definitions of installed services..."), Message.Finished ], "" @@ -2545,10 +2541,6 @@ Progress.NextStage if have_progress - SuSEFirewallServices.ReadServicesDefinedByRPMPackages - - Progress.NextStage if have_progress - # bnc #399217 # Converting built-in service definitions to services defined by packages ConvertToServicesDefinedByPackages() diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-3.1.101/library/network/src/modules/SuSEFirewallServices.rb new/yast2-3.1.102/library/network/src/modules/SuSEFirewallServices.rb --- old/yast2-3.1.101/library/network/src/modules/SuSEFirewallServices.rb 2014-08-21 13:39:49.000000000 +0200 +++ new/yast2-3.1.102/library/network/src/modules/SuSEFirewallServices.rb 2014-08-27 14:29:38.000000000 +0200 @@ -65,6 +65,9 @@ IGNORED_SERVICES = ["TEMPLATE", "..", "."] + TEMPLATE_SERVICE_NAME = "template service" + TEMPLATE_SERVICE_DESCRIPTION = "opens ports for foo in order to allow bar" + def main textdomain "base" @@ -312,15 +315,22 @@ "comments" => [ # jail followed by anything but jail (immediately) "^[ \t]*#[^#].*$", + # comments that are not commented key:value pairs (see "params") + # they always use two jails + "^[ \t]*##[ \t]*[^([a-zA-Z0-9_]+:.*)]$", + # comments with three jails and more + "^[ \t]*###.*$", # jail alone - "^[ \t]*\#$", + "^[ \t]*#[ \t]*$", # (empty space) "^[ \t]*$", # sysconfig entries "^[ \t]*[a-zA-Z0-9_]+.*" ], "params" => [ - { "match" => ["^##[ \t]*([^:]+):[ \t]*(.*)[ \t]*$", "%s: %s"] } + # commented key:value pairs + # e.g.: ## Name: service name + { "match" => ["^##[ \t]*([a-zA-Z0-9_]+):[ \t]*(.*)[ \t]*$", "%s: %s"] } ] } ) @@ -360,6 +370,7 @@ # # @return [Boolean] if successful def ReadServicesDefinedByRPMPackages + log.info "Reading SuSEfirewall2 services from #{SERVICES_DIR}" @services ||= {} if !FileUtils.Exists(SERVICES_DIR) || @@ -434,9 +445,17 @@ ) ) ) - next if definition.nil? || definition == "" + next if definition.nil? || definition.empty? # call gettext to translate the metadata @services[service_name][metadata_key] = Builtins.dgettext(SERVICES_TEXTDOMAIN, definition) + + # bnc#893583: Sanitize metadata, do not allow using texts from template service + case metadata_key + when "name" + @services[service_name][metadata_key] = filename if definition == TEMPLATE_SERVICE_NAME + when "description" + @services[service_name][metadata_key] = "" if definition == TEMPLATE_SERVICE_DESCRIPTION + end end SCR.UnregisterAgent(path(".firewall_service_metadata")) @@ -445,6 +464,8 @@ end end + log.info "Services found: #{@services.keys.sort}" + true end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-3.1.101/package/yast2.changes new/yast2-3.1.102/package/yast2.changes --- old/yast2-3.1.101/package/yast2.changes 2014-08-21 13:39:50.000000000 +0200 +++ new/yast2-3.1.102/package/yast2.changes 2014-08-27 14:29:39.000000000 +0200 @@ -1,4 +1,16 @@ ------------------------------------------------------------------- +Wed Aug 27 13:24:34 CEST 2014 - locilka@suse.com + +- Lazy-loading SuSEfirewall2 services - some new services can be + added when Yast is already running, and the config has been read + already (bnc#872960) +- Preventing from showing incorrect service name or description if + they are coming from TEMPLATE file (bnc#893583) +- Fixed service file parsing - comments starting with more than one + '#' characters were reported as unexpected input (just warning) +- 3.1.102 + +------------------------------------------------------------------- Thu Aug 21 13:29:52 CEST 2014 - locilka@suse.com - Fixed checking whether SuSEfirewall2 package is selected or diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-3.1.101/package/yast2.spec new/yast2-3.1.102/package/yast2.spec --- old/yast2-3.1.101/package/yast2.spec 2014-08-21 13:39:50.000000000 +0200 +++ new/yast2-3.1.102/package/yast2.spec 2014-08-27 14:29:39.000000000 +0200 @@ -17,7 +17,7 @@ Name: yast2 -Version: 3.1.101 +Version: 3.1.102 Release: 0 URL: https://github.com/yast/yast-yast2 continue with "q"... Checked in at Wed Aug 27 16:12:03 CEST 2014 by bg Remember to have fun... -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org
participants (1)
-
bg