Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package colm for openSUSE:Factory checked in at 2022-07-09 17:04:45 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/colm (Old) and /work/SRC/openSUSE:Factory/.colm.new.1523 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "colm" Sat Jul 9 17:04:45 2022 rev:9 rq:988029 version:0.14.7 Changes: -------- --- /work/SRC/openSUSE:Factory/colm/colm.changes 2021-04-29 22:53:28.545871082 +0200 +++ /work/SRC/openSUSE:Factory/.colm.new.1523/colm.changes 2022-07-09 17:06:12.348968086 +0200 @@ -1,0 +2,8 @@ +Fri Jul 8 12:50:08 UTC 2022 - Guillaume GARDET <guillaume.gar...@opensuse.org> + +- Backport patches to fix signed/unsigned char usage on Arm: + * e1f1c598.patch + * 6895f7d1.patch + * 9f0ac141.patch + +------------------------------------------------------------------- New: ---- 6895f7d1.patch 9f0ac141.patch e1f1c598.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ colm.spec ++++++ --- /var/tmp/diff_new_pack.RaB8Ql/_old 2022-07-09 17:06:12.792968752 +0200 +++ /var/tmp/diff_new_pack.RaB8Ql/_new 2022-07-09 17:06:12.796968758 +0200 @@ -1,7 +1,7 @@ # # spec file for package colm # -# Copyright (c) 2021 SUSE LLC +# Copyright (c) 2022 SUSE LLC # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -28,6 +28,9 @@ Source: https://www.colm.net/files/colm/%name-%version.tar.gz Patch1: fix-library.patch Patch2: reproducible.diff +Patch3: e1f1c598.patch +Patch4: 6895f7d1.patch +Patch5: 9f0ac141.patch BuildRequires: asciidoc BuildRequires: gcc-c++ BuildRequires: libtool ++++++ 6895f7d1.patch ++++++ >From 6895f7d153500a18f08fa9e64972799763c775e6 Mon Sep 17 00:00:00 2001 From: Adrian Thurston <thurs...@colm.net> Date: Sat, 6 Nov 2021 12:09:17 -0700 Subject: [PATCH] C codegen: use SCHAR_MIN and SCHAR_MAX for signed char host type --- src/libfsm/common.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libfsm/common.cc b/src/libfsm/common.cc index b46cc571..349d06f7 100644 --- a/src/libfsm/common.cc +++ b/src/libfsm/common.cc @@ -42,7 +42,7 @@ const char *defaultOutFnC( const char *inputFileName ) HostType hostTypesC[] = { { "char", 0, "char", (CHAR_MIN != 0), true, false, SCHAR_MIN, SCHAR_MAX, 0, UCHAR_MAX, sizeof(char) }, - { "signed", "char", "char", true, true, false, CHAR_MIN, CHAR_MAX, 0, 0, sizeof(char) }, + { "signed", "char", "char", true, true, false, SCHAR_MIN, SCHAR_MAX, 0, 0, sizeof(char) }, { "unsigned", "char", "uchar", false, true, false, 0, 0, 0, UCHAR_MAX, sizeof(unsigned char) }, { "short", 0, "short", true, true, false, SHRT_MIN, SHRT_MAX, 0, 0, sizeof(short) }, { "signed", "short", "short", true, true, false, SHRT_MIN, SHRT_MAX, 0, 0, sizeof(short) }, ++++++ 9f0ac141.patch ++++++ >From 9f0ac141731d635fbdc963c16c198d5de920f31d Mon Sep 17 00:00:00 2001 From: Adrian Thurston <thurs...@colm.net> Date: Sat, 6 Nov 2021 12:09:56 -0700 Subject: [PATCH] C codegen: use the 'signed' prefix when specifying the size of the signed types --- src/libfsm/common.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/libfsm/common.cc b/src/libfsm/common.cc index 349d06f7..9a14a290 100644 --- a/src/libfsm/common.cc +++ b/src/libfsm/common.cc @@ -42,16 +42,16 @@ const char *defaultOutFnC( const char *inputFileName ) HostType hostTypesC[] = { { "char", 0, "char", (CHAR_MIN != 0), true, false, SCHAR_MIN, SCHAR_MAX, 0, UCHAR_MAX, sizeof(char) }, - { "signed", "char", "char", true, true, false, SCHAR_MIN, SCHAR_MAX, 0, 0, sizeof(char) }, + { "signed", "char", "char", true, true, false, SCHAR_MIN, SCHAR_MAX, 0, 0, sizeof(signed char) }, { "unsigned", "char", "uchar", false, true, false, 0, 0, 0, UCHAR_MAX, sizeof(unsigned char) }, { "short", 0, "short", true, true, false, SHRT_MIN, SHRT_MAX, 0, 0, sizeof(short) }, - { "signed", "short", "short", true, true, false, SHRT_MIN, SHRT_MAX, 0, 0, sizeof(short) }, + { "signed", "short", "short", true, true, false, SHRT_MIN, SHRT_MAX, 0, 0, sizeof(signed short) }, { "unsigned", "short", "ushort", false, true, false, 0, 0, 0, USHRT_MAX, sizeof(unsigned short) }, { "int", 0, "int", true, true, false, INT_MIN, INT_MAX, 0, 0, sizeof(int) }, - { "signed", "int", "int", true, true, false, INT_MIN, INT_MAX, 0, 0, sizeof(int) }, + { "signed", "int", "int", true, true, false, INT_MIN, INT_MAX, 0, 0, sizeof(signed int) }, { "unsigned", "int", "uint", false, true, false, 0, 0, 0, UINT_MAX, sizeof(unsigned int) }, { "long", 0, "long", true, true, false, LONG_MIN, LONG_MAX, 0, 0, sizeof(long) }, - { "signed", "long", "long", true, true, false, LONG_MIN, LONG_MAX, 0, 0, sizeof(long) }, + { "signed", "long", "long", true, true, false, LONG_MIN, LONG_MAX, 0, 0, sizeof(signed long) }, { "unsigned", "long", "ulong", false, true, false, 0, 0, 0, ULONG_MAX, sizeof(unsigned long) }, }; ++++++ e1f1c598.patch ++++++ >From e1f1c598d5bf66484e4dccccddd420a8ed1ebc84 Mon Sep 17 00:00:00 2001 From: Adrian Thurston <thurs...@colm.net> Date: Sat, 30 Oct 2021 09:05:06 -0700 Subject: [PATCH] C codegen: decide char signedness based on value of CHAR_MIN Decide signedness of "char" on at runtime (probably compile time) using a check on CHAR_MIN. This should work with cross compiling setups. With this approach, if you plan to use "char" on ARM you cannot literally specify outside the range 0-127 using decimal/hex. If you need that you can use "signed char" or "unsigned char". --- src/libfsm/common.cc | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/src/libfsm/common.cc b/src/libfsm/common.cc index 9ebbd9052..b46cc5714 100644 --- a/src/libfsm/common.cc +++ b/src/libfsm/common.cc @@ -41,18 +41,18 @@ const char *defaultOutFnC( const char *inputFileName ) HostType hostTypesC[] = { - { "char", 0, "char", true, true, false, CHAR_MIN, CHAR_MAX, 0, 0, sizeof(char) }, - { "signed", "char", "char", true, true, false, CHAR_MIN, CHAR_MAX, 0, 0, sizeof(char) }, - { "unsigned", "char", "uchar", false, true, false, 0, 0, 0, UCHAR_MAX, sizeof(unsigned char) }, - { "short", 0, "short", true, true, false, SHRT_MIN, SHRT_MAX, 0, 0, sizeof(short) }, - { "signed", "short", "short", true, true, false, SHRT_MIN, SHRT_MAX, 0, 0, sizeof(short) }, - { "unsigned", "short", "ushort", false, true, false, 0, 0, 0, USHRT_MAX, sizeof(unsigned short) }, - { "int", 0, "int", true, true, false, INT_MIN, INT_MAX, 0, 0, sizeof(int) }, - { "signed", "int", "int", true, true, false, INT_MIN, INT_MAX, 0, 0, sizeof(int) }, - { "unsigned", "int", "uint", false, true, false, 0, 0, 0, UINT_MAX, sizeof(unsigned int) }, - { "long", 0, "long", true, true, false, LONG_MIN, LONG_MAX, 0, 0, sizeof(long) }, - { "signed", "long", "long", true, true, false, LONG_MIN, LONG_MAX, 0, 0, sizeof(long) }, - { "unsigned", "long", "ulong", false, true, false, 0, 0, 0, ULONG_MAX, sizeof(unsigned long) }, + { "char", 0, "char", (CHAR_MIN != 0), true, false, SCHAR_MIN, SCHAR_MAX, 0, UCHAR_MAX, sizeof(char) }, + { "signed", "char", "char", true, true, false, CHAR_MIN, CHAR_MAX, 0, 0, sizeof(char) }, + { "unsigned", "char", "uchar", false, true, false, 0, 0, 0, UCHAR_MAX, sizeof(unsigned char) }, + { "short", 0, "short", true, true, false, SHRT_MIN, SHRT_MAX, 0, 0, sizeof(short) }, + { "signed", "short", "short", true, true, false, SHRT_MIN, SHRT_MAX, 0, 0, sizeof(short) }, + { "unsigned", "short", "ushort", false, true, false, 0, 0, 0, USHRT_MAX, sizeof(unsigned short) }, + { "int", 0, "int", true, true, false, INT_MIN, INT_MAX, 0, 0, sizeof(int) }, + { "signed", "int", "int", true, true, false, INT_MIN, INT_MAX, 0, 0, sizeof(int) }, + { "unsigned", "int", "uint", false, true, false, 0, 0, 0, UINT_MAX, sizeof(unsigned int) }, + { "long", 0, "long", true, true, false, LONG_MIN, LONG_MAX, 0, 0, sizeof(long) }, + { "signed", "long", "long", true, true, false, LONG_MIN, LONG_MAX, 0, 0, sizeof(long) }, + { "unsigned", "long", "ulong", false, true, false, 0, 0, 0, ULONG_MAX, sizeof(unsigned long) }, }; const HostLang hostLangC = {
