Script 'mail_helper' called by mrueckert
Hello packager,
This is just FYI. Your package was checked in in distribution "sle12"
by autobuild-member: mrueckert.
Here comes the log...
---------------------------%<------------------------------
Hi,
here is the log from ci_new_pac /work/src/done/SLE12/yast2-bootloader -> sle12
## BNC# 880439 : "Unable to install FCOE BFS when have multiple multipath mappers" (RESOLVED/FIXED)
## BNC# 875819 : "Non Latin characters translation makes GRUB menu not to use it / Wrong JP translation" (RESOLVED/FIXED)
Changes:
--------
--- /work/SRC/SUSE:SLE-12:GA/yast2-bootloader/yast2-bootloader.changes 2014-06-04 08:33:40.000000000 +0200
+++ /work/src/done/SLE12/yast2-bootloader/yast2-bootloader.changes 2014-06-06 09:12:55.000000000 +0200
@@ -1,0 +2,13 @@
+Thu Jun 5 11:00:35 UTC 2014 - jsrain@suse.cz
+
+- adjusted wording if disk order could not be detected (bnc#880439)
+- 3.1.50
+
+-------------------------------------------------------------------
+Wed Jun 4 09:10:42 UTC 2014 - jreidinger@suse.com
+
+- remove translation of section as it is generated in GRUB2 and
+ never work reliably fro GRUB1 (bnc#875819)
+- 3.1.49
+
+-------------------------------------------------------------------
calling whatdependson for sle12-i586
Packages directly triggered for rebuild:
- yast2-bootloader
- yast2-kdump
- yast2-product-creator
- yast2-reipl
- yast2-schema
- yast2-vm
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/SUSE:SLE-12:GA/yast2-bootloader (Old)
and /work/src/done/SLE12/yast2-bootloader (BS:build ID:39081 MAIL:yast-commit@opensuse.org) (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "yast2-bootloader", Maintainer is "yast-commit@opensuse.org"
Old:
----
yast2-bootloader-3.1.48.tar.bz2
New:
----
yast2-bootloader-3.1.50.tar.bz2
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ yast2-bootloader.spec ++++++
--- /var/tmp/diff_new_pack.Ap3xoF/_old 2014-06-06 17:18:14.000000000 +0200
+++ /var/tmp/diff_new_pack.Ap3xoF/_new 2014-06-06 17:18:14.000000000 +0200
@@ -17,7 +17,7 @@
Name: yast2-bootloader
-Version: 3.1.48
+Version: 3.1.50
Release: 0
BuildRoot: %{_tmppath}/%{name}-%{version}-build
++++++ yast2-bootloader-3.1.48.tar.bz2 -> yast2-bootloader-3.1.50.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.1.48/package/yast2-bootloader.changes new/yast2-bootloader-3.1.50/package/yast2-bootloader.changes
--- old/yast2-bootloader-3.1.48/package/yast2-bootloader.changes 2014-06-03 18:16:21.000000000 +0200
+++ new/yast2-bootloader-3.1.50/package/yast2-bootloader.changes 2014-06-06 09:11:16.000000000 +0200
@@ -1,4 +1,17 @@
-------------------------------------------------------------------
+Thu Jun 5 11:00:35 UTC 2014 - jsrain@suse.cz
+
+- adjusted wording if disk order could not be detected (bnc#880439)
+- 3.1.50
+
+-------------------------------------------------------------------
+Wed Jun 4 09:10:42 UTC 2014 - jreidinger@suse.com
+
+- remove translation of section as it is generated in GRUB2 and
+ never work reliably fro GRUB1 (bnc#875819)
+- 3.1.49
+
+-------------------------------------------------------------------
Mon Jun 3 18:13:05 UTC 2014 - dvaleev@suse.com
- Setting boot flag on GPT PReP resets prep flag which leads to
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.1.48/package/yast2-bootloader.spec new/yast2-bootloader-3.1.50/package/yast2-bootloader.spec
--- old/yast2-bootloader-3.1.48/package/yast2-bootloader.spec 2014-06-03 18:16:21.000000000 +0200
+++ new/yast2-bootloader-3.1.50/package/yast2-bootloader.spec 2014-06-06 09:11:16.000000000 +0200
@@ -17,7 +17,7 @@
Name: yast2-bootloader
-Version: 3.1.48
+Version: 3.1.50
Release: 0
BuildRoot: %{_tmppath}/%{name}-%{version}-build
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.1.48/src/include/bootloader/routines/misc.rb new/yast2-bootloader-3.1.50/src/include/bootloader/routines/misc.rb
--- old/yast2-bootloader-3.1.48/src/include/bootloader/routines/misc.rb 2014-06-03 18:16:21.000000000 +0200
+++ new/yast2-bootloader-3.1.50/src/include/bootloader/routines/misc.rb 2014-06-06 09:11:16.000000000 +0200
@@ -753,13 +753,6 @@
- # Create translated name of a section
- # @param [String] orig string original section name
- # @return translated section name
- def translateSectionTitle(orig)
- GfxMenu.translateSectionTitle(orig, getLoaderType(false))
- end
-
# Check if device is MBR of a disk
# @param [String] device string device to check
# @return [Boolean] true if is MBR
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.1.48/src/modules/BootCommon.rb new/yast2-bootloader-3.1.50/src/modules/BootCommon.rb
--- old/yast2-bootloader-3.1.48/src/modules/BootCommon.rb 2014-06-03 18:16:21.000000000 +0200
+++ new/yast2-bootloader-3.1.50/src/modules/BootCommon.rb 2014-06-06 09:11:16.000000000 +0200
@@ -250,7 +250,7 @@
# @return a map describing the section
def CreateLinuxSection(title)
ret = {
- "name" => translateSectionTitle(title),
+ "name" => title,
"original_name" => title,
"type" => "image",
"__auto" => true,
@@ -1086,7 +1086,6 @@
publish :function => :restoreMBR, :type => "boolean (string)"
publish :function => :UpdateKernelParams, :type => "string (string)"
publish :function => :getSwapPartitions, :type => "map ()"
- publish :function => :translateSectionTitle, :type => "string (string)"
publish :function => :UpdateInstallationKernelParameters, :type => "void ()"
publish :function => :GetAdditionalFailsafeParams, :type => "string ()"
publish :function => :UpdateGfxMenuContents, :type => "boolean ()"
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.1.48/src/modules/BootGRUB.rb new/yast2-bootloader-3.1.50/src/modules/BootGRUB.rb
--- old/yast2-bootloader-3.1.48/src/modules/BootGRUB.rb 2014-06-03 18:16:21.000000000 +0200
+++ new/yast2-bootloader-3.1.50/src/modules/BootGRUB.rb 2014-06-06 09:11:16.000000000 +0200
@@ -421,7 +421,7 @@
label != "Vendor diagnostics" && # <- should probably be '&&' => not Thinkpad MBR AND not "Vendor diagnostics" partition type
label != "Vendor diagnostic")
m = {
- "name" => BootCommon.translateSectionTitle(label),
+ "name" => label,
"type" => "other",
"original_name" => label,
"chainloader" => BootStorage.Dev2MountByDev(dev),
@@ -440,7 +440,7 @@
out = Builtins.add(
out,
{
- "name" => BootCommon.translateSectionTitle("hard disk"),
+ "name" => "hard disk",
"original_name" => "hard_disk",
"type" => "other",
"chainloader" => BootStorage.Dev2MountByDev(BootCommon.mbrDisk),
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.1.48/src/modules/BootSupportCheck.rb new/yast2-bootloader-3.1.50/src/modules/BootSupportCheck.rb
--- old/yast2-bootloader-3.1.48/src/modules/BootSupportCheck.rb 2014-06-03 18:16:21.000000000 +0200
+++ new/yast2-bootloader-3.1.50/src/modules/BootSupportCheck.rb 2014-06-06 09:11:16.000000000 +0200
@@ -231,9 +231,7 @@
if BootStorage.bois_id_missing
AddNewProblem(
- _(
- "It was not possible to determine the exact order of disks for device map. The order of disks can be changed in \"Boot Loader Installation Details\""
- )
+ _("YaST could not determine the exact boot order of disks needed for the device map. Review and possibly adjust the boot order of disks in \"Boot Loader Installation Details\"")
)
return false
end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-bootloader-3.1.48/src/modules/GfxMenu.rb new/yast2-bootloader-3.1.50/src/modules/GfxMenu.rb
--- old/yast2-bootloader-3.1.48/src/modules/GfxMenu.rb 2014-06-03 18:16:21.000000000 +0200
+++ new/yast2-bootloader-3.1.50/src/modules/GfxMenu.rb 2014-06-06 09:11:16.000000000 +0200
@@ -30,293 +30,10 @@
@enable_sound_signals = false
end
- # Replace every match of given regular expression in a string with a
- # replacement string
- #
- # e.g. ReplaceRegexMatch( "abcdef12ef34gh000", "[0-9]+", "_A_" ) -> "abcdef_A_ef_A_gh_A_"
- #
- # @param [String] input string that may contain substrings matching regex
- # @param [String] regex regular expression to search for, must not contain brackets
- # @param [String] repl string that replaces every substring matching the regex
- # @return [String] that has matches replaced
- def ReplaceRegexMatch(input, regex, repl)
- return "" if input == nil || Ops.less_than(Builtins.size(input), 1)
- rest = input
- output = ""
- if Builtins.regexpmatch(rest, regex)
- p = Builtins.regexppos(rest, regex)
- begin
- output = Ops.add(
- Ops.add(
- output,
- Builtins.substring(rest, 0, Ops.get_integer(p, 0, 0))
- ),
- repl
- )
- rest = Builtins.substring(
- rest,
- Ops.add(Ops.get_integer(p, 0, 0), Ops.get_integer(p, 1, 0))
- )
- p = Builtins.regexppos(rest, regex)
- end while Ops.greater_than(Builtins.size(p), 0)
- end
- Ops.add(output, rest)
- end
-
- # Create translated name of a section
- # @param [String] orig string original section name
- # @param [String] loader string bootloader type
- # @return translated section name
- def translateSectionTitle(orig, loader)
- #
- # FIXME: handling of bootloader-specific restrictions should be done
- # in perl-Bootloader
- #
- trans = {
- # entry of bootloader menu - only a-z, A-Z, 0-9, _ and blank space
- # are allowed, otherwise translartion won't be used
- # try to keep short, may be shortened due to bootloader limitations
- "linux" => _(
- "Linux"
- ),
- # entry of bootloader menu - only a-z, A-Z, 0-9, _ and blank space
- # are allowed, otherwise translartion won't be used
- # try to keep short, may be shortened due to bootloader limitations
- "failsafe" => _(
- "Failsafe"
- ),
- # entry of bootloader menu - only a-z, A-Z, 0-9, _ and blank space
- # are allowed, otherwise translartion won't be used
- # try to keep short, may be shortened due to bootloader limitations
- "floppy" => _(
- "Floppy"
- ),
- # entry of bootloader menu - only a-z, A-Z, 0-9, _ and blank space
- # are allowed, otherwise translartion won't be used
- # try to keep short, may be shortened due to bootloader limitations
- "hard disk" => _(
- "Hard Disk"
- ),
- # entry of bootloader menu - only a-z, A-Z, 0-9, _ and blank space
- # are allowed, otherwise translartion won't be used
- # try to keep short, may be shortened due to bootloader limitations
- "memtest86" => _(
- "Memory Test"
- ),
- # entry of bootloader menu - only a-z, A-Z, 0-9, _ and blank space
- # are allowed, otherwise translartion won't be used
- # try to keep short, may be shortened due to bootloader limitations
- "original MBR" => _(
- "MBR before Installation"
- ),
- # entry of bootloader menu - only a-z, A-Z, 0-9, _ and blank space
- # are allowed, otherwise translartion won't be used
- # try to keep short, may be shortened due to bootloader limitations
- "previous" => _(
- "Previous Kernel"
- ),
- # entry of bootloader menu - only a-z, A-Z, 0-9, _ and blank space
- # are allowed, otherwise translartion won't be used
- # try to keep short, may be shortened due to bootloader limitations
- "Vendor diagnostics" => _(
- "Vendor Diagnostics"
- )
- }
- not_trans = {
- "linux" => "Linux",
- "failsafe" => "Failsafe",
- "floppy" => "Floppy",
- "hard disk" => "Hard Disk",
- "memtest86" => "Memory Test",
- "original MBR" => "MBR before Installation",
- "windows" => "Windows",
- "xen" => "XEN"
- }
- translated = Ops.get_string(trans, orig, "\n") # not allowed character
- # not_translated version will be used
- filtered = Builtins.filterchars(
- translated,
- "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890 _"
- )
- if Builtins.size(filtered) != Builtins.size(translated)
- Builtins.y2warning("Incorrect translation %1 -> %2", orig, translated)
- return Ops.get_string(not_trans, orig, orig)
- end
- if loader != "grub"
- # This may become an alternative setup for grub in the future
- # (requiring a separate menu.lst on an extra partition for the
- # first level, along with the changes in several parts of the
- # BootGRUB code for this).
- # AI: rw/od should discuss this with the grub maintainer and
- # create a feature for this.
- if orig == "linux"
- Yast.import "Product"
- product = Product.short_name
- prod_filtered = Builtins.filterchars(
- product,
- "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890 _."
- )
- filtered = prod_filtered if product == prod_filtered && product != " "
- end
- Builtins.y2milestone("adapting section title: %1", filtered)
- # label tag for lilo.conf has a restricted valid character set and
- # limited allowed string length
- cutoff = ""
-
- # Limit length to 11 characters, but keep it "nice"
- # 1. cut off linux- prefix if found
- if Ops.greater_than(Builtins.size(filtered), 11)
- cutoff = Builtins.regexpsub(filtered, "^[Ll][Ii][Nn][Uu][Xx]-", "")
- filtered = cutoff if cutoff != nil
- end
-
- while Ops.greater_than(Builtins.size(filtered), 11)
- # 2. cut off last word, break if no more found
- cutoff = Builtins.regexpsub(filtered, "^(.*) [^ ]*$", "\\1")
- Builtins.y2milestone("cutoff is: %1", cutoff)
- if cutoff == nil || Builtins.size(cutoff) == Builtins.size(filtered)
- break
- end
- filtered = cutoff
- end
- Builtins.y2milestone("section title without excess words: %1", filtered)
-
- # 3. last resort: cutoff excess characters
- filtered = Builtins.substring(filtered, 0, 11)
- Builtins.y2milestone("section title limited to 11 chars: %1", filtered)
-
- # 4. convert not allowed chars to "_"
- # (NOTE: this converts according to lilo requirements, ATM we do
- # not allow ".-" above already; so ATM this converts only " ")
- filtered = ReplaceRegexMatch(
- filtered,
- "[^abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890.-]",
- "_"
- )
- Builtins.y2milestone(
- "section title: filtered unallowed characters: %1",
- filtered
- )
- elsif Builtins.contains(["linux", "failsafe", "previous", "xen"], orig) &&
- !Mode.test
- # for bootloaders that support long section names, like grub:
- Yast.import "Product"
- product = Product.name
- prod_filtered = Builtins.filterchars(
- product,
- "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890 _."
- )
- if product != nil && product == prod_filtered && product != " " &&
- product != ""
- if orig == "linux"
- filtered = prod_filtered
- else
- filtered = Builtins.sformat("%1 -- %2", filtered, prod_filtered)
- end
- end
- end
- filtered
- end
-
- # Get translated section names, including diacritics
- # @param [String] loader string bootloader type
- # @return a map section names translations
- def getTranslationsToDiacritics(loader)
- trans = {
- # entry of bootloader menu - only ISO 8859-1, -2 and -15 characters
- # are allowed. Always remove the leading '_', its just to
- # be able to have translations with and without diacritics
- # please use diacritics here
- "linux" => _(
- "_Linux"
- ),
- # entry of bootloader menu - only ISO 8859-1, -2 and -15 characters
- # are allowed. Always remove the leading '_', its just to
- # be able to have translations with and without diacritics
- # please use diacritics here
- "failsafe" => _(
- "_Failsafe"
- ),
- # entry of bootloader menu - only ISO 8859-1, -2 and -15 characters
- # are allowed. Always remove the leading '_', its just to
- # be able to have translations with and without diacritics
- # please use diacritics here
- "floppy" => _(
- "_Floppy"
- ),
- # entry of bootloader menu - only ISO 8859-1, -2 and -15 characters
- # are allowed. Always remove the leading '_', its just to
- # be able to have translations with and without diacritics
- # please use diacritics here
- "hard disk" => _(
- "_Hard Disk"
- ),
- # entry of bootloader menu - only ISO 8859-1, -2 and -15 characters
- # are allowed. Always remove the leading '_', its just to
- # be able to have translations with and without diacritics
- # please use diacritics here
- "memtest86" => _(
- "_Memory Test"
- ),
- # entry of bootloader menu - only ISO 8859-1, -2 and -15 characters
- # are allowed. Always remove the leading '_', its just to
- # be able to have translations with and without diacritics
- # please use diacritics here
- "original MBR" => _(
- "_MBR before Installation"
- ),
- # entry of bootloader menu - only ISO 8859-1, -2 and -15 characters
- # are allowed. Always remove the leading '_', its just to
- # be able to have translations with and without diacritics
- # please use diacritics here
- "previous" => _(
- "_Previous Kernel"
- ),
- # entry of bootloader menu - only ISO 8859-1, -2 and -15 characters
- # are allowed. Always remove the leading '_', its just to
- # be able to have translations with and without diacritics
- # please use diacritics here
- "Vendor diagnostics" => _(
- "_Vendor Diagnostics"
- ),
- "xen" => "XEN"
- }
- # trans = filter (string k, string v, trans, {
- # if (substring (v, 0, 1) == "_")
- # {
- # y2warning ("Translation %1 contains leading underscore", v);
- # return false;
- # }
- # return true;
- # });
- trans = Builtins.mapmap(trans) do |k, v|
- v = Builtins.substring(v, 1) if Builtins.substring(v, 0, 1) == "_"
- { k => v }
- end
- ret = Builtins.mapmap(trans) do |k, v|
- il1 = translateSectionTitle(k, loader)
- if Builtins.contains(["linux", "failsafe", "previous", "xen"], k) &&
- !Mode.test
- Yast.import "Product"
- product = Product.name
- if product != " " && product != "" && product != nil
- if k == "linux"
- v = product
- else
- v = Builtins.sformat("%1 (%2)", product, v)
- end
- end
- end
- { il1 => v }
- end
- deep_copy(ret)
- end
-
# FATE#305403: Bootloader beep configuration
# Read status of acoustic signals
# set global variable enable_sound_signals
#
-
def ReadStatusAcousticSignal
ret = -1 # off
@@ -469,20 +186,9 @@
end
end
- # create translation map (in temp file) for the currently active language
- # for gettext (AFAICT), i.e. whatever is found in LANG or LC_MESSAGES --
- # this should be RC_LANG
- trans_file = Builtins.sformat("%1/boot_translations", tmpdir)
- trans_map = getTranslationsToDiacritics(loader)
- trans_list = Builtins.maplist(trans_map) do |k, v|
- Builtins.sformat("%1\n%2", k, v)
- end
- trans_str = Builtins.mergestring(trans_list, "\n")
- trans_str = Ops.add(trans_str, "\n")
- if !lang_supported
- Builtins.y2milestone("Avoiding providing bootloader menu translations")
- trans_str = ""
- end
+ trans_file = tmpdir + "/boot_translations"
+ # do not create translation of section (bnc#875819)
+ trans_str = ""
SCR.Write(path(".target.string"), trans_file, trans_str)
lang_params = "en_EN en" if lang_params == ""
@@ -522,9 +228,6 @@
end
publish :variable => :enable_sound_signals, :type => "boolean"
- publish :function => :ReplaceRegexMatch, :type => "string (string, string, string)"
- publish :function => :translateSectionTitle, :type => "string (string, string)"
- publish :function => :getTranslationsToDiacritics, :type => "map (string)"
publish :function => :ReadStatusAcousticSignal, :type => "void ()"
publish :function => :UpdateGfxMenuContents, :type => "boolean (string)"
publish :function => :Update, :type => "boolean ()"
continue with "q"...
Checked in at Fri Jun 6 17:18:34 CEST 2014 by mrueckert
Remember to have fun...
--
To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org
For additional commands, e-mail: yast-commit+help@opensuse.org