[opensuse-packaging] Reminder to fix your packages for compiling with GCC 4.3
Hi,
this is a reminder for you to have a look at your packages that fail
in the BETA distribution (which has GCC 4.3 as its compiler). If you
think that the compiler is at fault feel free to report these issues
back to me or file a bugzilla for them.
Builds with a slightly newer version of GCC for x86_64 and i586 are
also available in the home:rguenther:playground build service project.
Thanks,
Richard.
--
Richard Guenther
Richard Guenther escribió:
Hi,
this is a reminder for you to have a look at your packages that fail in the BETA distribution (which has GCC 4.3 as its compiler)
and it is quite a bit of work ;-) 438 packages fails atm at least on x86 ( but not all due to the compiler change) I will try to compile my packages against your repo in the OBS. -- "Two things are infinite: the universe and human stupidity; and I'm not sure about the universe." --Albert Einstein Cristian Rodríguez R, Core Services SUSE LINUX Products GmbH Research & Development http://www.opensuse.org/ --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-packaging+help@opensuse.org
On Wed, 17 Oct 2007, Cristian Rodriguez wrote:
Richard Guenther escribió:
Hi,
this is a reminder for you to have a look at your packages that fail in the BETA distribution (which has GCC 4.3 as its compiler)
and it is quite a bit of work ;-) 438 packages fails atm at least on x86 ( but not all due to the compiler change)
I know ;) Which is the reason I ask for the packagers to do this work.
Otherwise I would have fixed them myself (I did so for a few important
packages).
Richard.
--
Richard Guenther
On Wed 17. Oct - 12:01:04, Richard Guenther wrote:
Hi,
this is a reminder for you to have a look at your packages that fail in the BETA distribution (which has GCC 4.3 as its compiler). If you think that the compiler is at fault feel free to report these issues back to me or file a bugzilla for them.
On a related note: What I'm seeing for quite a while now, not an error but a warning, is the following: char *prop = "laptop_panel.num_levels"; Compiler output: warning: deprecated conversion from string constant to 'char*' What would result in char *prop = (char*)"laptop_panel.num_levels"; Uhm, that's ugly ;-) What's the rationale behind this? Thanks, Holger --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-packaging+help@opensuse.org
On 2007-10-17 12:56:51 +0200, Holger Macht wrote:
What would result in
char *prop = (char*)"laptop_panel.num_levels"; const char *prop = "laptop_panel.num_levels";
should fix it too no? darix -- openSUSE - SUSE Linux is my linux openSUSE is good for you www.opensuse.org --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-packaging+help@opensuse.org
On Oct 17 2007 13:09, Marcus Rueckert wrote:
On 2007-10-17 12:56:51 +0200, Holger Macht wrote:
What would result in
char *prop = (char*)"laptop_panel.num_levels"; const char *prop = "laptop_panel.num_levels";
should fix it too no?
Yes. And if it is supposed to be writable, use char prop[]. --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-packaging+help@opensuse.org
On Wed 17. Oct - 13:09:30, Marcus Rueckert wrote:
On 2007-10-17 12:56:51 +0200, Holger Macht wrote:
What would result in
char *prop = (char*)"laptop_panel.num_levels"; const char *prop = "laptop_panel.num_levels";
should fix it too no?
Of course it does. I also have to change everything like: void testfunc(char *str); testfunc("foo"); to testfunc((char*)"foo"); I fully understand that this is correct along with the syntax, but I don't like it ;-) Regards, Holger --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-packaging+help@opensuse.org
On Wed, 17 Oct 2007, Holger Macht wrote:
On Wed 17. Oct - 13:09:30, Marcus Rueckert wrote:
On 2007-10-17 12:56:51 +0200, Holger Macht wrote:
What would result in
char *prop = (char*)"laptop_panel.num_levels"; const char *prop = "laptop_panel.num_levels";
should fix it too no?
Of course it does.
I also have to change everything like:
void testfunc(char *str); testfunc("foo"); to testfunc((char*)"foo");
No, the correct fix is to make testfunc take a const char *.
I fully understand that this is correct along with the syntax, but I don't like it ;-)
Well, it's only a warning ;) (for now!)
Richard.
--
Richard Guenther
On 2007-10-17 13:32:17 +0200, Richard Guenther wrote:
On Wed, 17 Oct 2007, Holger Macht wrote:
On Wed 17. Oct - 13:09:30, Marcus Rueckert wrote:
On 2007-10-17 12:56:51 +0200, Holger Macht wrote:
What would result in
char *prop = (char*)"laptop_panel.num_levels"; const char *prop = "laptop_panel.num_levels";
should fix it too no?
Of course it does.
I also have to change everything like:
void testfunc(char *str); testfunc("foo"); to testfunc((char*)"foo");
No, the correct fix is to make testfunc take a const char *.
and what should we do if the function is called with char* and string constants? darix -- openSUSE - SUSE Linux is my linux openSUSE is good for you www.opensuse.org --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-packaging+help@opensuse.org
* Marcus Rueckert
On 2007-10-17 13:32:17 +0200, Richard Guenther wrote:
On Wed, 17 Oct 2007, Holger Macht wrote:
On Wed 17. Oct - 13:09:30, Marcus Rueckert wrote:
On 2007-10-17 12:56:51 +0200, Holger Macht wrote:
What would result in
char *prop = (char*)"laptop_panel.num_levels"; const char *prop = "laptop_panel.num_levels";
should fix it too no?
Of course it does.
I also have to change everything like:
void testfunc(char *str); testfunc("foo"); to testfunc((char*)"foo");
No, the correct fix is to make testfunc take a const char *.
and what should we do if the function is called with char* and string constants?
When const char * is required, you can always specify char *. But not vice versa. Thanks, Bernhard --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-packaging+help@opensuse.org
participants (6)
-
Bernhard Walle -
Cristian Rodriguez -
Holger Macht -
Jan Engelhardt -
Marcus Rueckert -
Richard Guenther