Re: [Slackbuilds-users] Modifying config.mk's CFLAGS variable

2022-04-24 Thread Brandon Pribula
On Sun, Apr 24, 2022, 7:40 AM B. Watson, wrote: > > > On Fri, 22 Apr 2022, Brandon Pribula wrote: > > > The solution I came up with is to change '=' to '+=' > > > > sed -i '/CFLAGS/s/=/+=/' config.mk > > > > and remove the optimization flag -Os > > > > sed -i '/CFLAGS/s/-Os//' config.mk > > > > T

Re: [Slackbuilds-users] Modifying config.mk's CFLAGS variable

2022-04-24 Thread B. Watson
On Fri, 22 Apr 2022, Brandon Pribula wrote: The solution I came up with is to change '=' to '+=' sed -i '/CFLAGS/s/=/+=/' config.mk and remove the optimization flag -Os sed -i '/CFLAGS/s/-Os//' config.mk This works fine but I would like to know if there is a more preferable solution with

Re: [Slackbuilds-users] Modifying config.mk's CFLAGS variable

2022-04-24 Thread Brandon Pribula
On Sat, Apr 23, 2022, 3:04 AM Tim Dickson via SlackBuilds-users, < slackbuilds-users@slackbuilds.org> wrote: > for longer term solution submit a patch upstream. you might want to check > the optimization flag because some applications don't run well/have runtime > issues with some flags. In those

Re: [Slackbuilds-users] Modifying config.mk's CFLAGS variable

2022-04-23 Thread Tim Dickson via SlackBuilds-users
for longer term solution submit a patch upstream. you might want to check the optimization flag because some applications don't run well/have runtime issues with some flags. In those cases you don't want to override them. (I have at least one package with that issue.) Regards Tim On 23/04/2022

[Slackbuilds-users] Modifying config.mk's CFLAGS variable

2022-04-22 Thread Brandon Pribula
Hi, I have a situation with an application I'm packaging. The CFLAGS variable, et al in config.mk can't be prepended. CFLAGS = -std=c99 ... instead of CFLAGS += -std=c99 ... so as is I can't prepend $SLKFLAGS to $CFLAGS. It also contains an optimization flag: -Os which conflicts with Slackware