Thanks. For now, I updated file manually :) 2013/11/27 Marcus Meissner <meissner@suse.de>:

Hi,

I can confirm that.

I opened https://bugzilla.novell.com/show_bug.cgi?id=852546 for you.

Ciao, Marcus On Wed, Nov 27, 2013 at 10:49:27AM +0100, Henri Gomez wrote:

...

More on this :

It seems defined in /usr/lib/rpm/suse_macros

... # find-supplements.ksyms parses this macro directly out of the spec file: %supplements_kernel_module() \ %{expand:%(if ! rpm -q kernel-syms > /dev/null; then echo "%fail Please add the kernel-syms package to BuildRequires"; fi)}

%suse_version 1230 %sles_version 0 %ul_version 0 ....

2013/11/27 Henri Gomez <henri.gomez@gmail.com>:

...

Hi all,

on openSUSE 13.1, both 32 and 64bits x86 distro, suse_version is reported as 1230 :

rpm --eval "%{suse_version}" 1230

Is it a bug or expected ?

Cheers -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org

-- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org

Hello, Am Mittwoch, 27. November 2013 schrieb Stephan Kulow:

On 27.11.2013 10:45, Henri Gomez wrote:

...

on openSUSE 13.1, both 32 and 64bits x86 distro, suse_version is reported as 1230 :

rpm --eval "%{suse_version}" 1230

It's bug 851877

I just submitted a fix (SR208678) to Base:System/rpm, including a check that makes sure this doesn't happen again in future versions: grep "^%%suse_version $(source /etc/os-release ; echo ${VERSION_ID/./})0" %{SOURCE4} Build failures are much easier to detect than outdated version numbers in a file ;-) Regards, Christian Boltz PS: non-random sig ;-) -- There is nothing wrong with making mistakes, but... make *new* ones. [D.Sim] -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org

Hello, Am Mittwoch, 27. November 2013 schrieb Stephan Kulow:

On 27.11.2013 14:18, Christian Boltz wrote:

...

I just submitted a fix (SR208678) to Base:System/rpm, including a check> that makes sure this doesn't happen again in future versions: grep "^%%suse_version $(source /etc/os-release ; echo ${VERSION_ID/./})0" %{SOURCE4}> Build failures are much easier to detect than outdated version numbers in a file ;-)

You just created a monster of a build cycle. rpm is one of the core-core-core packages and you can't just go and take packages from the very top and build against it.

Oh, nice. I already wondered because it was too easy to add the check ;-) (and I have to admit that I didn't consider build cycles when I added it) Anyway, Michael already declined my SR, so we'll have to wait until someone implements the method he proposed. I hope this someone does it before we see the same problem in 13.2 ;-) Regards, Christian Boltz -- Get back there in front of the computer NOW. Christmas can wait. [Linus "the Grinch" Torvalds, 24 Dec 2000 on linux-kernel] -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org

On Wed, Nov 27, 2013 at 02:18:24PM +0100, Christian Boltz wrote:

Hello,

Am Mittwoch, 27. November 2013 schrieb Stephan Kulow:

...

On 27.11.2013 10:45, Henri Gomez wrote:

...

on openSUSE 13.1, both 32 and 64bits x86 distro, suse_version is reported as 1230 :

rpm --eval "%{suse_version}" 1230

...

It's bug 851877

I just submitted a fix (SR208678) to Base:System/rpm, including a check that makes sure this doesn't happen again in future versions:

grep "^%%suse_version $(source /etc/os-release ; echo ${VERSION_ID/./})0" %{SOURCE4}

Build failures are much easier to detect than outdated version numbers in a file ;-)

Maybe, but /etc/os-release comes from the openSUSE-release, which should not be installed for package building. It's much saner to move the definition from rpm to the release package, like I suggested in the bug report. M. -- Michael Schroeder mls@suse.de SUSE LINUX Products GmbH, GF Jeff Hawn, HRB 16746 AG Nuernberg main(_){while(_=~getchar())putchar(~_-1/(~(_|32)/13*2-11)*13);} -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org