Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#175 closed defect (fixed)

-Zhigh-mem

Reported by: Silvan Scherrer Owned by:
Priority: major Milestone:
Component: rpm Version:
Severity: low Keywords:
Cc: Dave Yeo

Description

Some rpm's don't have -Zhigh-mem and friends exported as LDFLAGS. We should take care that most if not all have it.

Eventually it's best to add the LDFLAGS to netlabs-rpmbuild-all.sh

Change History (4)

comment:1 Changed 8 years ago by Silvan Scherrer

related to #106

comment:2 Changed 8 years ago by dmik

Resolution: fixed
Status: newclosed

On the second thought I'm not really sure if it should be put in the script rather than in the .spec file itself. The build script is a completely separate entity and it controls the given distribution channel rules, not the OS-specific build specifics. Therefore I'm sure -Zhigh-mem logically belongs to .spec (i.e. should be explicitly specified in each .spec file) rather than to the build bot environment. Regardless of the distribution channel, -Zhigh-mem must always be used.

So I'm closing this.

Last edited 8 years ago by dmik (previous) (diff)

comment:3 Changed 8 years ago by Dave Yeo

Cc: Dave Yeo added

Should a package such as SDL use -Zhigh-mem?

comment:4 Changed 8 years ago by Silvan Scherrer

I looked at the spec and it uses high-mem switch. Or do you think it should not use it?

Note: See TracTickets for help on using tickets.