240005 – app-emulation/spim: CFLAGS are ignored

Bug 240005 - app-emulation/spim: CFLAGS are ignored

Summary: app-emulation/spim: CFLAGS are ignored

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	New packages (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	Ricardo Mendoza (RETIRED)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2008-10-05 17:05 UTC by Diego Elio Pettenò (RETIRED)
Modified:	2009-12-22 19:53 UTC (History)
CC List:	1 user (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Patch enabling CFLAGS to be specified (use w/ updated ebuild below) (spim-7.3-flags.patch,628 bytes, patch) 2008-11-12 00:14 UTC, Colin Bennett	Details \| Diff
Patch supporting FEATURES=nostrip (use w/ ebuild below) (spim-7.3-nostrip.patch,454 bytes, patch) 2008-11-12 00:14 UTC, Colin Bennett	Details \| Diff
spim ebuild supporting FEATURES=nostrip and proper CFLAGS handling (spim-7.3.ebuild,2.39 KB, text/plain) 2008-11-12 00:16 UTC, Colin Bennett	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Diego Elio Pettenò (RETIRED) gentoo-dev

2008-10-05 17:05:33 UTC

I don't know the build system for this program (and this is a generic bug template so I cannot tell you which program exactly is), but my tests shows that it's not respecting CFLAGS properly.

Please look into it, since it's important to respect user CFLAGS.

Thanks,
Diego

Comment 1 Colin Bennett 2008-11-12 00:14:21 UTC

Created attachment 171480 [details, diff]
Patch enabling CFLAGS to be specified (use w/ updated ebuild below)

Comment 2 Colin Bennett 2008-11-12 00:14:50 UTC

Created attachment 171482 [details, diff]
Patch supporting FEATURES=nostrip (use w/ ebuild below)

Comment 3 Colin Bennett 2008-11-12 00:16:14 UTC

Created attachment 171484 [details]
spim ebuild supporting FEATURES=nostrip and proper CFLAGS handling

Updated app-emulation/spim-7.3 ebuild:

Support FEATURES=nostrip by patching out the '-s' option to install.  Use EAPI=2.  Properly pass CFLAGS and LDFLAGS to the build.

Comment 4 Łukasz Damentko (RETIRED) gentoo-dev

2009-03-17 21:54:45 UTC

Moving to maintainer-needed@ since compnerd left Gentoo (#71985).

Comment 5 Dror Levin (RETIRED) gentoo-dev

2009-12-22 19:53:24 UTC

Fixed in ebuild for version 7.5.